VPF::Вектор и деструктор - Форум программистов

B3cK

Дата 30.5.2006, 16:03 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 49
Регистрация: 29.5.2005
Где: Минск

Репутация: 1
Всего: 1

Код


class A{
char *p1;
char *p2;
....
//bla bla bla
public:
A(char* a, char* b)
{
p1=new char[strlen(a)+1];
strcpy (p1,a); 
p2=new char[strlen(b)+1];
strcpy (p2,b);
}
~A(){delete[]p1;
         delete[]p2;
        }
....
//bla bla bla
};

При добавлении нового объекта данного класса в вектор вылетает ошибка "Debug Assertion Failed!".
Компилятор VC6++ ошибок не выводит. Если деструктор закоментировать, то все отлично работает.
Как можно объяснить такое поведение деструктора при добавлении объекта в вектор?

maxim1000

Дата 30.5.2006, 16:12 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Участник
Сообщений: 3334
Регистрация: 11.1.2003
Где: Киев

Репутация: 17
Всего: 110

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

--------------------

qqq

B3cK

Дата 30.5.2006, 16:39 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 49
Регистрация: 29.5.2005
Где: Минск

Репутация: 1
Всего: 1

Подожди, какое копирование, присваивание.
Я через функцию push_back() помещаю в конец вектора
новый полностью инициализированный объект.

maxim1000

Дата 30.5.2006, 17:15 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Участник
Сообщений: 3334
Регистрация: 11.1.2003
Где: Киев

Репутация: 17
Всего: 110

Цитата(B3cK @ 30.5.2006, 15:39

)

Я через функцию push_back() помещаю в конец вектора
новый полностью инициализированный объект

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

--------------------

qqq

MAKCim

Дата 30.5.2006, 17:15 (ссылка)

(нет голосов)

Загрузка ...

Воін дZэна

Профиль
Группа: Экс. модератор
Сообщений: 5644
Регистрация: 10.12.2005
Где: Менск, РБ

Репутация: 52
Всего: 207

Цитата

создается копия объекта, а т к у тебя не определен копирующий конструктор и ты работаешь с чистыми указателями, возможны проблемы
напиши

Код


class A
{
...
public:
    A(const A& obj)
        : p1(new char [strlen(obj.p1)+1]), p2(new char [strlen(obj.p2)+1])
    {
        strcpy(p1,obj.p1);
        strcpy(p2,obj.p2)
    }
};

--------------------

B3cK

Дата 30.5.2006, 17:28 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 49
Регистрация: 29.5.2005
Где: Минск

Репутация: 1
Всего: 1

Спасиб, теперь конфликт понятен.
Вопрос закрыт.

Earnest

Дата 30.5.2006, 17:37 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Экс. модератор
Сообщений: 5962
Регистрация: 17.6.2005
Где: Рязань

Репутация: 53
Всего: 183

Оператор присваивания тоже нужен. Самый простой способ проверить - декларировать в private-секции оператор присваивания и выслушать от компилятора массу приятных слов.

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

--------------------

...

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »