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

При перегрузке операторов не записывает значение - C++

Восстановить пароль Регистрация
 
Laky
6 / 6 / 0
Регистрация: 07.02.2011
Сообщений: 39
02.03.2011, 19:27     При перегрузке операторов не записывает значение #1
Собственно вот мой код:
class.h
C++
1
2
3
4
5
6
7
8
9
10
11
12
class Otrezku
{
public:
    Otrezku();
    Otrezku(float , float , float , float );
    Otrezku(Otrezku &);
    Otrezku operator +(Otrezku);
    Otrezku operator -(int);
    Otrezku operator =(Otrezku);
private:
    float X1, X2, Y1, Y2, L;
};
main.cpp
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
using namespace std;
#include <iostream>
#include "class.h"
 
void main()
{
    Otrezku W1;
    float X1, X2, Y1, Y2;
    cout<<"Enter the co-ordinates of the second cutting-off(A(x1;y1) and B(x2;y2) through a space in a next order: x1, y1, x2, y2"<<endl;
    cin>>X1>>Y1>>X2>>Y2;
    Otrezku W2(X1, X2, Y1, Y2);
    Otrezku W3 = W2;
    W3 = W3 - 3;
    W1.operator=(W2.operator+(W3));
}
other.cpp
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
using namespace std;
#include <iostream>
#include "class.h"
#include <math.h>
 
Otrezku::Otrezku()
{
    X1 = 0;
    X2 = 0;
    Y1 = 0;
    Y2 = 0;
}
 
Otrezku::Otrezku(float XX1, float XX2, float YY1, float YY2)
{
    X1 = XX1;
    X2 = XX2;
    Y1 = YY1;
    Y2 = YY2;
}
 
Otrezku::Otrezku(Otrezku &x)
{
    X1 = x.X1;
    X2 = x.X2;
    Y1 = x.Y1;
    Y2 = x.Y2;
}
 
Otrezku Otrezku::operator+(Otrezku x)
{
    Vidrizku tmp;
    tmp.X1 = X1 + x.X1;
    tmp.X2 = X2 + x.X2;
    tmp.Y1 = Y1 + x.Y1;
    tmp.Y2 = Y2 + x.Y2;
    return tmp;
}
 
Otrezku Otrezku::operator-(int A)
{
    Otrezku tmp;
    tmp.X1 = X1 - A;
    tmp.X2 = X2 - A;
    tmp.Y1 = Y1;
    tmp.Y2 = Y2;
    return tmp;
}
 
Otrezku Otrezku::operator=(Otrezku x)
{
    Otrezku tmp;
    tmp.X1 = X1;
    tmp.X2 = X2;
    tmp.Y1 = Y1;
    tmp.Y2 = Y2;
    return tmp;
}
Компилятор ошибок не видит. При выполнении кода:
C++
1
    W3 = W3 - 3;
Можно увидеть в watch-e "Otrezku::operator- returned" с темы значениями, которые считает при перегрузке. Но оно не записывает в W3. Тоже самое при выполнении
C++
1
W1.operator=(W2.operator+(W3));
В чем здесь проблема? На лекции код давали, я по нему делал, но у меня не работает... Спасибо за помощь!!!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
02.03.2011, 19:27     При перегрузке операторов не записывает значение
Посмотрите здесь:

Ошибка при перегрузке операторов. C++
Тестовые вопросы по перегрузке операторов C++
О перегрузке операторов C++
Ошибка при перегрузке операторов в классе C++
Ошибки при перегрузке операторов C++
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
02.03.2011, 19:48     При перегрузке операторов не записывает значение #2
Оператор присваивания, очевидно, перегружен неверно. Вы там меняете какие-то значения, но сам-то объект, для которого вызван operator=, не изменяется. Вы просто поменяли значения у tmp и вернули их в пустоту. Присваивание в вашем случае вообще не надо перегружать (компилятор сам сгенерирует необходимую функцию), но если очень хочется, следует перегружать так:
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
const Otrezku &Otrezku::operator=(const Otrezku &x)
{
    if (this != &x)
    {
        X1 = x.X1;
        X2 = x.X2;
        Y1 = x.Y1;
        Y2 = x.Y21;
        L = x.L;
    }
 
    return *this;
}
Laky
6 / 6 / 0
Регистрация: 07.02.2011
Сообщений: 39
02.03.2011, 20:03  [ТС]     При перегрузке операторов не записывает значение #3
объясните,пожалуйста, вот эту строку кода, я впервые такое вижу и не могу догнать, почему так...
C++
1
const Otrezku &Otrezku::operator=(const Otrezku &x)
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
02.03.2011, 20:12     При перегрузке операторов не записывает значение #4
Это всё для того, чтобы повторить поведение оператора присваивания для встроенных типов. Принимаемый параметр - константная ссылка на объект того же класса. Ссылка потому, что в общем случае может передаваться очень большой объект (хранящий, например, двусвязный список из 1000 элементов), и, передавай мы значение, пришлось бы совершать абсолютно не нужную здесь операцию копирования всего списка. А так мы будем таскать нужные значения не из копии, а из оригинала, используя для этого ссылку. Константная же она потому, что оператор присваивания по смыслу не должен менять присваиваемый объект. Защита от дурака, короче.
Та же история с возвращаемым значением. Когда мы выполняем, например, сложение, там обязательно нужен временный объект, который и будет представлять сумму, и его мы должны возвратить по значению (т.е. скопировать его из функции operator+ в место вызова, а потом удалить объект). Когда же мы выполняем присваивание, результатом операции является присвоенное значение, которое у нас лежит в том объекте, для которого выполнялось присваивание. Поэтому смысла копировать весь объект нет, ведь он после завершения функции operator= никуда не денется. Поэтому мы просто возвращаем ссылку на сам вызывающий объект - *this. Константная же она для того, чтобы мы не могли совершить бессмысленную операцию (A = B) = C;
rangerx
1908 / 1517 / 139
Регистрация: 31.05.2009
Сообщений: 2,876
03.03.2011, 02:37     При перегрузке операторов не записывает значение #5
Цитата Сообщение от silent_1991 Посмотреть сообщение
Константная же она для того, чтобы мы не могли совершить бессмысленную операцию (A = B) = C;
И тем не менее для простых типов это работает. Почему в таком случае это не должно работать для сложных? )
silent_1991
Эксперт C++
4938 / 3014 / 149
Регистрация: 11.11.2009
Сообщений: 7,024
Завершенные тесты: 1
03.03.2011, 09:46     При перегрузке операторов не записывает значение #6
rangerx, да, действительно. Законный вопрос)))
Yandex
Объявления
03.03.2011, 09:46     При перегрузке операторов не записывает значение
Ответ Создать тему
Опции темы

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