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

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

27.11.2014, 10:27. Показов 1204. Ответов 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
Ответ Создать тему
Новые блоги и статьи
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка.
Programma_Boinc 23.12.2025
Thinkpad X220 Tablet — это лучший бюджетный ноутбук для учёбы, точка. Рецензия / Мнение Это мой обзор планшета X220 с точки зрения школьника. Недавно я решила попытаться уменьшить свой. . .
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Как объединить две одинаковые БД Access с разными данными
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru