演算子のオーバーロード奇妙な結果

Question

HERESに私のクラス

​​

私は間違った結果を取得しています。たとえば、このワークス：

Vector vChange = velocity * time; 
position += vChange; 

しばらくこのdoesntの：

position += velocity * time; 

それがコンパイルされ、実行されますが、この1のために

同じ位置にいくつかの偽の書き込みを行う。すなわち：

Vector& Reflect(const Vector& I, const Vector& N) 
{ 
Vector v = I - 2 * Dot(N, I) * N; 
} 

何が間違っているのかアドバイスできますか？ありがとう！

Answer 1

ローカル変数への参照をoperator*に戻しています。それは未定義の動作です。代わりに値で返します。

Vector Vector::operator*(float n) const 
{ 
    Vector result = *this; 
    result.x *= n; 
    result.y *= n; 
    result.z *= n; 
    return result; 
} 

operator+と同じです。

Answer 2

ローカル変数への参照を返します。それをしないでください。あなたの配属していない演算子は、参考ではなく、価値によって返されるべきです。

Answer 3

operator*とoperator+の場合は、参考値ではなくVectorを値で返す必要があります。あなたが持っているものは、ぶら下がっている参照を返すことです。それは未定義の動作です。