0

Условие и постановка задачи: Ломаная на плоскости задаётся последовательностью пар координат, определяющих положение точек этой ломаной на координатной плоскости. Для двух ломаных определяется сумма — ломаная, состоящая последовательно из точек первой ломаной и, затем, второй ломаной. Геометрически это можно представить соединением отрезком последней точки первой ломаной и первой точки второй.Задание. Написать класс "ломаная на плоскости" и оператор "сумма ломаных";

В качестве кода я сделал следующее: ввожу 2 массива произвольной длины, ну а дальще перегрузкой оператора "+" объединяю их. НО, возникает ошибка "double free detected in tcache 2".

#include <iostream>
using namespace std;
int i=0;
class Array
{
private:
    int size;
   unsigned int* data;
public:
// конструктор
    Array(){}
    Array(int asize) {
        size=asize;
        data=(unsigned int*)new int[size];
        if(data) cout<<"Massiv inicializir\n";
        else cout<<"ne hvatka pamiati\n";
    }
~Array(){
    delete[] data;
    cout<<"Object uni4tojen\n";
}
        //Далее методы
Array(const Array &ob)   //Определение конструктора //копирования
{                           
 size=ob.size;
 data=(unsigned int*)new int [size];
 for(int i=0;i<ob.size;i++)
  data[i]=ob.data[i];
}
void print(){
    for(i=0;i<size;i++){
    cout<<*(data+i)<<" ";
}
cout<<endl;
}

void scan(){
cout<<"Vvedite massiv iz:"<<size<<"simvolov \n";
for(int i=0;i<size;i++){
cin>>*(data+i);
}
}
Array operator+(const Array& ob){
    this->size=size;
    Array tmp(size+ob.size);
    for(i=0;i<size;i++) 
    {
        tmp.data[i]=*(data+i);//this->data[i]=data[i];
    }
    for(i=size;i<size+ob.size;i++) tmp.data[i]=ob.data[i-size];
 return tmp;
}
};
int main(){
    Array first(3);
    Array second(2);
    first.scan();
    second.scan();
    Array third(5);
    third=first+second;
    third.print();
 return 0;
}

Прошу объяснить Почему возникла эта ошибка(скорее всего всё из-за выделения доп.памяти) и способ ее исправления. Спасибо.

Кстати, я задавал ранее вопрос про Ломаную,условие такое же, но код не подходящий, поэтому я решил задать отдельный вопрос про эту задачу,но уже с нужным мне кодом.

2
  • 1
    Возможный дубликат вопроса: Задача Ломаная на плоскости 8 авг 2022 в 8:33
  • @Эникейщик Да, но там уже есть ответ,поэтому сделал новый. И в новом вопросе снизу отредактировал этот момент.
    – Stan
    8 авг 2022 в 9:13

1 ответ 1

1

У вас нет оператора присваивания, а само присваивание есть:

third=first+second;

Соответственно, он генерируется компилятором, при этом выполняется простое почленное присваивание (т.е. просто присваивание указателей, а не копирование содержимого памяти), что в результате вызывает утечку памяти (память объекта third не удаляется) + приводит к освобождению одной и той же памяти дважды (временного объекта, создаваемого при суммировании, и объекта third).

Помните о правиле большой тройки :) — если уж работаете с памятью — деструктор, копирующий конструктор и оператор присваивания обязательны...

Рекомендую идиому присваивания через копирование: пишется функция swap для обмена членов двух объектов, потом создается временный объект, обмен с ним — и все готово. В вашем варианте примерно так:

void swap(Array& a) {
    std::swap(size,a.size);
    std::swap(data,a.data);
}

Array& operator=(const Array& a)
{
    if (this != &a) { // Необязательно!!
        Array tmp(a);
        swap(tmp);
    }
    return *this;
}
9
  • Спасибо, без Вас долго бы еще мучился. Не моги бы объяснить построчно что там делается? Почему нужен обмеен двух членов объектов? - @Harry
    – Stan
    8 авг 2022 в 9:17
  • Это просто очень удобная идиома. Создаем временную копию того, что присваивается (конечно, если есть правильно написанный конструктор :)) Далее — очень быстрый обмен — просто указателей. Теперь наш объект становится копией, а временный — оригиналом :). Каковой корректно автоматически уничтожается. При этом корректно отработает даже присваивание самому себе, так что такая проверка нужна не для правильности работы, а для скорости.
    – Harry
    8 авг 2022 в 9:37
  • Мне еще интересно, есть ли какой то другой способ, более традицонный? Нам просто про такой не рассказывали. ---@Harry
    – Stan
    8 авг 2022 в 10:24
  • Другой? Уничтожить текущий объект, скопировать присваиваемый. Т.е. по сути продублировать деструктор и конструктор копирования, что в указанном методе делается и так.
    – Harry
    8 авг 2022 в 10:51
  • 1
    А вы подумайте, что будет, если его не делать :)
    – Harry
    8 авг 2022 в 12:02

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.