（求解）运算符重载为什么会改变原来的值？

[teardrop]

程序代码如下：

#include <iostream>
#include <string>
#include <math.h>

using namespace std;

class Point
{
private:
	float x;
	float y;
public:
	Point()
	{
		x = 0;
		y = 0;
	}
	Point (float a, float b):x(a), y(b){}
	Point &operator-(const Point& other)
	{
		x = x - other.x ;
		y = y - other.y;
		return *this;
	}
	void disp()
	{
		cout<<"Point:";
		cout<<"("<<x<<","<<y<<")"<<endl;
	}
};

int main(int argc, char*argv[])
{
	Point objone;
	Point objtwo(1,1);
	Point objthree;
	objone.disp();
	objtwo.disp();
	//objthree = objtwo-objone;
	objthree = objone-objtwo;
	objthree.disp();
	objone.disp();
	objtwo.disp();

	return 0;
}

输出是点1，2，3，1，2
Point:(0,0)
Point:(1,1)
Point:(-1,-1)
Point:(-1,-1)
Point:(1,1)
为什么在进行-操作后点1由原来的（0，0）也变为了（-1，-1）？

[caozl]

因为你在-操作的时候 改变了this自己的成员变量呀 如果你不想改变当前的数据 先把这个operator改成const 能编译过的时候 就好了

[芥川龙之谷]

很明显是出现在operator-函数这个问题上啊，main函数中，objthree是通过objone-objtwo是吧，你知道这是一个什么样的过程么，就是通过objone调用重载-的函数，然后把objtwo实参传给-函数的形参，这个函数改变了调用对象成员变量，然后通过默认复制构造好函数给objthree赋值，就是这样的一个过程，所以你就看到objone和objthree的结果是一样的了。

[赵4老师]

计算机组成原理→DOS命令→汇编语言→C语言（不包括C++）、代码书写规范→数据结构、编译原理、操作系统→计算机网络、数据库原理、正则表达式→其它语言（包括C++）、架构…… 对学习编程者的忠告： 眼过千遍不如手过一遍！ 书看千行不如手敲一行！ 手敲千行不如单步一行！ 单步源代码千行不如单步对应汇编一行！ 单步类的实例“构造”或“复制”或“作为函数参数”或“作为函数返回值返回”或“参加各种运算”或“退出作用域”的语句对应的汇编代码几步后，就会来到该类的“构造函数”或“复制构造函数”或“运算符重载”或“析构函数”对应的C/C++源代码处。 VC调试时按Alt+8、Alt+7、Alt+6和Alt+5,打开汇编窗口、堆栈窗口、内存窗口和寄存器窗口看每句C对应的汇编、单步执行并观察相应堆栈、内存和寄存器变化，这样过一遍不就啥都明白了吗。 对VC来说，所谓‘调试时’就是编译连接通过以后，按F10或F11键单步执行一步以后的时候，或者在某行按F9设了断点后按F5执行停在该断点处的时候。 （Turbo C或Borland C用Turbo Debugger调试,Linux或Unix下用GDB调试时,看每句C对应的汇编并单步执行观察相应内存和寄存器变化。）

[threenewbee]

Point &operator-(const Point& other) { Point p; p.x = x - other.x ; p.y = y - other.y; return p; }

[难题]

你把这个代码

    Point &operator-(const Point& other)
    {
        x = x - other.x ;
        y = y - other.y;
        return *this;
    }

改成

    Point &operator-(const Point& other)
    {
        return point(x-other.x,y-other.y);
    }

[芥川龙之谷]

引用 4 楼 caozhy 的回复:

Point &operator-(const Point& other) { Point p; p.x = x - other.x ; p.y = y - other.y; return p; }

你这个程序是有问题的，因为在函数里声明的对象，在方法完成后就会调用析构函数，所以返回引用就会出问题呢，你觉得呢，把引用&去掉的话，应该就可以了吧

[芥川龙之谷]

引用 5 楼 u013823973 的回复:

你把这个代码

    Point &operator-(const Point& other)
    {
        x = x - other.x ;
        y = y - other.y;
        return *this;
    }

改成

    Point &operator-(const Point& other)
    {
        return point(x-other.x,y-other.y);
    }

我就是想探讨一下，你这个应该算是返回一个临时声明的匿名对象吧，返回引用会不会出问题，因为方法结束后就会调用析构函数的吧

[勤奋的小游侠]

引用 7 楼 tangyongsa 的回复:

[quote=引用 5 楼 u013823973 的回复:] 你把这个代码

    Point &operator-(const Point& other)
    {
        x = x - other.x ;
        y = y - other.y;
        return *this;
    }

改成

    Point &operator-(const Point& other)
    {
        return point(x-other.x,y-other.y);
    }

我就是想探讨一下，你这个应该算是返回一个临时声明的匿名对象吧，返回引用会不会出问题，因为方法结束后就会调用析构函数的吧[/quote] 你的成员变量都是在栈中定义的，没有new出来的东西，并且拷贝构造函数已经定义了。所以不用担心这个问题。

[hongxiang895164403]

你在减号 - 的函数里改掉了x,y。谁的成员变量x,y呢，当然是objone对象的成员变量啊。所以输出的就是改变后的值