Форум программистов, компьютерный форум CyberForum.ru

Деструктор - C++

Восстановить пароль Регистрация
 
a_lebedev
6 / 6 / 0
Регистрация: 05.02.2011
Сообщений: 48
18.04.2011, 20:01     Деструктор #1
Доброго времени суток, есть некоторый класс и 2 функции подобны тем, что описаны ниже. В первой объявляется данный класс, обрабатывается и после передается во вторую функцию
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
class myclass
{
    struct str
    {
        int field1;
        float field2;
        bool field3;
    };
    unsigned int k;
    public:
    str *st;
    myclass()
    {
        st = NULL;
    }
    void method1 (unsigned int a)
    {
        delete [] st;
        k = a;
        st = new str [k];
    }
    unsigned int method2 ()
    {
        return k;
    }
    ~myclass()
    {
        delete [] st;
    }
};
void fi ()
{
    myclass cl;
    cl.method1(5);
    for (int i = 0; i < cl.method2; i++)
    {
        cl.st[i].field1 = i;
        cl.st[i].field2 = i / 2.;
        cl.st[i].field3 = false;
    }
    f2(cl);
}
void f2(myclass cls)
{
    for (int i = 0; i < cls.method2; i++)
    {
        cout<<cls.st[i].field1<<cls.st[i].field2<<cls.st[i].field3;
    }
}
Проблема: время от времени программа тупо зависает. Проверил несколько раз все участки кода, где могла закрасться "проблема". Оказалось, что "все проходит", когда я комментирую освобождение памяти в деструкторе. Кто подскажет почему так?
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
18.04.2011, 20:01     Деструктор
Посмотрите здесь:

Деструктор C++
Деструктор C++
C++ деструктор
Деструктор C++
C++ Деструктор
C++ Почему создается виртуальный деструктор A, а в таблице виртуальных функций лежит деструктор B
C++ Деструктор
C++ Деструктор

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
NiRamz
209 / 209 / 16
Регистрация: 26.12.2010
Сообщений: 691
18.04.2011, 20:08     Деструктор #2
а разве при срабатывании деструктора не освобождается вся память, которая была выделена для работы класса? если так, то вы получается пытаетесь очистить память, которая и так очищается..
p.s. поправьте если не прав))
Nick Alte
Эксперт С++
1590 / 982 / 115
Регистрация: 27.09.2009
Сообщений: 1,897
Завершенные тесты: 1
18.04.2011, 21:06     Деструктор #3
Всё дело в передаче по значению: void f2(myclass cls)
при вызове f2 создаётся КОПИЯ cl, которая уничтожается при выходе из f2. При этом вызывается деструктор. Ну а поскольку для myclass не определён корректный конструктор копирования, в cls попадают копии всех полей, в том числе и адрес st. Строка по этому адресу освобождается при вызове деструктора для cls, при выходе из f2. После чего в cl.st остаётся тот же самый адрес, который уже показывает на освобождённый участок памяти. При уничтожении cl происходит попытка освободить его повторно, что и вызывает ошибку.
Выходы:
а) определить для myclass конструктор копирования, который создаёт копию строки;
б) передавать в функцию ссылку на myclass
void f2(const myclass& cls)
ну и для комплекта запретить копирование myclass, объявив в секции private заголовок конструктора копирования: myclass (const myclass&);
Yandex
Объявления
18.04.2011, 21:06     Деструктор
Ответ Создать тему
Опции темы

Текущее время: 08:20. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru