basic c++ Declaration of an array of objects inside another class

[Shashank Setty]

Hello!
i've been working on a code and I tried to declare an array of objects inside the private of another class,
I have a class called Contacts which goes like this,

class Contacts
{
 private:
 char Name[50], Num[10];
 public:
 void InputContact()
 {
 }
 void OutputContact()
 {
 }
};

And I have another class called Addressbook which goes like this:

class Addressbook
{
 private:
 static n;
 Contacts rec[n=1];
 public:
 void AddContact()
 {
 }
 void DisplayContact()
 {
 }
};

But when I compile this code I get the following error:

D:\C++ Project\test.cpp|49|error: data member may not have variably modified type `Contact[(((unsigned int)((AddressBook::n <unknown operator> 1), 0)) + 1u)]'|

How do I declare an array of objects inside another class?

[mutexe]

http://www.informit.com/articles/article.aspx?p=1852519

[Shashank Setty]

I'm sorry, but that didn't help can someone please tell me how to declare/initialize an array of objects of a class inside another class?

[coder777]

Shashank Setty wrote:

I'm sorry, but that didn't help can someone please tell me how to declare/initialize an array of objects of a class inside another class?

That's not your problem. On line 4/5 of Addressbook you try to declare a dynamic array. This is not how it works. Use vector instead:

http://www.cplusplus.com/reference/vector/vector/?kw=vector

[48T7M4Gy]

class Contact
{
private:
    char* name;
    char* number;
    
public:
    
    Contact()
    {
    }
    
    Contact(char* nme, char* num)
    {
        name = nme;
        number = num;
    }
    
};

class AddressBook
{
private:
    Contact* contacts;

public:
    AddressBook(Contact* cs)
    {
        contacts = cs;
    }
    
    void addContact( Contact* c)
    {
        contacts = c;
    }
};

int main()
{
    const int limit = 50;
    Contact *contacts = new Contact[limit];
    
    AddressBook addbook = AddressBook(contacts);
    
    char name[50] = "Bob";
    char num[10] = "2143";
    
    contacts[1] = Contact(name, num);
    
    addbook.addContact(&contacts[1]);
    
    return 0;
}

This needs to be refined a bit and there are a couple of bloopers but the essence is there. The 'essence' here means you don't create the actual array in the addressbook but refer to it by pointers. It becomes cumbersome to keep track of entries so, vectors aren't a bad move unless you want the challenge. You could also do it with a linked list of your own design.

[Shashank Setty]

Thank you so much @coder777 and @kemort I fixed it :)