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

почему деструктор работает вначале? - C++

Восстановить пароль Регистрация
 
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 20:01     почему деструктор работает вначале? #1
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
#pragma once
class Student
{
private:
    char* name;
    int course;
public:
    Student(void);
    ~Student(void);
    Student(char * nm, int c);
    void Show(void);
    Student(const Student & s);
    Student operator =(Student s)
    {
        this->name=s.name;
        this->course=s.course;
    }
    Student operator +(const int s)
    {
        course+=s;
        return *this;
    }
    Student operator ++()
    {
        course++;
        return *this;
    }
    
};
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
#include "StdAfx.h"
#include "Student.h"
#include <cstring>
 
 
Student::Student(void)
{
    
    name=new char[7];
    strcpy(name,"NoName");
    course=0;
    
}
 
 
Student::~Student(void)
{
    cout<<"Destructor"<<endl;
    delete name;
}
 
 
Student::Student(char * nm, int c)
{
    name=new char[strlen(nm)+1];
    strcpy(name,nm);
    course=c;
}
 
 
void Student::Show(void)
{
    cout<<"Name is "<<name<<'\n'<<"Course is "<<course<<endl;
}
 
Student::Student(const Student & s)
{
    name=new char[strlen(s.name)+1];
    strcpy(name,s.name);
    course=s.course;
}
C++
1
2
3
4
5
6
7
int _tmain(int argc, _TCHAR* argv[])
{
    Student s("Tom",3);
    s++;
    s.Show();
    return 0;
}
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11845 / 6824 / 771
Регистрация: 27.09.2012
Сообщений: 16,919
Записей в блоге: 2
Завершенные тесты: 1
28.05.2013, 20:05     почему деструктор работает вначале? #2
Потому что возвращаете копии объектов. И деструкторы вызываются для временных объектов
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 20:11  [ТС]     почему деструктор работает вначале? #3
Croessmah, а почему здесь не выдает ошибки,не написала return,но компилируется.
C++
1
2
3
4
5
Student operator =(Student s)
    {
        this->name=s.name;
        this->course=s.course;
    }
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11845 / 6824 / 771
Регистрация: 27.09.2012
Сообщений: 16,919
Записей в блоге: 2
Завершенные тесты: 1
28.05.2013, 20:18     почему деструктор работает вначале? #4
А Вы попробуйте вызвать эту функцию в коде.
C++
1
2
3
Student s1("Tom",3);
Student s2("Jeff",4);
s2=s1;
Добавлено через 1 минуту
И это
C++
1
2
        this->name=s.name;
        this->course=s.course;
Не правильно, т.к. name указатель на динамически выделенную память.
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 20:21  [ТС]     почему деструктор работает вначале? #5
Croessmah, вот так написала,и из за этого кода опять деструктор работает вначале
C++
1
2
3
4
5
6
7
Student& operator =(Student s)
    {
        name=new char[strlen(s.name)+1];
        strcpy(this->name,s.name);
        this->course=s.course;
        return *this;
    }
Olivеr
 Аватар для Olivеr
411 / 407 / 13
Регистрация: 06.10.2011
Сообщений: 830
28.05.2013, 20:25     почему деструктор работает вначале? #6
Mar-12, неправильно так как память не освобождается
И какой резон передавать по значению?
C++
1
2
3
4
5
6
7
    Student& operator = (const Student &s) {
        delete name;
        name = new char[ strlen(s.name) + 1 ];
        strcpy(name, s.name);
        course = s.course;
        return *this;
    }
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 20:31  [ТС]     почему деструктор работает вначале? #7
и опять деструктор работает вначале

Добавлено через 2 минуты
уже все работает,спасибо
Olivеr
 Аватар для Olivеr
411 / 407 / 13
Регистрация: 06.10.2011
Сообщений: 830
28.05.2013, 20:31     почему деструктор работает вначале? #8
Потому, что конструкторы кривые
C++
1
2
3
4
5
6
7
Student::Student(void):
    name( new char[7] ), course(-1)
{ strcpy(name,"NoName"); }
 
Student::Student(const char * nm, int c):
    name( new char[ strlen(nm) + 1 ] ), course(c)
{ strcpy(name,nm); }
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 20:36  [ТС]     почему деструктор работает вначале? #9
Olivеr, a в чем разница конструкторов?
alsav22
5282 / 4801 / 442
Регистрация: 04.06.2011
Сообщений: 13,587
28.05.2013, 20:42     почему деструктор работает вначале? #10
Цитата Сообщение от Mar-12 Посмотреть сообщение
и опять деструктор работает вначале
Работает - значит нужно. Что это вас беспокоит? Он здесь работает:
C++
1
s++;
Croessmah
Модератор
Эксперт С++
 Аватар для Croessmah
11845 / 6824 / 771
Регистрация: 27.09.2012
Сообщений: 16,919
Записей в блоге: 2
Завершенные тесты: 1
28.05.2013, 20:46     почему деструктор работает вначале? #11
C++
1
delete name;
name - это массив, а не один элемент, поэтому delete []

Цитата Сообщение от alsav22 Посмотреть сообщение
Что это вас беспокоит?
Думаю, что из-за двойного удаления памяти в деструкторе
Olivеr
 Аватар для Olivеr
411 / 407 / 13
Регистрация: 06.10.2011
Сообщений: 830
28.05.2013, 20:46     почему деструктор работает вначале? #12
В ваших конструкторах name и course создаются и ничем не инициализируются, а потом в теле конструктора им присваивается значение. В моих они сразу инициализируются.
И у вас ошибка в постфиксном декременте:
C++
1
2
3
4
5
6
7
8
9
10
    Student operator++(int) //постфиксный декремент
    {
        Student temp(*this);
        course++;
        return temp;
    }
    
    Student(const Student & s): //конструктор копирования
        name( new char[ strlen(s) + 1 ] ), course(s.course) 
    { strcpy(name,s.name); }
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 20:56  [ТС]     почему деструктор работает вначале? #13
alsav22, я начинающий,учусь перегружать операторы и беспокоюсь правильно ли я делаю или нет

Добавлено через 1 минуту
Olivеr, а этому классу еще версии конструкторов нужны или нет?
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
28.05.2013, 21:07     почему деструктор работает вначале?
Еще ссылки по теме:

Почему деструктор не работает? C++
Почему не работает деструктор? C++
C++ Почему не вызывается деструктор?

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

Или воспользуйтесь поиском по форуму:
Mar-12
 Аватар для Mar-12
-21 / 5 / 1
Регистрация: 14.03.2013
Сообщений: 131
28.05.2013, 21:07  [ТС]     почему деструктор работает вначале? #14
alsav22,Конструктор копирования есть,я написала
Yandex
Объявления
28.05.2013, 21:07     почему деструктор работает вначале?
Ответ Создать тему
Опции темы

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