Не могу понять, где ошибка(

[IROKEZ]

Пытаюсь сделать двунаправленный ассоциативный контейнер, хранящий структуры. Собственно до того, чтобы он был ассоциативным далеко, но пока сделал создание самого массива со структурами. Однако программа падает (по-видимому где-то утечка памяти), но где именно и как исправить не пойму.

#include <iostream>
#include <assert.h>
using namespace std;
 
class Massive
{
public:
    Massive();
    ~Massive();
    void getElement(int N);
    void setElement(char *NAME, int num, int N);
    void setSize(int N);
    void getSize(int &N);
private:
    struct PhoneBook {
        char *name;
        int number;
    } *PhoneBookTab;
    int N;
};
 
Massive::Massive()
{
    N=0;
    PhoneBookTab=NULL;
}
 
Massive::~Massive()
{
    for(int i=0; i<N; i++)
        delete []PhoneBookTab;
}
 
void Massive::getElement(int n) // Функцию вывода элементов пока не писал
{
 
}
 
void Massive::setElement(char *NAME, int num, int n)
{
    assert(n>=0 && n<=N);
    PhoneBookTab[n].name=NAME;
    PhoneBookTab[n].number=num;
}
 
void Massive::setSize(int lenght)
{
    assert(lenght>=0);
    for(int i=0; i<N; i++)
        delete[] PhoneBookTab;
    PhoneBook *PhoneBookTab = new PhoneBook[lenght];
    N=lenght;
}
 
void Massive::getSize(int &lenght)
{
    N=lenght;
}
 
int main()
{
    Massive tab;
    int n, num;
    char *NAME;
 
    cout<<"Input number: ";
    cin>>n;
 
    tab.setSize(n);
    for(int i=0; i<n; i++)
    {
        cout << "Input name of abonent: ";
        cin >> NAME;
        cout << "Input tel number: ";
        cin >> num;
        tab.setElement(NAME, num, i);
    }
 
    return 0; 
}

[netrino]

Massive::~Massive()
{
    for(int i=0; i<N; i++)
        delete []PhoneBookTab;
}

Вы пытаетесь удалить одну и ту же память N раз. Уберите цикл, оставьте только delete[] PhoneBookTab;. В setSize тоже

[IROKEZ]

Сообщение от netrino

Вы пытаетесь удалить одну и ту же память N раз. Уберите цикл, оставьте только delete[] PhoneBookTab;. В setSize тоже

Убрал, но проблема осталась...

[netrino]

Действительно, ошибок там куда больше)

#include <iostream>
#include <assert.h>
using namespace std;
 
class Massive
{
public:
	Massive();
	~Massive();
	
	void getElement(int N);
	void setElement(char *NAME, int num, int N);
	void setSize(int N);
	void getSize(int &N);
private:
	struct PhoneBook {
		char *name;
		int number;
	} *PhoneBookTab;
	
	int N;
};
  
Massive::Massive()
{
	N=0;
	PhoneBookTab=NULL;
}
 

// !!!
Massive::~Massive()
{
	delete []PhoneBookTab;
}
    
void Massive::getElement(int n) // Функцию вывода элементов пока не писал
{
	 
}


void Massive::setElement(char *NAME, int num, int n)
{
	assert(n>=0 && n<=N);
	PhoneBookTab[n].name=NAME;
	PhoneBookTab[n].number=num;
}

// !!!
void Massive::setSize(int lenght)
{
	assert(lenght>=0);
	delete[] PhoneBookTab;
	PhoneBookTab = new PhoneBook[lenght]; // !
	N=lenght;
}
	  
// !!!
void Massive::getSize(int &lenght)
{
	length = N;
}
	    
int main()
{
    Massive tab;
    int n, num;
    char NAME[100]; // !! Выделение буфера, куда будет производится чтение.
						 
    cout << "Input number: ";
    cin  >> n;
	
    tab.setSize(n);
    for(int i=0; i<n; i++)
    {
        cout << "Input name of abonent: ";
        cin >> NAME;
        cout << "Input tel number: ";
        cin >> num;
        tab.setElement(NAME, num, i);
    }

    return 0; 
}

Вот исправленный код. Я пометил восклицательными знаками функции, в которые я внёс изменения. Изменил только чтобы перестало ругаться

[IROKEZ]

netrino
Большое спасибо!