C++二维向量类中用this调用重载*运算符报错求助

首先，你遇到的错误核心原因是：this是一个指向当前Vector2对象的指针（类型为Vector2*），而你重载的operator*是针对Vector2对象（值类型）的运算符。编译器无法识别指针之间的*运算，所以抛出了expression must have arithmetic or unscoped enum type的错误。

解决方法 #

这里有几种可行的修改方案：

方案1：解引用this指针

把(this) * (this)改成*this * *this，这样就变成了两个Vector2对象调用你重载的点积运算符：

float Magnitude() {
    float dot = *this * *this; // 解引用指针，得到对象本身
    return std::sqrt(dot);
}

方案2：直接计算模长的平方（更高效）

既然模长的平方就是x² + y²，你可以跳过调用点积运算符，直接计算：

float Magnitude() {
    return std::sqrt(x * x + y * y);
}

这种方式少了一次函数调用的开销，代码也更直观。

额外的代码规范建议 #

另外，你的点积运算符和Magnitude函数都不会修改对象的状态，建议把它们声明为const成员函数，这样这些函数可以被const Vector2对象调用，代码更健壮：

class Vector2 {
private: 
    float x, y;
public: 
    Vector2(float _x, float _y) : x(_x), y(_y) {} // 建议用初始化列表初始化成员变量

    // 点积：声明为const成员函数
    float operator*(const Vector2& other) const { 
        return x * other.x + y * other.y; 
    }

    // 模长：声明为const成员函数
    float Magnitude() const {
        float dot = *this * *this;
        return std::sqrt(dot);
        // 或者直接 return std::sqrt(x*x + y*y);
    }

    Vector2 operator+(const Vector2& other) const { // 加法也建议设为const
        return Vector2(x + other.x, y + other.y); 
    }
};

这样修改后，你的代码不仅解决了报错问题，还更符合C++的编码规范。

内容的提问来源于stack exchange，提问作者Jordan Zdimirovic