Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.80/5: Рейтинг темы: голосов - 5, средняя оценка - 4.80
 Аватар для Foxa
0 / 0 / 0
Регистрация: 24.05.2010
Сообщений: 27

Очередное деструктивное: при вызове деструктора все падает

27.11.2014, 10:27. Показов 1237. Ответов 18
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Доброго времени суток! Есть некий класс H, с конструктором, деструктором:

C++
1
2
3
4
5
6
7
H(int size, int top){
this->size = size;
this->top = top;
this->set = new int[size];
}
 
~H(){ delete[] this->set; }
Объект создается, все хорошо, но при выходе из main, т.е. при вызове деструктора все падает на удалении массива. Не пойму,почему, ведь создан new[], значит и удалять delete[].
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
27.11.2014, 10:27
Ответы с готовыми решениями:

Ошибка при вызове деструктора (при вызове delete в деструкторе)
Не могу найти ошибку, при вызове деструктора от класса, который был создан при помощи конструктора с параметром const String &,...

Ошибка при вызове деструктора
Подскажите, что не так с этим деструктором? Просто выдает ошибку. Если его убираю то все идет норм. Если же оставляю, то куча всяких...

Ошибка при вызове деструктора
Здравствуйте, помогите разобраться с проблемой, она довольно распространенная и на форуме куча сообщений про это но в моей программе я...

18
В астрале
Эксперт С++
 Аватар для ForEveR
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
27.11.2014, 10:29
Foxa, Нужно больше кода.
0
654 / 575 / 164
Регистрация: 13.12.2012
Сообщений: 2,124
27.11.2014, 10:34
C++
1
~H(){ delete[] (this->set;) }
а так?

Добавлено через 25 секунд
и покажите создание объекта в мейне)
0
Эксперт С++
2924 / 1274 / 114
Регистрация: 27.05.2008
Сообщений: 3,465
27.11.2014, 10:37
Foxa, приведи код конструктора копирования и оператора=.
0
В астрале
Эксперт С++
 Аватар для ForEveR
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
27.11.2014, 10:37
Вцелом я почти уверен, что в коде не соблюдено "Правило трех" и создается копия этого объекта на стеке.
0
27.11.2014, 10:37

Не по теме:

ForEveR, кстати да

0
 Аватар для taras atavin
4226 / 1796 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
27.11.2014, 10:39
Foxa, теговать Билл будет?
0
120 / 142 / 46
Регистрация: 31.10.2014
Сообщений: 721
Записей в блоге: 1
27.11.2014, 10:58
Ты уверен что у тебя вызывается конструктор с параметрами H(int size, int top) ?
0
 Аватар для Foxa
0 / 0 / 0
Регистрация: 24.05.2010
Сообщений: 27
27.11.2014, 17:22  [ТС]
C++
1
2
3
4
5
6
int main(){
    Fun arr(3);
    H test(3, 1);
    arr.add(test);
    return 0;
}
Fun еще один класс, используется как хранилище нескольких объектов H
0
В астрале
Эксперт С++
 Аватар для ForEveR
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
27.11.2014, 17:24
Foxa, Показывайте весь класс H.
0
 Аватар для Foxa
0 / 0 / 0
Регистрация: 24.05.2010
Сообщений: 27
27.11.2014, 17:25  [ТС]
Вот, все и сразу:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
class H{
    int size;
    int top;
    int * set;
public:
    H(int size, int top){
        this->size = size;
        this->top = top;
        this->set = new int[size];
    }
 
    H(int size, int top, int * list){
        this->size = size;
        this->top = top;
        this->set = new int[size];
        for (int i = 0; i < this->size; i++)
            this->set[i] = list[i];
    }
 
    ~H(){ delete[] (this->set); }
 
    bool contain(int m){
        for (int i = 0; i < this->size; i++){
            if (m == this->set[i])
                return true;
        }
        return false;
    }
 
    int getTop(){ return this->top; }
    int getSet(int i){ return this->set[i]; }
};
 
class Fun{
    int counter;
    int dim;
    vector<H> list;
public:
    Fun(int dim){
        this->counter = 0;
        this->dim = dim;
    }
 
    void add(H & elem){
        this->counter++;
        (this->list).push_back(elem);
    }
 
    int getSize(){ return this->counter; }
 
    H getElem(int i){ return this->list[i]; }
};
0
654 / 575 / 164
Регистрация: 13.12.2012
Сообщений: 2,124
27.11.2014, 17:36
C++
1
2
3
4
void add(H & elem){
        this->counter++;
        (this->list).push_back(elem);
    }
тут копирование,а в классе H нет конструктора копии

Добавлено через 1 минуту
C++
1
2
3
4
5
6
7
8
9
H(const H & h){
        this->size = h.size;
        this->top = h.top;
        this->set = new int[size];
        for(int i = 0; i < size; i++)
        {
              set[i] = h.set[i];
        }
    }
1
 Аватар для Foxa
0 / 0 / 0
Регистрация: 24.05.2010
Сообщений: 27
27.11.2014, 17:39  [ТС]
Мне не нужно копирование. В функцию я подаю объект по ссылке, т.е. и в вектор должен добавляться сам объект. Нет?
0
654 / 575 / 164
Регистрация: 13.12.2012
Сообщений: 2,124
27.11.2014, 17:48
Цитата Сообщение от Foxa Посмотреть сообщение
т.е. и в вектор должен добавляться сам объект. Нет?
тогда должен быть вектор ссылок, еще можно указателей попробовать, лучше просто определите конструктор копии, и будет счастье

Добавлено через 28 секунд
Цитата Сообщение от Foxa Посмотреть сообщение
Мне не нужно копирование.
Цитата Сообщение от Foxa Посмотреть сообщение
C++
1
H getElem(int i){ return this->list[i]; }
тут вот тоже копирование

Добавлено через 2 минуты
и это, this->list[i] Не обязательно, можно просто list[i]
1
 Аватар для Foxa
0 / 0 / 0
Регистрация: 24.05.2010
Сообщений: 27
27.11.2014, 17:53  [ТС]
C++
1
2
3
4
5
6
7
H(const H & h){
        this->size = h.size;
        this->top = h.top;
        this->set = new int[size];
        for (int i = 0; i < size; i++)
            this->set[i] = h.getSet(i);
    }
Спасибо. Но в предпоследний строчке при вызове getSet(i) ругается на "cannot convert 'this' pointer from 'const H' to 'H &'
0
 Аватар для taras atavin
4226 / 1796 / 211
Регистрация: 24.11.2009
Сообщений: 27,562
27.11.2014, 18:01
Цитата Сообщение от Foxa Посмотреть сообщение
Мне не нужно копирование. В функцию я подаю объект по ссылке, т.е. и в вектор должен добавляться сам объект. Нет?
Сам объект? Что серьёзно? Интересно, с какого перепугу он переместится то?
0
 Аватар для Foxa
0 / 0 / 0
Регистрация: 24.05.2010
Сообщений: 27
27.11.2014, 18:13  [ТС]
Сам объект? Что серьёзно? Интересно, с какого перепугу он переместится то?
Уже, спасибо вышеотписавшимся, осознал ошибку, извините.

Добавлено через 10 минут
Проблема решилась не очень честно, у Н поле set стало public и обращение идет напрямую по индексу, но лучше не придумалось. Спасибо всем.
0
В астрале
Эксперт С++
 Аватар для ForEveR
8049 / 4806 / 655
Регистрация: 24.06.2010
Сообщений: 10,562
27.11.2014, 19:37
Foxa, getSet должен быть константной функцией и все сразу будет работать.
1
654 / 575 / 164
Регистрация: 13.12.2012
Сообщений: 2,124
28.11.2014, 09:51
[quote="Foxa;6909321"]у Н поле set стало public и обращение идет напрямую по индексу, но лучше не придумалось. зачем его public то, мы же из самого класса к своему же полю обращаемся
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
28.11.2014, 09:51
Помогаю со студенческими работами здесь

Ошибка при вызове деструктора
Добрый вечер! Почему при вызове деструктора вылетает ошибка после компиляции кода? #include &lt;conio.h&gt; #include &lt;iostream&gt; ...

SIGABRT при вызове деструктора
mainwindow.h #ifndef MAINWINDOW_H #define MAINWINDOW_H #include &lt;QMainWindow&gt; #include &quot;plot.h&quot; #include &lt;QFormLayout&gt; ...

Ошибка при вызове деструктора
Не могу дописать лабу... Надо создать 4 класса, описать поля и методы. Выкладываю исходники: Самый главный класс - Order. Выбивает ошибку в...

Ошибка при вызове деструктора
Программа компилиться нормально,но после выполнения system(&quot;pause&quot;); выползает непонятное окно,и указывает на ошибку в деструкторе.В чем...

Ошибка при вызове деструктора
Всем привет ! Есть класс class building { private: string *location_street; unsigned int *number_of_building; public...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
19
Ответ Создать тему
Новые блоги и статьи
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами
8Observer8 20.02.2026
Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .
Конвертировать закладки radiotray-ng в m3u-плейлист
damix 19.02.2026
Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI
Eddy_Em 18.02.2026
Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru