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

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
 
Рейтинг: Рейтинг темы: голосов - 10, средняя оценка - 4.60
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
#1

Переопределение операции присваивания - C++

29.07.2014, 04:21. Просмотров 1387. Ответов 40
Метки нет (Все метки)

Есть вот такой класс:
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
#ifndef cd_h
#define cd_h
 
class Cd
{
private:
    char* performance;
    char* label;
    int selections;
    double playtime;
public:
    Cd();
    Cd(const char* s1, const char* s2, int n, double x);
    Cd(const Cd& c);
    virtual Cd& operator=(const Cd& c);
    virtual ~Cd();
    virtual void Report() const;
};
 
class Classic : public Cd
{
private:
    char* main_cmps;
public:
    Classic();
    Classic(const char* mc, const char* s1, const char* s2, int n, double x);
    Classic(char* mc, const Cd& c);
    Classic(const Classic& c);
    virtual Classic& operator=(const Cd& c);
    virtual ~Classic();
    virtual void Report() const;
};
#endif
И после переопределения операции присваивания:
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
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
#include "cd.h"
#include <iostream>
 
//class Cd
Cd::Cd()
{
    performance = label = nullptr;
    selections = playtime = 0;
}
 
Cd::Cd(const char* s1, const char* s2, int n, double x)
{
    performance = new char[strlen(s1) + 1];
    strcpy(performance, s1);
    label = new char[strlen(s2) + 1];
    strcpy(label, s2);
    selections = n;
    playtime = x;
}
 
Cd::Cd(const Cd& c)
{
    performance = new char[strlen(c.performance) + 1];
    strcpy(performance, c.performance);
    label = new char[strlen(c.label) + 1];
    strcpy(label, c.label);
    selections = c.selections;
    playtime = c.playtime;
}
 
Cd& Cd::operator=(const Cd& c)
{
    performance = new char[strlen(c.performance) + 1];
    strcpy(performance, c.performance);
    label = new char[strlen(c.label) + 1];
    strcpy(label, c.label);
    selections = c.selections;
    playtime = c.playtime;
    return *this;
}
 
Cd::~Cd()
{
    delete[] performance;
    delete[] label;
}
 
void Cd::Report() const
{
    std::cout << "Performance: " << performance << "\t" << "Label: " << label << std::endl
        << "Selections: " << selections << "\t" << "Playtime: " << playtime << std::endl;
}
 
 
// Class Classic
Classic::Classic() :Cd()
{
    main_cmps = nullptr;
}
 
Classic::Classic(const char* mc, const char* s1, const char* s2, int n, double x) :Cd(s1, s2, n, x)
{
    main_cmps = new char[strlen(mc) + 1];
    strcpy(main_cmps, mc);
}
 
Classic::Classic(char* mc, const Cd& c) : Cd(c)
{
    main_cmps = new char[strlen(mc) + 1];
    strcpy(main_cmps, mc);
}
 
Classic::Classic(const Classic& c):Cd(c)
{
    main_cmps = new char[strlen(c.main_cmps) + 1];
    strcpy(main_cmps, c.main_cmps);
}
 
Classic& Classic::operator=(const Cd& c)
{
    if (this == &c)
        return *this;
    Classic temp = (const Classic&)c;
    Cd::operator=(c);
    main_cmps = new char[strlen(temp.main_cmps) + 1];
    strcpy(main_cmps, temp.main_cmps);
    return *this;
}
 
Classic::~Classic()
{
    delete[] main_cmps;
    
}
 
void Classic::Report() const
{
    Cd::Report();
    std::cout << "Main composition: " << main_cmps << std::endl;
}
Почему то вызвается метод базового класа при присваивании межу собой обьектов производного класса.
Заранее спасибо.

Добавлено через 7 минут
Вследствие чего выводистя окно с ошибкой, как я понимаю из за того, что деструктор пытается удалить несуществующий блок памяти.
0
Лучшие ответы (1)
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
29.07.2014, 04:21
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Переопределение операции присваивания (C++):

Переопределение операции присваивания - C++
В случае, когда в классе есть члены, память под которые выделяется динамически, операцию присваивания, как и конструкторы с деструкторами...

Переопределение оператора присваивания - C++
Имеется такой простой класс: class TClass { private: float* A; int N; public: TClass(int _N) ...

Для чего нужно переопределение оператора присваивания? - C++
HumansClass&amp; operator=(const HumansClass&amp; right); //переопределение операции присваивания Для чего нужно переопределение оператора...

Операции присваивания - C++
Как можно проиллюстрировать возможности операций присваивания на примере программы?

Перегрузка операции присваивания и сложения - C++
Необходимо составить описание класса для определения одномерных массивов целых чисел. Границы индексов можно задавать произвольно, т.е....

Переопределение операции [] - C++
В классе строка переопределить операцию , которая будет возвращать символ по индексу, который передан в квадратные скобки. Помогите, не...

Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
dalay_lama
77 / 77 / 16
Регистрация: 22.09.2012
Сообщений: 470
29.07.2014, 08:50 #31
Цитата Сообщение от Gwini Посмотреть сообщение
Но мне просто интересно почему не работает тот код, ведь в теории вроде все сходится.
Да нефига не сходится)
Цитата Сообщение от dalay_lama Посмотреть сообщение
Classic temp = (const Classic&)c;
ок, теперь у тебя есть temp класса Classic. Что у него находится в main_cmps?
0
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
29.07.2014, 09:00  [ТС] #32
Цитата Сообщение от dalay_lama Посмотреть сообщение
ок, теперь у тебя есть temp класса Classic. Что у него находится в main_cmps?
ну здесь по идее должен вызыватся конструктор копирования производного класса:
C++
1
2
3
4
5
Classic::Classic(const Classic& c):Cd(c)
{
    main_cmps = new char[strlen(c.main_cmps) + 1];
    strcpy(main_cmps, c.main_cmps);
}
Он должен был бы все коректно скопировать. Но пробелма даже в том что при присвоении Classic Classic-у вызывается присваение базового класса, а не производного. (Я добавил вывод сообщение при вызове операциий присвоения — вот как я это понял).
0
dalay_lama
77 / 77 / 16
Регистрация: 22.09.2012
Сообщений: 470
29.07.2014, 09:19 #33
Цитата Сообщение от Gwini Посмотреть сообщение
ну здесь по идее должен вызыватся конструктор копирования производного класса:
Цитата Сообщение от Gwini Посмотреть сообщение
Classic::Classic(const Classic& c):Cd(c) { main_cmps = new char[strlen(c.main_cmps) + 1]; strcpy(main_cmps, c.main_cmps); }
Цитата Сообщение от Gwini Посмотреть сообщение
Он должен был бы все коректно скопировать.
да как же он у тебя скопирует то всё корректно?)
как ты из объекта c, который является экземпляром класса Cd, вытащишь свойство main_cmps??? даже если ты сделаешь приведение типа для него, с какого перепугу ты решил, что там возьмётся корректное значение main_cmps? сам родит и угадает, что там должно оказаться?
именно поэтому и не рекомендуется так делать, как сделал ты. у производного объекта, есть все свойства, которые есть у базового, у базового не все, которые есть у производного, поэтому и нельзя присваивать базовый, производному.
Приведение типа доступно только для того, чтобы если бы ты точно был уверен, что в какой-то функции используются только базовые методы и свойства какого-то класса, но при этом, функция принимает только объекты производного класса, у тебя бы всё равно оставалась возможность передать объект - у тебя это не так, ты пытаешься воспользоваться свойством производного класса. Ты присваиваешь объект базового типа производному, используя привидение типа, ок, но у него ВСЁ РАВНО НЕТУ СВОЙСТВА ПРОИЗВОДНОГО КЛАССА, ОНО НЕ ПОЯВЛЯЕТСЯ ПО ВОЛЕ БОЖЬЕЙ.

Добавлено через 7 минут
Цитата Сообщение от Gwini Посмотреть сообщение
ну здесь по идее должен вызыватся конструктор копирования производного класса:
Цитата Сообщение от Gwini Посмотреть сообщение
Classic::Classic(const Classic& c):Cd(c) { main_cmps = new char[strlen(c.main_cmps) + 1]; strcpy(main_cmps, c.main_cmps); }
Цитата Сообщение от Gwini Посмотреть сообщение
Он должен был бы все коректно скопировать.
да как же он у тебя скопирует то всё корректно?)
как ты из объекта c, который является экземпляром класса Cd, вытащишь свойство main_cmps??? даже если ты сделаешь приведение типа для него, с какого перепугу ты решил, что там возьмётся корректное значение main_cmps? сам родит и угадает, что там должно оказаться?
именно поэтому и не рекомендуется так делать, как сделал ты. у производного объекта, есть все свойства, которые есть у базового, у базового не все, которые есть у производного, поэтому и нельзя присваивать базовый, производному.
Приведение типа доступно только для того, чтобы если бы ты точно был уверен, что в какой-то функции используются только базовые методы и свойства какого-то объекта, но при этом, функция принимает только объекты производного класса - у тебя это не так. Ты присваиваешь объект базового типа производному, используя привидение типа, ок, но у него ВСЁ РАВНО НЕТУ СВОЙСТВА ПРОИЗВОДНОГО КЛАССА, ОНО НЕ ПОЯВЛЯЕТСЯ ПО ВОЛЕ БОЖЬЕЙ.


Цитата Сообщение от dalay_lama Посмотреть сообщение
Classic temp = (const Classic&)c;
Цитата Сообщение от dalay_lama Посмотреть сообщение
конструктор копирования производного класса:
Цитата Сообщение от Gwini Посмотреть сообщение
Classic::Classic(const Classic& c):Cd(c)
что будет в конструкторе, в c.main_cmps?
0
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
29.07.2014, 09:20  [ТС] #34
Цитата Сообщение от dalay_lama Посмотреть сообщение
как ты из объекта c, который является экземпляром класса Cd, вытащишь свойство main_cmps???
Это формальный параметр типа Cd а если я передам аргумент типа Classic то это уже будет экземпляр типа Classic, разве не так?

И если здесь все и так, как ты говоришь, то компилятор все равно не исполняет этот код. Я уже говорил что вызывается не это присвоение, а присвоение, которое определено в базовом классе.
0
dalay_lama
77 / 77 / 16
Регистрация: 22.09.2012
Сообщений: 470
29.07.2014, 09:22 #35
ох)
C++
1
Classic temp = (const Classic&)c;
конструктор копирования производного класса:
C++
1
Classic::Classic(const Classic& c):Cd(c)
что будет в конструкторе, в c.main_cmps?
0
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
29.07.2014, 09:28  [ТС] #36
Цитата Сообщение от dalay_lama Посмотреть сообщение
что будет в конструкторе, в c.main_cmps?
откуда я знаю?) Я ж спрашиваю, разве если формальный параметр ссылка на базовый класс, то при передаче ссылки на производный класс функция будет использовать экземпляр базового калсса, урезая даные-члены производного класса?
Но все равно вызвается даже не это присваение а базовое.
0
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
29.07.2014, 09:38  [ТС] #37
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
Cd& Cd::operator=(const Cd& c)
{
    std::cout << "===base"; //Вот
    delete[] performance;
    delete[] label;
    performance = new char[strlen(c.performance) + 1];
    strcpy(performance, c.performance);
    label = new char[strlen(c.label) + 1];
    strcpy(label, c.label);
    selections = c.selections;
    playtime = c.playtime;
    return *this;
}
0
Миниатюры
Переопределение операции присваивания  
dalay_lama
77 / 77 / 16
Регистрация: 22.09.2012
Сообщений: 470
29.07.2014, 09:51 #38
Лучший ответ Сообщение было отмечено автором темы, экспертом или модератором как ответ
Цитата Сообщение от Gwini Посмотреть сообщение
Я ж спрашиваю, разве если формальный параметр ссылка на базовый класс, то при передаче ссылки на производный класс функция будет использовать экземпляр базового калсса, урезая даные-члены производного класса?
ну так а откуда он возьмёт те свойства, которых у него в помине не существует?.
Код
Базовый_класс
{
   свойство 1;
}
Производный_класс
{
   свойство_2;
}
Базовый_класс v;//у него есть свойство 1, допустим в конструкторе оно приняло какое-то значение
(Производный_класс)v;//свойство_1 осталось тем-же, своство_2 - ?????
короче, нельзя так делать - фу-фу-фу.

если честно, я не помню, как досконально происходит приведение типа поэтому не могу ответить на твой вопрос по поводу того, какое присвоение будет вызвано, домой приду, сам ещё разбираться буду) Но факт в том, что у полученого объекта, который ты приводишь к типу производного класса просто не будет свойства, которое ты пытаешься скопировать и получить его длину strlen;
Попробуй сейчас оставить только присвоение вида Classic& Classic::operator=(Cd &cd);
только не делай этого в методе - Classic temp = (Classic &)cd а просто дай нулевой указатель свойству main_cmpc, проверь, будет ли ошибка

Добавлено через 6 минут
тьфу ты)
само собой у тебя будет вызываться сначала оператор присваивания базового класса. Тебе же сначала нужно задать значения для свойств базового класса, так? для этого вызываем оператор присваивания БАЗОВОГО класса Cd::operator=(argument) - у тебя были эти строки, я помню

Добавлено через 2 минуты
C++
1
2
3
4
5
6
7
8
9
10
Classic& Classic::operator=(const Cd& c)
{
    if (this == &c)
        return *this;
    Classic temp = (const Classic&)c;
    Cd::operator=(c);//вот же твой вызов оператора базового класса
    main_cmps = new char[strlen(temp.main_cmps) + 1];
    strcpy(main_cmps, temp.main_cmps);
    return *this;
}
короче, вот так надо:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
Classic& Classic::operator=(const Cd& c)
{
    if (this == &c)
        return *this;
    Cd::operator=(c);
    delete [] main_cmps;
    main_cmps = nullptr;
    return *this;
}
Classic& Classic::operator=(const Сlassic& c)
{
    if (this == &c)
        return *this;
    Cd::operator=(c);
    delete [] main_cmps;
    main_cmps = new char[ strlen(c.main_cmps) + 1 ];
    strcpy(main_cmps, c.main_cmps);
    return *this;
}
1
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
29.07.2014, 09:56  [ТС] #39
Цитата Сообщение от dalay_lama Посмотреть сообщение
Попробуй сейчас оставить только присвоение вида Classic& Classic::operator=(Cd &cd);
только не делай этого в методе - Classic temp = (Classic &)cd а просто дай нулевой указатель свойству main_cmpc, проверь, будет ли ошибка
Без приведения типа не получится дать нулевой указатель свойству main_cmpc, но все равно вызывается базовый метод а не этот, так что это не имеет значения — все равно та же ошибка.
Я вот думаю что для операции присвоения просто действуют строгие правила по поводу аргументов и нужно явно указать тип. И те правила что действуют для обычних методов не действуют для операции присвоения. Хотя для конструкторов действуют, так что я не уверен. В любом случае спасибо что уделил время и если узнаешь что то об этих правилах, то напиши пожалуйста. Удачи

Добавлено через 2 минуты
Цитата Сообщение от dalay_lama Посмотреть сообщение
тьфу ты)
само собой у тебя будет вызываться сначала оператор присваивания базового класса. Тебе же сначала нужно задать значения для свойств базового класса, так? для этого вызываем оператор присваивания БАЗОВОГО класса Cd::operator=(argument) - у тебя были эти строки, я помню
а все, ясно. Нету никаких строгих правил, просто main_cmps не инициализированый и при удалени невыделеного блока памяти вот эта ошибка. Все, спасибо!
0
dalay_lama
77 / 77 / 16
Регистрация: 22.09.2012
Сообщений: 470
29.07.2014, 09:57 #40
Кури ответ выше и забудь ты про это приведение типа уже)
0
Gwini
10 / 10 / 3
Регистрация: 08.03.2014
Сообщений: 70
29.07.2014, 09:57  [ТС] #41
Цитата Сообщение от dalay_lama Посмотреть сообщение
короче, вот так надо:
да код то я уже давно переписал, меня интересовало не это)
0
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
29.07.2014, 09:57
Привет! Вот еще темы с ответами:

Переопределение операции инкремента - C++
Добрый вечер! Задачка простая и кода с примерами в сети куча, но меня волнует вопрос почему компилятор ругается на мой код: Myclass...

Вызов функции с левой стороны от операции присваивания - C++
Здорова! Нужно перегрузить в классе String операцию выделения строки, но так чтобы она стояла слева от операции присваивания например...

Создаются ли копии объектов при операции присваивания? - C++
Доброго времени суток! Допустим есть объекты одного класса a1, a2, a3. Насколько я понимаю, при операции присваивания a1=a2 создаётся...

Переопределение операции вычитания строк - C++
Нужно переопределить операцию вычитания строк. То есть из первой строки вычитаются все символы,содержащиеся во второй строке. Я...


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

Или воспользуйтесь поиском по форуму:
Yandex
Объявления
29.07.2014, 09:57
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru