Уже несколько дней пытаюсь найти ошибку, все никак не получается :(. При этом смотрю на пример, который мы делали с другом, делаю в этой работе все также, но что-то не так.
Привожу фрагмент кода:
class stack{ // класс "стек"
public:
class cars{ //класс машины (марки машин)
public:
string mark;
string color;
string adress;
int price;
cars(){} // конструктор умолчания
cars(string m, string c, int p, string a){ // конструктор инициализации
mark = m;
color = c;
adress = a;
price = p;
}
cars operator=(cars cc){ // оператор присваивания машин
mark = cc.mark;
color = cc.color;
adress = cc.adress;
price = cc.price;
return *this;
}
void print(){ // вывод элементов на уровне класса "машины"
cout << endl << mark << ", цвет: " << color << ", цена: " << price << " адрес: " << adress << endl;
}
~cars(){} // деструктор
};
stack (){ // конструктор умолчания
car = NULL;
size = 0;
}
int size = 0; // количество машин в стекек (сначала 0)
cars *car; // машины в стеке
void push(){ // добавление элементов
int price;
string mark, color, adress, pric;
cout << "\nВведите марку машины: ";
cin >> mark;
cout << "\nВведите цвет машины: ";
cin >> color;
cout << "\nВведите цену: ";
cin >> pric;
price = iscorrect(pric);
cout << "\nВведите адрес: ";
cin.ignore();
getline(cin, adress);
cars ccc(mark, color, price, adress);
ccc.print();
size++;
if (size == 1) car = (cars*)malloc(1 * sizeof(cars)); // если в стэке пока нет машин, назначить память для первой
else car = (cars*)realloc(car, size * sizeof(cars)); //иначе назначит на текущее количество + следующий элемент
car[size - 1]=ccc;
cout << "\nЭта машина успешно добавлена в список\n";
}
void pop() { // удаление элементов
car[size - 1].print();
size--;
car = (cars*)realloc(car, (size) * sizeof(cars)); //переназначение памяти
cout << "\nЭта машина успешно удалена.\n";
}
class stack{ // класс "стек"
public:
class cars{ //класс машины (марки машин)
public:
string mark;
string color;
string adress;
int price;
cars(){} // конструктор умолчания
cars(string m, string c, int p, string a){ // конструктор инициализации
mark = m;
color = c;
adress = a;
price = p;
}
cars operator=(cars cc){ // оператор присваивания машин
mark = cc.mark;
color = cc.color;
adress = cc.adress;
price = cc.price;
return *this;
}
void print(){ // вывод элементов на уровне класса "машины"
cout << endl << mark << ", цвет: " << color << ", цена: " << price << " адрес: " << adress << endl;
}
~cars(){} // деструктор
};
stack (){ // конструктор умолчания
car = NULL;
size = 0;
}
int size = 0; // количество машин в стекек (сначала 0)
cars *car; // машины в стеке
void push(){ // добавление элементов
int price;
string mark, color, adress, pric;
cout << "\nВведите марку машины: ";
cin >> mark;
cout << "\nВведите цвет машины: ";
cin >> color;
cout << "\nВведите цену: ";
cin >> pric;
price = iscorrect(pric);
cout << "\nВведите адрес: ";
cin.ignore();
getline(cin, adress);
cars ccc(mark, color, price, adress);
ccc.print();
size++;
if (size == 1) car = (cars*)malloc(1 * sizeof(cars)); // если в стэке пока нет машин, назначить память для первой
else car = (cars*)realloc(car, size * sizeof(cars)); //иначе назначит на текущее количество + следующий элемент
car[size - 1]=ccc;
cout << "\nЭта машина успешно добавлена в список\n";
}
void pop() { // удаление элементов
car[size - 1].print();
size--;
car = (cars*)realloc(car, (size) * sizeof(cars)); //переназначение памяти
cout << "\nЭта машина успешно удалена.\n";
}
clang version 7.0.0-3~ubuntu0.18.04.1 (tags/RELEASE_700/final) Весь можно запустить здесь: https://repl.it/repls/DarksalmonCrookedPaint#main.cpp
В чем сама проблема: почему-то только если добавить один элемент, потом сразу удалить его, после попытки добавления еще одного элемента выходит ошибка сегментации.
Но дело в том, что если изначально добавить два элемента и больше, никакая ошибка не появляется: хоть добавь два, потом удали их, опять же как выше добавь один и сразу удали его. Все будет хорошо. Привожу скриншоты:
Вот первый случай:
В среде Visual Studio 19 ошибка такая:
Кажется, моих неглубоких знаний не хватит, чтобы понять суть этой проблемы, поэтому я очень рассчитываю на помощь. Спасибо.
realloc
и С++ (не POD-типы), мягко говоря, очень взрывоопасная смесь. Тем более что вы нигде не проверяете его результат.