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

@Gwini · Регистрация: 08.03.2014

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

Есть вот такой класс:

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 минут
Вследствие чего выводистя окно с ошибкой, как я понимаю из за того, что деструктор пытается удалить несуществующий блок памяти.

alsav22 · 29.07.2014, 08:20

Сообщение от Gwini

но strlen все равно применяется к строке в уже существующему объекту.

Этот объект как создавался? Так?

Сообщение от Gwini

C++
1
2
3
4
5
Cd obj1; // базовый класс
Classic obj2; //производный класс
Cd* ptr = &obj1; 
*ptr = obj2; *
ptr->Report(); *// ну и вот здесь хотелось бы что бы использовался метод производного класса

И какое значение имеют указатели в этих объектах?
Конструкторы, которые их инициализируют:

Сообщение от Gwini

C++
1
2
3
4
5
Cd::Cd()
{
    performance = label = nullptr;
    selections = playtime = 0;
}

Сообщение от Gwini

C++
1
2
3
4
Classic::Classic() :Cd()
{
    main_cmps = nullptr;
}

@Gwini · 29.07.2014, 08:26 **[ТС]**

Сообщение от dalay_lama

что именно не понятно?

Стоп, но у меня присвоение однотипное. То есть ошибка вылетает когда я присваеваю обьект Classic обьекту Classic.
Вот:

C++
1
2
Classic copy;
    copy = c2; // с2 также Classic

А для такого у меня есть перегружонное присовоение в базовом класе, котрое я переопределяю в производном классе:
1) Базовое: (обьявлено также вирутальным)

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
Cd& Cd::operator=(const Cd& c)
{
    std::cout << "\n===base\n";
    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;
}

2) Производное:

C++
1
2
3
4
5
6
7
8
9
10
11
12
Classic& Classic::operator=(const Cd& c)
{
    std::cout << "\n===der\n";
    if (this == &c)
        return *this;
    Cd::operator=(c);
    delete[] main_cmps;
    Classic temp = (const Classic&)c;   
    main_cmps = new char[strlen(temp.main_cmps) + 1];
    strcpy(main_cmps, temp.main_cmps);
    return *this;
}

Добавлено через 2 минуты

Сообщение от alsav22

Этот объект как создавался? Так?

Да нет же, это был пример, все обьекты инициализированы явными конструкторами:

C++
1
2
Cd c1("Beatles", "Capitol", 14, 35.5);
    Classic c2 = Classic("Piano Sonata in B flat, Fantasia in C", "Alfred Brendel", "Philips", 2, 57.17);

@dalay_lama · 29.07.2014, 08:30

а кто будет определять для производного класса это:

C++
1
Classic& Classic::operator=(Classic &cl)

?
он у тебя не определён, так? - что в таком случае будет, когда объект производнго, присвоишь объекту производного? наверняка операция присвоения по умолчанию и как ты думаешь она работает?.
И я бы ещё на твоём месте подумал над тем, что тебе выше написал alsav22,

@Gwini · 29.07.2014, 08:35 **[ТС]**

По идее сначало должно вызыватся присвоение определенное в производном класе т.к.

C++
1
copy = c2;

соответсвует его прототипу Classic& Classic::operator=(const Cd& c) — ссылка на базовый класс может же приниматть ссылку на производный. Потом это присваение вызывает базовое т.к. в нем определено вот это :

C++
1
Cd::operator=(c);

Все это и просиходит когда операции присваения отдельны то есть то есть не виртуальные и в производном классе ссылка не на базовый, а на производный класс, то есть вот:

C++
1
Classic& Classic::operator=(const Classic& c)

Но в случае виртуальной операции присвоения вылетает эта ошибка...

@dalay_lama · 29.07.2014, 08:37

И по поводу, когда производному присваивается базовый:

Сообщение от Gwini

Classic temp = (const Classic&)c;
main_cmps = new char[strlen(temp.main_cmps) + 1];

зачем эти лишние телодвижения? у объекта c нету свойства main_cmps, так зачем ты для этого создаешь ещё один объект используя привидение типа? всё равно туда дырка с бубликом запишется, так не лучше ли избежать ненужного вызова конструктора?
что тебе просто мешает сделать указатель на nullptr? к тому же такое приведение типа не очень хорошая практика, тем более она здесь совсем не оправдана

@Gwini · 29.07.2014, 08:39 **[ТС]**

Сообщение от dalay_lama

он у тебя не определён, так?

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

alsav22 · 29.07.2014, 08:42

Сообщение от Gwini

Да нет же, это был пример, все обьекты инициализированы явными конструкторами:

Вот так?

Сообщение от Gwini

C++
1
2
Classic copy;
copy = c2;  // здесь вместо присваения производного класа, вызывается присваение базового

@Gwini · 29.07.2014, 08:42 **[ТС]**

Сообщение от dalay_lama

зачем эти лишние телодвижения? у объекта c нету свойства main_cmps, так зачем ты для этого создаешь ещё один объект используя привидение типа? всё равно туда дырка с бубликом запишется, так не лучше ли избежать ненужного вызова конструктора?
что тебе просто мешает сделать указатель на nullptr? к тому же такое приведение типа не очень хорошая практика, тем более она здесь совсем не оправдана

Без приведения типа будут не возможны операции с производным классом. Это сделаано в случаее пеердачи аргумента типа Classic производного класса.

@dalay_lama · 29.07.2014, 08:42

Сообщение от Gwini

соответсвует его прототипу Classic& Classic::operator=(const Cd& c)

Согласен, моя ошибка. Но в любом случае. c - это объект базового класса, у него нету того свойства, которое ты пытаешься скопировать, если ты хочешь корректное присвоение одного Classic другому, создай соответствующий метод

@Gwini · 29.07.2014, 08:48 **[ТС]**

Сообщение от alsav22

Вот так?

Причем здесь обьект copy? strlen используется в операции присваения к строке что находится в уже существующем обьекте с2.

Добавлено через 4 минуты

Сообщение от dalay_lama

создай соответствующий метод

До этого я уже дошел

Можно просто определить не виртуальную и не переопределенную а отдельную операцию присвоения в производном классе Classic. Но мне просто интересно почему не работает тот код, ведь в теории вроде все сходится.

@dalay_lama · 29.07.2014, 08:50

Сообщение от Gwini

Но мне просто интересно почему не работает тот код, ведь в теории вроде все сходится.

Да нефига не сходится)

Сообщение от dalay_lama

Classic temp = (const Classic&)c;

ок, теперь у тебя есть temp класса Classic. Что у него находится в main_cmps?

@Gwini · 29.07.2014, 09:00 **[ТС]**

Сообщение от 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-у вызывается присваение базового класса, а не производного. (Я добавил вывод сообщение при вызове операциий присвоения — вот как я это понял).

@dalay_lama · 29.07.2014, 09:19

Сообщение от 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?

@Gwini · 29.07.2014, 09:20 **[ТС]**

Сообщение от dalay_lama

как ты из объекта c, который является экземпляром класса Cd, вытащишь свойство main_cmps???

Это формальный параметр типа Cd а если я передам аргумент типа Classic то это уже будет экземпляр типа Classic, разве не так?

И если здесь все и так, как ты говоришь, то компилятор все равно не исполняет этот код. Я уже говорил что вызывается не это присвоение, а присвоение, которое определено в базовом классе.

@dalay_lama · 29.07.2014, 09:22

ох)

C++
1
Classic temp = (const Classic&)c;

конструктор копирования производного класса:

C++
1
Classic::Classic(const Classic& c):Cd(c)

что будет в конструкторе, в c.main_cmps?

@Gwini · 29.07.2014, 09:28 **[ТС]**

Сообщение от dalay_lama

что будет в конструкторе, в c.main_cmps?

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

@Gwini · 29.07.2014, 09:38 **[ТС]**

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;
}

@dalay_lama · 29.07.2014, 09:51

Сообщение от Gwini

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

ну так а откуда он возьмёт те свойства, которых у него в помине не существует?.

Code
1
2
3
4
5
6
7
8
9
10
Базовый_класс
{
   свойство 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;
}

@Gwini · 29.07.2014, 09:56 **[ТС]**

Сообщение от dalay_lama

Попробуй сейчас оставить только присвоение вида Classic& Classic::operator=(Cd &cd);
только не делай этого в методе - Classic temp = (Classic &)cd а просто дай нулевой указатель свойству main_cmpc, проверь, будет ли ошибка

Без приведения типа не получится дать нулевой указатель свойству main_cmpc, но все равно вызывается базовый метод а не этот, так что это не имеет значения — все равно та же ошибка.
Я вот думаю что для операции присвоения просто действуют строгие правила по поводу аргументов и нужно явно указать тип. И те правила что действуют для обычних методов не действуют для операции присвоения. Хотя для конструкторов действуют, так что я не уверен. В любом случае спасибо что уделил время и если узнаешь что то об этих правилах, то напиши пожалуйста. Удачи

Добавлено через 2 минуты

Сообщение от dalay_lama

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

а все, ясно. Нету никаких строгих правил, просто main_cmps не инициализированый и при удалени невыделеного блока памяти вот эта ошибка. Все, спасибо!

@dalay_lama · 29.07.2014, 09:57

Кури ответ выше и забудь ты про это приведение типа уже)

Новые блоги и статьи Все статьи Все блоги /
Транслитерация текста с кириллицы на латиницу (вариант №1) Maks 11.11.2026 Решил облегчить жизнь себе и коллегам админам при создании электронных почтовых ящиков для пользователей, в результате написал обработку "Транслитерация текста" с кириллицы на латиницу, а в ней. . .	Книги по геймдизайну, левел-дизайну, игровым механикам и проектированию игрового опыта 8Observer8 08.11.2026 Содержание блога \| English Version 1. 2003 - 06 - Designing Virtual Worlds (1st Edition) Автор: Richard A. Bartle \| Объем: 768 стр. Ссылки: Amazon . . .	Родители токсичные абьюзеры по отношению к своим детям ругают их оскорбляют не любят zorxor 08.11.2026 Жить в атмосфере постоянных оскорблений, критики и холодности со стороны родителей — это тяжелое и болезненное испытание, которое оставляет глубокие эмоциональные раны. Токсичное и абьюзивное. . .	Как только я захожу к себе в свою комнату я сразу же с себя снимаю свои трусы. Ответ от Джемени ИИ. zorxor 08.11.2026 Джемени, как только я захожу к себе в свою комнату я сразу же с себя снимаю свои трусы. Многие люди, возвращаясь домой, сразу переодеваются в максимально свободную одежду или вовсе предпочитают. . .
Паттерн очереди действий/команд (Action / Command Queue). Система очереди действий и кат-сцен с SDL3 Callbacks, cglm и OpenGL 8Observer8 07.11.2026 Содержание блога \| English Version Руководство для начинающих по реализации паттерна очереди действий/ команд (Action / Command Queue) на C++ (C++20) с использованием официального SDL3 Callbacks. . .	Проверка полного ФИО в типовом справочнике Maks 07.11.2026 Решение реализовано в конфигурации КА2, на примере справочника "КонтактныеЛицаПартнеров". Задача: Запрет инициалов. Не допускаеть использование сокращений вида "И. И. ", "И. И. ", "И И" и. . .	Про "Windows must die!" kumehtar 04.11.2026 Знаете, чет накатило. . . . Когда-то это было почти боевым кличем: мол, нормальный человек сидит на DOS, OS/ 2 или Linux, а вот это вот - Must Die. А теперь человек произносит «мастдай» с усмешкой - и. . .	Красочные поля с номером страницы svg+background-image russiannick 04.11.2026 Красочные поля с номером страницы не менее важны чем в эпоху бумажных книг. Посему спешу представить почтеннейшей публике этот важный инструмент который эти поля и формирует. В процессе разработки. . .

@dalay_lama 82 / 82 / 50 Регистрация: 22.09.2012 Сообщений: 495
	29.07.2014, 09:57
	Кури ответ выше и забудь ты про это приведение типа уже) 0

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

Решение