Pole stringů/struktur ve třídě – C / C++ – Fórum – Programujte.com

Zdravím, mám menší problém s třídou "telefoní seznam"

typedef struct contact{
        string jmeno;
        string adresa;
        string telefon;
} contact;
class CPhoneBook {
private:
        contact * kontakt;
        int size;  //počet položek v seznamu
        int max;  // velikost pole kontaktů
public:
....
}


CPhoneBook::CPhoneBook( void ){
        size=0;
        max=1000;
        kontakt = ( contact*) malloc ( max * sizeof ( *kontakt ));
}
bool CPhoneBook::Add( const string & name, const string & address, const string & phone){
        kontakt[size].jmeno=name;
        kontakt[size].adresa=address;
        kontakt[size].telefon=phone;
        size++;
        return true;

program při použití metody .Add spadne, neví někdo, v čem může být problém?zkoušel jsem to i bez struktur a místo toho ve tříde 3 pole stringu, dynamicky alokovaný, ale taky to nešlo :-/

Ukázka

#include <iostream>

using namespace std;

typedef struct
{
    char* name;
    int age;
} Osoba;

class Pole
{
    int top;
    int max;
    Osoba* osoby;

public:
    Pole(int maxSize) : max(maxSize), top(0)
    {
        osoby = new Osoba[maxSize];
    }
    void add(char* name, int age)
    {
        Osoba* os = new Osoba();
        
        strcpy(os->name, name);
        os->age = age;
        *(osoby + top++) = *os;
    }
};

int main()
{
    Pole p(10);

    p.add("jmeno1", 10);
    p.add("jmeno2", 20);

    return 0;
}

Jinak něco jde vyřešit líp.. určitě by byla lepší nějaká hotová kolekce, místo vlastního pole (pokud se jedná o takovou knihu)
Udělal jsem ještě na rychlo jeden příklad ve smyslu tý knihy... si z toho můžeš odvodit co máš špatně.. jinak nechápu, proč vlastně používáč C syntax v C++ (třeba ta alokace) 

#include <iostream>
#include <string>

using namespace std;

typedef struct sHuman
{
    string Title;
    string Name;
    string Address;

    sHuman() { }
    sHuman(string title, string name, string address)
    {
        Title = title;
        Name = name;
        Address = address;
    }
} Human;

class Book
{
    Human** m_people;
    int m_capacity;
    int m_current;

public:
    Book(const int capacity) : m_capacity(capacity), m_current(0)
    {
        *m_people = new Human[capacity];
    }

    ~Book()
    {
        clear();
    }

    bool add(Human* human)
    {
        if (m_current < m_capacity)
        {
            *(m_people + m_current++) = human;
            return true;
        }
        return false;
    }

    bool remove(const int index)
    {
        if (m_current >= 0)
        {
            delete *(m_people + index);

            for (int i = index; i < m_current - 1; i++)
                *(m_people + i) = *(m_people + i + 1);

            m_current--;
            return true;
        }
        return false;
    }

    void clear()
    {
        for (int i = 0; i < m_current; i++)
            delete *(m_people + i);

        m_current = 0;
    }

    void outPeople() const
    {
        for (int i = 0; i < m_current; i++)
        {
            Human* h = *(m_people + i);
            cout << "Jmeno: " << h->Title << " " << h->Name << " - " << h->Address << endl;
        }
    }

    int Capacity() const { return m_capacity; }
    int CurrentIndex() const { m_current; }
};

int main()
{
    Book book(2);
    book.add(new Human("Ing","Jan","Praha 1"));
    book.add(new Human("Mgr","Karel","Praha 2"));

    if (!book.add(new Human("Ing","Petr","Praha 3")))
    {
        cout << "Kniha je plna!\n";
    }
    book.remove(0);
    book.add(new Human("Bc", "Lenka", "Praha 4"));

    book.outPeople();
    book.clear();
    book.outPeople();

    return 0;
}

string je objekt s vlastním operator=, který se při tom přiřazení vyvolá a protože tam nemáš platný objekt, tak to pochopitelně nefunguje. Na alokaci s vytvořením objektů je v C++ operátor new a new[].

kontakt = new contact[max];

A mimochodem je v C++ taky třeba vector, mrkni na to.

Tak ako pise zlz, treba alokovat pomocou new, new[].

Rozdiel medzi new a malloc je hlavne volanie konstruktorov (pri mazani destructorov). malloc nevola knstructor, new vola.

Na prvkovy v telefonnom zozname nebudes potrebovat nahodny pristup, takze bude lepsie pouzit zretazeny zoznam napr: list.

V c++ je struct uplnym datovym typom ako trieda, takze toto je zbytocne:

typedef struct contact{
        string jmeno;
        string adresa;
        string telefon;
} contact;

Rovno moze napisat:

struct contact{
        string jmeno;
        string adresa;
        string telefon;
};