Вызов деструкторов членов класса

@Alexanches · Регистрация: 04.06.2010

Студворк — интернет-сервис помощи студентам

у меня есть класс поля которого тоже классы...при выходе из конструктора вызывается деструктор для его полей..даже для указателей...что делать?как этого избежать?

@patriot2008 · 08.11.2010, 00:59

поля-классы представлены у Вас просто как переменные или как указатели (ссылки) ?

@Alexanches · 08.11.2010, 01:00 **[ТС]**

как указатели

@alex_x_x · 08.11.2010, 01:01

как обявлены поля, как объявлен деструктор
деструкторы для указателей просто так не вызовутся

@Alexanches · 08.11.2010, 01:03 **[ТС]**

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
class cName
{
    MyString* fio;
public:
    cName():fio()
    {
    }
    cName(char* n, char* f, char* nof)
    {
        int l;
        char* fio1=new char[l=(strlen(n)+(strlen(f)+strlen(nof)+3))];
        fio1=strcat(n," ");
        fio1=strcat(fio1,f);
        fio1=strcat(fio1," ");
        fio1=strcat(fio1,nof);
        fio1[l]='\0';
        fio=&MyString(fio1);
    }
 
 
class MyString
{
    char* str;
    int length;
public:
    ~MyString()
    {
        delete[]str;
    }

@CheshireCat · 08.11.2010, 12:32

Гм. Я бы выразился в том смысле, что приведенный код не просто плохой - он очень плохой. Надеюсь, коллега Alexanches найдет время его поправить. Проблемы, которые я вижу с ходу:
1. Совершенно непонятно, какой из классов владеет памятью, выделяемой под строку. Выделяется память в одном классе, удаляется в совершенно другом.....
2. В конструкторе класса cName переменной fio присваивается адрес временного объекта; и куда будет указывать эта переменная после завершения тела конструктора - даже Б.Г. не знает....

Я бы предложил переделать этот код следующим образом:
1. Четко определить "зоны ответственности" классов. Так, класс MyString должен только хранить строку и предоставлять определенные операции над ней. Он же исключительно отвечает за выделение/удаления необходимой для строки памяти. Класс же cName должен хранить имя как готовый объект класса MyString, при этом внутреннее устройство класса MyString ему совершенно не интересно.
2. Исходя из этого, реализовать в классе MyString:
- конструктор по умолчанию, который должен инициализировать пустую строку;
- конструктор из const char*;
- правильные конструктор копирования и оператор присваивания;
- оператор + для конкатенации строк;
- ну и, возможно, оператор преобразования в const char* для вывода строки функцией типа printf, или же еще лучше дружественный оператор << для вывода в поток.
При этом все выделение и освобождение памяти должен выполнять класс MyString.
3. А класс cName должен просто хранить имя в член-переменной класса MyString, ну и предоставлять какой-то необходимый доступ к ней. Необходимо реализовать:
- конструктор из набора параметров - const char*, как написано - фамилия, имя, отчество.... и т.д.
- возможно, какие-то еще функции....

Во.

@Alexanches · 09.11.2010, 21:56 **[ТС]**

собственно если в cName объект то он удаляется деструктором майстринга с тем же успехом)конструкторы и перегруженные операторы есть просто здесь не приведены...операторы ввода вывода не перегружаю потому что задача для формы..

@Alexanches · 18.11.2010, 00:24 **[ТС]**

заменил

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
        cName(char* n, char* f, char* nof)
        {
                int l;
                char* fio1=new char[l=(strlen(n)+(strlen(f)+strlen(nof)+3))];
                fio1=strcat(n," ");
                fio1=strcat(fio1,f);
                fio1=strcat(fio1," ");
                fio1=strcat(fio1,nof);
                fio1[l]='\0';
                fio=&MyString(fio1);
        }
на 
cName(char* f):fio(f)
{};

где действия приведенные в 1 варианте выполняются вне конструктора...почему то деструктор не вызывается...

@Alexanches · 22.12.2010, 22:03 **[ТС]**

блин о5 та же шляпа..
вызываю функцию..оператор = перегружен при выходе из функции вызывается деструктор...
класс:

C++
1
2
3
4
5
6
class MyString
{
    char* str;
    int length;
public:
...};

вызов:
(name.input(m[first++])
функция:

C++
1
2
3
4
5
6
7
    int input(TextBox^ t)
{char buf[256];
    MyConvert::copyto(buf,t->Text->ToString(),255);
    MyString p(buf);
    this->assign(p);
return length=strlen(str); 
return 0;}

оператор=:

C++
1
2
3
4
5
6
7
8
9
10
11
12
    MyString& operator=(const MyString& st)
    {
        if(str!=st.str)
        {
            length=st.length;
            delete[] str;
            str=new char [st.length];
            strcpy(str,st.str);
            return *this;
        }
        return *this;
    }

@lemegeton · 23.12.2010, 00:10

Рекомендую почитать вдумчиво, что написал CheshireCat. Там, конечно, многабукаф, но вполне доходчиво.

Вообще, постоянно имейте в виду, что все переменные, объявленные в функции, будут уничтожены после выхода из функции.

C++
1
2
3
4
5
6
7
8
9
int input(TextBox^ t) {
  char buf[256];
  MyConvert::copyto(buf,t->Text->ToString(),255);
  MyString p(buf);  // объявленная в функции переменная.
  this->assign(p);
  return length=strlen(str); 
  return 0;   // выход из функции. будет уничтожена переменная p 
                // и вызван деструктор MyString::~MyString(p)
}

Никакой магии. Повторю, все, объявленное в функции будет уничтожено при выходе из функции.

@Alexanches · 23.12.2010, 15:45 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .
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 секунды (а то и больше),. . .

@Alexanches 86 / 7 / 2 Регистрация: 04.06.2010 Сообщений: 80

	Вызов деструкторов членов класса 08.11.2010, 00:40. Показов 4176. Ответов 10 Метки нет (Все метки) у меня есть класс поля которого тоже классы...при выходе из конструктора вызывается деструктор для его полей..даже для указателей...что делать?как этого избежать? 0

@patriot2008 73 / 73 / 10 Регистрация: 04.12.2008 Сообщений: 458
	08.11.2010, 00:59
	поля-классы представлены у Вас просто как переменные или как указатели (ссылки) ? 0

@Alexanches 86 / 7 / 2 Регистрация: 04.06.2010 Сообщений: 80
	08.11.2010, 01:00 [ТС]
	как указатели 0

@alex_x_x бжни 2473 / 1684 / 135 Регистрация: 14.05.2009 Сообщений: 7,162
	08.11.2010, 01:01
	как обявлены поля, как объявлен деструктор деструкторы для указателей просто так не вызовутся 0

@CheshireCat 2924 / 1274 / 114 Регистрация: 27.05.2008 Сообщений: 3,465
	08.11.2010, 12:32
	Гм. Я бы выразился в том смысле, что приведенный код не просто плохой - он очень плохой. Надеюсь, коллега Alexanches найдет время его поправить. Проблемы, которые я вижу с ходу: 1. Совершенно непонятно, какой из классов владеет памятью, выделяемой под строку. Выделяется память в одном классе, удаляется в совершенно другом..... 2. В конструкторе класса cName переменной fio присваивается адрес временного объекта; и куда будет указывать эта переменная после завершения тела конструктора - даже Б.Г. не знает.... Я бы предложил переделать этот код следующим образом: 1. Четко определить "зоны ответственности" классов. Так, класс MyString должен только хранить строку и предоставлять определенные операции над ней. Он же исключительно отвечает за выделение/удаления необходимой для строки памяти. Класс же cName должен хранить имя как готовый объект класса MyString, при этом внутреннее устройство класса MyString ему совершенно не интересно. 2. Исходя из этого, реализовать в классе MyString: - конструктор по умолчанию, который должен инициализировать пустую строку; - конструктор из const char; - правильные конструктор копирования и оператор присваивания; - оператор + для конкатенации строк; - ну и, возможно, оператор преобразования в const char для вывода строки функцией типа printf, или же еще лучше дружественный оператор << для вывода в поток. При этом все выделение и освобождение памяти должен выполнять класс MyString. 3. А класс cName должен просто хранить имя в член-переменной класса MyString, ну и предоставлять какой-то необходимый доступ к ней. Необходимо реализовать: - конструктор из набора параметров - const char*, как написано - фамилия, имя, отчество.... и т.д. - возможно, какие-то еще функции.... Во. 0

@Alexanches 86 / 7 / 2 Регистрация: 04.06.2010 Сообщений: 80
	09.11.2010, 21:56 [ТС]
	собственно если в cName объект то он удаляется деструктором майстринга с тем же успехом)конструкторы и перегруженные операторы есть просто здесь не приведены...операторы ввода вывода не перегружаю потому что задача для формы.. 0

@Alexanches 86 / 7 / 2 Регистрация: 04.06.2010 Сообщений: 80
	23.12.2010, 15:45 [ТС]
	к сожалению все это прочитал, и уже учел) что объявлено в функции не динамическое удалится, это понятно..к сожалению он(деструктор) удаляет тот объект который создан вне функции..при том он же вызывает эту ф-ю инпут .. то есть он вызвал его удалили) 0