Creating an array of objects c++

Question

I'm trying to create an array (c++) that consists of 4 objects and I'm using this syntax, something is obviously wrong, but what?

for (int octet = 0; octet < 4; octet++) {
        cout << "OCTET NO." << octet << endl;
        cout << "IP:     "; cin >> ip;
        cout << "Subnet: "; cin >> subnet;

        networkOctet[octet] = networkOctet(ip, subnet); //The line where the problem is

        }

Thank you for your help!

!-- UPDATE --!

Allright, so I changed the code to this but now it says "no matching constructor for initialization of 'networkOctet[4]'"... I have a constructor that is declared, defined and works perfectly on objects that are not in arrays.

Updated code:

int ip;

int subnet;

networkOctet networkOctetObject[4];

for (int octet = 0; octet < 4; octet++) {
    cout << "OCTET NO." << octet << endl;
    cout << "IP:     "; cin >> ip;
    cout << "Subnet: "; cin >> subnet;
    if (octet == 3) {
        networkOctetObject[octet] = networkOctet(ip, subnet, true);
    }
    else {
        networkOctetObject[octet] = networkOctet(ip, subnet, false);
    }
}

Change one of the networkOctet to something like getNetworkOctet — user2249683
– user2249683, Commented Feb 14, 2014 at 18:00
I'm assuming networkOctet is the class name. You can't use class name as an array variable — G one
– G one, Commented Feb 14, 2014 at 18:00
this question is like a really bad trick question...... is that an array variable name? is that a class name? is it something with [] and () overloaded? is it a mistake? — thang
– thang, Commented Feb 14, 2014 at 18:05

Luchian Grigore · Accepted Answer · 2014-02-14 18:04:44Z

1

I doubt you overloaded both [] and () on a type decltype(networkOctet), so the problem is that you're treating the name as both a variable networkOctet[octet] and a type networkOctet(ip, subnet). If your variable of type X[4] is named y, you'd use the following syntax to make this work:

y[octet] = X(ip, subnet);

where y itself is declared as X y[4].

answered Feb 14, 2014 at 18:04

Luchian Grigore

260k68 gold badges465 silver badges634 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Axel Kennedal Over a year ago

Allright, so I changed the code to this but now it says "no matching constructor for initialization of 'networkOctet[4]'"... I have a constructor that is declared, defined and works perfectly on objects that are not in arrays. @LuchianGrigore

Luchian Grigore Over a year ago

@AxelKennedal-TechTutor is it a default constructor?

sleepy1771 · Accepted Answer · 2014-02-14 18:53:39Z

You can only create an array of types that do have a default constructor. If you defined your own constructor and did not add a default constructor then you cannot create an array of this type because the compiler tries to value-initializes all elements (using the default constructor) in the array when the array is created. If you drop the default constructor the elements in the array cannot be value-initialized (because there is no default constructor) and your mentioned compiler error appears.

When the compiler sees the line

networkOctet networkOctetObject[4];

it tries to create 4 objects of type networkOctet. To create those objects a constructor of networkOctet must be called. As you create an array the compiler tries to call the default constructor that is not available. This is the line where the error occures. So to make your code work add a default constructor by adding

networkOctet() {
    // Initialize a networkOctet to a valid default value
}

to your networkOctet class (notice that this default constructor must be public).

Collectives™ on Stack Overflow

Creating an array of objects c++

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related