Не работает деструктор. В чем ошибка?

Question

[BelDm]

Простенький код создания объекта класса (полином к примеру). За конструктором следует деструктор очистки памяти, все по букварю. Код (приведен ниже) написан в Code::Blocks. В иных средах работает.

#include <bits/stdc++.h>

using namespace std;

class Polinomial
{
private:
    //Свойства класса
    int degree = 0;
    double *coefficient = NULL;

public:
    //Конструктор
    Polinomial (double *arr, int x)
    {
        degree = x - 1;
        coefficient = (double*)malloc(sizeof(double)*x);
        for (int i = 0; i < x; i++)
        {
            coefficient[i] = arr[i];
        }
    }

    //Деструктор
    ~Polinomial()
    {
        if (coefficient != NULL)
        {
            free (coefficient);
        }
    }

    friend ostream & operator << (ostream & out, Polinomial &p);
};

ostream & operator << (ostream & out, Polinomial &p)
{
    for (int i = 0; i <= p.degree; i++)
    {
        out << p.coefficient[i] << " ";
    }

    return out;
}

int main() {

    double arr1[] = {1.0, 3.0, 5.0};
    Polinomial pol1(arr1, 3);
    cout << pol1;


    return 0;
}

Comments

[Wexter]

а с чего решили что деструктор не работает?

Answer 1

[jcmvbkbc]

Не работает деструктор

"Не работает деструктор" -- это когда программу компилируешь, запускаешь, а деструктор не делает того, что в нём написано. У тебя же программа не компилируется. Из-за того, что в настройках компилятора стоит "трактовать предупреждения как ошибки" и "предупреждать о неэффективном коде С++". Ну и вдобавок, предупреждение тебе правильно говорит, что если ты не реализовал конструктор копирования и оператор присваивания для класса самостоятельно управляющего памятью, то жди проблем, когда начнёшь копировать объекты этого класса.

Comments

[BelDm]

Спасибо! Для меня за отсутствием опыта совсем не очевидно, что подобные вещи в тесной связке с конструктором копирования и оператором присваивания. Сейчас реализуем.

[Wataru]

BelDm, ну читайте же сообщение об ошибке. Там явно сказано, какие штуки классу надо определить и почему (указатели в классе - их просто копировать скорее всего опасно).

[BelDm]

Wataru, добавил необходимое, но...(((
#include

using namespace std;

class Polinomial
{
private:
//Свойства класса
int degree = 0;
double *coefficient = NULL;

public:
//Конструктор
Polinomial (double *arr, int x)
{
degree = x - 1;
coefficient = (double*)malloc(sizeof(double)*x);
for (int i = 0; i < x; i++)
{
coefficient[i] = arr[i];
}
}

//Конструктор копирования
Polinomial (Polinomial &p)
{
degree = p.degree;
coefficient = (double*)malloc(sizeof(double)*(degree + 1));
for (int i = 0; i < degree; i++)
{
coefficient[i] = p.coefficient[i];
}
}

//Перегруженный оператор присваивания
Polinomial operator = (const Polinomial &p)
{
if (this != &p)
{
degree = p.degree;
coefficient = (double*)realloc(coefficient, sizeof(double)*(degree + 1));
for (int i = 0; i < degree; i++)
{
coefficient[i] = p.coefficient[i];
}
} else
{
cout << "Самоприсваивание" << endl;
}
return *this;
}

//Деструктор
~Polinomial()
{
if (coefficient != NULL)
{
free(coefficient);
}
}

friend ostream & operator << (ostream & out, Polinomial &p);
};

ostream & operator << (ostream & out, Polinomial &p)
{
for (int i = 0; i <= p.degree; i++)
{
out << p.coefficient[i] << " ";
}

return out;
}

int main() {

double arr1[] = {1.0, 3.0, 5.0};
double arr2[] = {1.1, 3.1, 5.1, 6.1};
Polinomial pol1(arr1, 3);
Polinomial pol2(arr2, 4);
//pol1 = pol2;
cout << pol2;

return 0;
}

[Wataru]

BelDm, ну там же явно скзано, что оператор поисваивания должен возвращать ссылку на this. А у вас тип возвращаемого значения - просто объект, а не ссылка. Амперсант туда воткните.

[BelDm]

Wataru, я - дуб)))

[BelDm]

Wataru, спасибо!

Answer 2

[vanyamba-electronics]

Если в деструкторе выполняется очистка памяти, то его следует объявлять виртуальным:
virtual ~Polynomial() { ... }