2015-07-10 76 views
0

我有以下结构用于存储rgb值。我为rgb和标量操作重载了常用操作符+, - ,*。结构运算符超载

struct rgb { 
    float r; 
    float g; 
    float b; 

    rgb() : r(0.f), g(0.f), b(0.f) {} 
    rgb(const float& rr, const float& gg, const float& bb): r(rr), g(gg), b(bb) {} 
    rgb(const float& f): r(f), g(f), b(f) {} 

    rgb& operator +(const rgb& a) { 
     rgb t; 
     t.r = this->r + a.r; 
     t.g = this->g + a.g; 
     t.b = this->b + a.b; 
     return t; 
    } 
    rgb& operator +(const float& a) { 
     rgb t; 
     t.r = this->r + a; 
     t.g = this->g + a; 
     t.b = this->b + a; 
     return t; 
    } 
    rgb& operator -(const rgb& a) { 
     rgb t; 
     t.r = this->r - a.r; 
     t.g = this->g - a.g; 
     t.b = this->b - a.b; 
     return t; 
    } 
    rgb& operator -(const float& a) { 
     rgb t; 
     t.r = this->r - a; 
     t.g = this->g - a; 
     t.b = this->b - a; 
     return t; 
    } 
    rgb& operator *(const rgb& a) { 
     rgb t; 
     t.r = this->r * a.r; 
     t.g = this->g * a.g; 
     t.b = this->b * a.b; 
     return t; 
    } 
    rgb& operator *(const float& a) { 
     rgb t; 
     t.r = this->r * a; 
     t.g = this->g * a; 
     t.b = this->b * a; 
     return t; 
    } 
    float sum() { 
     return r + g + b; 
    } 
}; 

当我在下面的函数使用这个结构,我得到一个意想不到的输出:

inline void foo(rgb &N, 
       rgb &X, 
       rgb &P, 
       rgb &R) { 
    float d = 0.0; 
    rgb d_v; 
    //find the dot product between N and (P-X) 
    d_v = N * (P - X); //this is always 0, 0, 0 
    d = d_v.sum(); 
    R = P - N * d; 
} 

但是,如果使用中介RGB的存储中间业务,它工作正常:

inline void subspaceProjectCPU(rgb &N, 
          rgb &X, 
          rgb &P, 
          rgb &R) { 
    float d = 0.0; 
    rgb d_v; 
    rgb PX = P - X; 
    //find the dot product between N and (P-X) 
    d_v = PX * N; 
    d = d_v.sum(); 
    rgb Nd = N*d; 
    R = P - Nd; 
} 

有人可以解释一些这方面的灯​​光? (我知道我没有处理双方的标量运算,所以N * d被定义,但不是d * N,其中N是rgb,d是float)。

+4

运算符'+','-','*','/'等不应返回引用。无论如何,这个参考是指什么?按价值返回。看看http://stackoverflow.com/questions/4421706/operator-overloading – juanchopanza

+0

如果你重载一个运算符'@',你应该重载'@ ='。 – celticminstrel

回答

2

您的运算符函数返回一个局部变量的引用,该运算符函数返回后该局部变量将停止存在。不要那样做;这是未定义的行为。只需返回rgb值。