Удаление экземпляра класса в функции самого класса (Ошибка при отладке)

@Polenoff · Регистрация: 26.09.2015

Допустим, у нас есть класс Buffer, который хранит в себе указатель на класс некоторой матрицы и количество этих указателей refcounter

Buffer.h

C++

class matrix;
class buffer
{
public:
    const matrix* buf;
    int refcounter=0;
    buffer(const matrix *link);
    void addref(); //увеличивает refcounter на единицу
    void decref(); //уменьшает refcounter на единицу
};

Buffer.cpp

C++

buffer::buffer(const matrix *link)
{
    if (refcounter == 0)
    {
        buf = new matrix;
        buf = link;
    }
    addref();
}
void buffer::addref() 
{
    refcounter++;
}; 
void buffer::decref()
{
    refcounter--;
    if (refcounter == 0)
        delete this;
}

Если программа доходит до самоудаления экземпляра класса в функции decref, то вылезает debug-ошибка

@0x0152 · 03.11.2015, 22:57

Так делать нельзя. После вызова delete вызывается функция free, в стек ложится адрес возврата который указывает на конец функции decref. После очистки происходит возврат по адресу возврата, но так как там нечего нет, то выходит ошибка.(могу ошибаться

)

Не по теме:

Матрица развратница :D

@castaway · 03.11.2015, 23:01

Polenoff, может быть ты перепутал this с buf!?

@Polenoff · 03.11.2015, 23:13 **[ТС]**

С buf тоже пробовал - такая же ошибка

@Kastaneda · 03.11.2015, 23:16

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

С buf тоже пробовал - такая же ошибка

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

buf = new matrix;
buf = link;

адрес затирается, утечка + по delete удаляется не пойми что.

Добавлено через 52 секунды
И я так понимаю там подразумевался конструктор копирования, параметр должен быть ссылкой, а не указателем.

@Polenoff · 04.11.2015, 01:06 **[ТС]**

В конструктор мы передаём некоторый указатель на матрицу, который мы храним в классе Buffer

Почему затирается? Мы же просто присваиваем один указатель другому. Память как была, так и осталась. Цель состоит в том, чтобы в decref как раз удалить этот присвоенный указатель link, который мы положили в Buffer

@0x0152 31 / 31 / 23 Регистрация: 23.03.2015 Сообщений: 210
	03.11.2015, 22:57	2
	Так делать нельзя. После вызова delete вызывается функция free, в стек ложится адрес возврата который указывает на конец функции decref. После очистки происходит возврат по адресу возврата, но так как там нечего нет, то выходит ошибка.(могу ошибаться ) Не по теме: Матрица развратница :D 0

@castaway 4978 / 3085 / 456 Регистрация: 10.11.2010 Сообщений: 11,164 Записей в блоге: 10
	03.11.2015, 23:01	3
	Polenoff, может быть ты перепутал `this` с `buf`!? 0

@Polenoff 2 / 2 / 1 Регистрация: 26.09.2015 Сообщений: 85
	03.11.2015, 23:13 [ТС]	4
	С buf тоже пробовал - такая же ошибка 0

@Kastaneda 5225 / 3197 / 362 Регистрация: 12.12.2009 Сообщений: 8,101 Записей в блоге: 2
	03.11.2015, 23:16	5
	Сообщение от Polenoff С buf тоже пробовал - такая же ошибка Сообщение от Polenoff buf = new matrix; buf = link; адрес затирается, утечка + по delete удаляется не пойми что. Добавлено через 52 секунды И я так понимаю там подразумевался конструктор копирования, параметр должен быть ссылкой, а не указателем. 0

@Polenoff 2 / 2 / 1 Регистрация: 26.09.2015 Сообщений: 85
	04.11.2015, 01:06 [ТС]	6
	В конструктор мы передаём некоторый указатель на матрицу, который мы храним в классе Buffer Почему затирается? Мы же просто присваиваем один указатель другому. Память как была, так и осталась. Цель состоит в том, чтобы в decref как раз удалить этот присвоенный указатель link, который мы положили в Buffer 0

Опции темы