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

@_lacrimosa_ · Регистрация: 10.06.2010

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

Имеется такой простой класс:

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
class TClass
{
    private:
        float* A;
        int N;
    public:
        TClass(int _N)                      //конструктор
        {
            N = _N;
            A = new float [N];
        }
        ~TClass()                           //деструктор
        {
            delete [] A;
        }
        TClass& operator= (const TClass& B) //переопределённый оператор =
        {
            if (this == &B)
                return *this;
 
            N = B.N;
            delete [] A;
            A = new float [N];
            memcpy(A,B.A,N*sizeof(float));
            
            return *this;
        }
};

Суть проблемы в поведении операции присваивания. Примеры использования объектов класса:

1) В случае, если объекты класса статические, то всё ОК:

C++
1
2
3
4
5
6
void main()
{
    TClass C(10);
    TClass D(5);
    C=D;
}

2) В случае, если объекты класса создаются динамически:

C++
1
2
3
4
5
    TClass* C = new TClass(10);
    TClass* D = new TClass(5);
    C=D;  // FAIL
    delete C;
    delete D;

После выполнения операции присваивания во 2-ом случае, объекты C и D идентичны. Более того, расположены на одном участке памяти, причём массивы, содержащиеся в C и D, также расположены на одном участке памяти. При вызове delete C память освобождается. При вызове delete D получается ситуация, когда нужно удалить ту же самую память (удалённую при помощи delete C, так как C=D). Отмечу, что оператор = вообще не вызывается при выполнении программы, что странно. Если не ошибаюсь, то вызывается оператор присваивания по умолчанию и объекты становятся копиями. Всё работает как надо, если:

вместо

C++
1
C=D;  // FAIL

использовать

C++
1
C->operator=(*D);

Собственно вопрос, как такого избежать и что предпринять? Требуемый результат: данные в C и D должны быть одинаковые, но адреса объектов различны.

@easybudda · 12.06.2010, 00:36

вот так вроде работает:

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
#include <iostream>
#include <algorithm>
 
class TClass {
private:
    float* A;
    int N;
public:
    TClass() : N(0), A(0) { }
    TClass(int _N) {
        N = _N;
        A = new float [N];
    }
    TClass(const TClass & t) {
        *this = t;
    }
    ~TClass() {
        if ( A )
            delete [] A;
    }
    const TClass& operator= (const TClass& B) {
        if (this == &B)
            return *this;
 
        N = B.N;
        if ( A )
            delete [] A;
        A = new float [N];
        std::copy(B.A, B.A + B.N, A);
//      memcpy(A,B.A,N*sizeof(float));
                       
        return *this;
    }
    
    friend std::ostream & operator << (std::ostream & ost, const TClass & t) {
        for ( int i = 0; i < t.N; ++i )
            ost << t.A[i] << std::endl;
        return ost;
    }
    
    friend std::istream & operator >> (std::istream & ist, TClass & t) {
        for ( int i = 0; i < t.N; ++i )
            ist >> t.A[i];
        return ist;
    }
};
 
int main(){
    TClass * a, * b, * c;
    const int size = 3;
    
    std::cout << "enter " << size << " double values: " << std::endl;
    a = new TClass(size);
    std::cin >> *a;
    std::cout << "\nA values:\n" << *a << std::endl;
    
    b = new TClass(*a);
    delete a;
    std::cout << "B values:\n" << *b << std::endl;
    
    c = new TClass();
    *c = *b;
    delete b;
    std::cout << "C values:\n" << *c << std::endl;
    
    delete c;
    return 0;
}

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

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

C=D;

ну ещё бы! копировать нужно не указатели, а их значения!

C++
1
*C = *D;

@_lacrimosa_ · 12.06.2010, 00:42 **[ТС]**

Спасибо. Особенно за строчку 62 =) Навело на правильную мысль.

Мне видимо нужно было вместо C=D использовать *C=*D. То есть в первом случае я копировал адрес объекта, а не сам объект. Теперь дошло.

Новые блоги и статьи Все статьи Все блоги /
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 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@_lacrimosa_ 1 / 1 / 2 Регистрация: 10.06.2010 Сообщений: 26
	12.06.2010, 00:42 [ТС]
	Спасибо. Особенно за строчку 62 =) Навело на правильную мысль. Мне видимо нужно было вместо C=D использовать C=D. То есть в первом случае я копировал адрес объекта, а не сам объект. Теперь дошло. 0