オペレータのオーバーロードとVS2010でのLNK2019のエラー

Question

私は、3つの浮動小数点、x y z、いくつかの関数とオーバーロードされた演算子を保持するPointクラスを作成しています。私は以下の形式で演算子をコード化しました：

inline Point Point::operator+ (Point point) 
{ 
    return Point(x + point.x, y + point.y, z + point.z); 
} 

inline void Point::operator+= (Point point) 
{ 
    x += point.x; 
    y += point.y; 
    z += point.z; 
} 

これは正しい演算子ですか？私はそれを試して、それが動作しますが、私はそういう別のフォームを見た：

inline Point& Point::operator+ (Point& point) 
{ 
    return Point(x + point.x, y + point.y, z + point.z); 
} 

inline Point& Point::operator+= (Point& point) 
{ 
    x += point.x; 
    y += point.y; 
    z += point.z; 
     return *this; 
} 

2つのフォームの違いは何ですか？

また、私はPoint.cppファイル内の演算子を使用できますが、Main.cppで使用しようとすると、未解決の外部シンボルに対してlnk2019エラーが発生します。奇妙なことに、私の関数は定義ファイルの外で動作します。これらの演算子を定義したファイルの外で動作させるには、何が欠けていますか？

Answer 1

最初のオペレータは、既存のもの（一般的な経験則）を変更新しいオブジェクトを構築しないように除外するので、それは参照を返すべきではない

inline Point Point::operator+ (const Point &point) const 
{ 
    return Point(x + point.x, y + point.y, z + point.z); 
} 

であるべきです。その呼び出された点が変更されてはならないため、関数にconstが追加されます。それが参照を返す必要がありますので、既存のポイントを修正することになっているので、引数が追加される点を基準不要なコピーができなくなりますので

は、第1には、

inline Point& Point::operator+= (const Point& point) 
{ 
    x += point.x; 
    y += point.y; 
    z += point.z; 
    return *this; 
} 

でなければなりません。 constは変更されるべきではないので、引数に追加されます。関数自身はポイント自体を変更する必要があるため、関数自体はconstではありません。

リンカーエラーはインラインのためです。ヘッダーファイルに完全な実装を提供するか、インラインを削除してください。

Answer 2

inline Point& Point::operator+ (Point& point) 
{ 
    return Point(x + point.x, y + point.y, z + point.z); 
} 

これは間違っていますが、関数が返されたときに一時的に存在しなくなった参照を返します。あなたが十分に幸運であれば、それはセグメンテーションにつながるでしょう。ほとんどのコンパイラは、実行しようとすると警告を出します。理想的には、この演算子を自由な関数として記述し、メンバーoperator+=の形で実装します。

inline Point& Point::operator+= (Point& point) 
{ 
    x += point.x; 
    y += point.y; 
    z += point.z; 
     return *this; 
} 

これは、あなたがそうでなければ、あなたが一時にそれを使用することはできません、const参照としてpointを服用しなければならないことを除いて、ほとんど好ましい方法です。連鎖演算子を許可するのは、自身への参照を返すことです。一般的に、疑わしいときはintとして、intはそうすると言います。要約

、「cannonical」の実装は次のようになります持っoperator+はフリー機能することが

inline Point& Point::operator+=(Point const& point) 
{ 
    x += point.x; 
    y += point.y; 
    z += point.z; 
    return *this; 
} 

inline Point const operator+(Point left, Point const& right) 
{ 
    return left += right; 
} 

注Pointへの変換は、両方のオペランドだけでなく、右の1で発生することができます。 operator+=で便利に実装されています。左の引数は、変更可能な一時的なコピーを持つために値で取り込まれ、それに正しい引数を追加することによって行います。 operator+=は参照を返しますので、関数の結果としてコピーされる値を提供するためにこのような戻り値を使用します。