複素数の*演算子のオーバーロード（C++）

Question

したがって、複素数の基本的な算術演算を実装するように演算子をオーバーロードするように求められています。私は+、 - 、そして=働いていますが、私は正しいロジックを持っていると思っていても働くことはできません。

私のコードで何が問題になっていますか？

#ifndef COMPLEX_HPP 
#define COMPLEX_HPP 
#include <string> 

class Complex 
{ 
    public: 
     Complex(double = 0.0, double = 0.0); // default constructor 
     Complex add(const Complex&) const;   // function add 
     Complex subtract(const Complex&) const; // function subtract 
     Complex multiply(const Complex&) const; // function multiply 
     std::string toString() const; // return string representation 
     void setComplexNumber(double, double); // set complex number 

     void operator=(const Complex& obj) 
     { 
      (*this).realPart = obj.realPart; 
      (*this).imaginaryPart = obj.imaginaryPart; 
     } 

     Complex operator+(const Complex& obj) 
     { 
      Complex tmp_obj = *this; 
      tmp_obj.realPart = tmp_obj.realPart + obj.realPart; 
      tmp_obj.imaginaryPart = tmp_obj.imaginaryPart + obj.imaginaryPart; 
      return tmp_obj; 
     } 

     Complex operator-(const Complex& obj) 
     { 
      Complex tmp_obj = *this; 
      tmp_obj.realPart = tmp_obj.realPart - obj.realPart; 
      tmp_obj.imaginaryPart = tmp_obj.imaginaryPart - obj.imaginaryPart; 
      return tmp_obj; 
     } 

     Complex operator*(const Complex&obj) 
     { 
      Complex tmpObj = *this; 
      tmpObj.realPart = (tmpObj.realPart * obj.realPart) - (tmpObj.imaginaryPart * obj.imaginaryPart); 
      tmpObj.imaginaryPart = (tmpObj.realPart * obj.imaginaryPart) + (tmpObj.imaginaryPart * obj.realPart); 
      return tmpObj; 

     } 


    private: 
     double realPart; 
     double imaginaryPart; 
}; 

Answer 1

あなたはtmpObj（具体的には、tmpObj.realPart）を上書き起動したら、元の値を失ってしまいました。だから、tmpObjから読み込まれますが、*thisからしません：

tmpObj.realPart = realPart * obj.realPart - imaginaryPart * obj.imaginaryPart; 
//    ^^^^^^^^     ^^^^^^^^^^^^^ 
tmpObj.imaginaryPart = realPart * obj.imaginaryPart + imaginaryPart * obj.realPart; 
//      ^^^^^^^^      ^^^^^^^^^^^^^ 

Answer 2

operator*()の実装では、本質的です。 tmpObj.realPartが変更されていないかのように

Complex operator*(const Complex&obj) 
{ 
    Complex tmpObj = *this; 
    tmpObj.realPart = (tmpObj.realPart * obj.realPart) - tmpObj.imaginaryPart * obj.imaginaryPart); 
    tmpObj.imaginaryPart = (tmpObj.realPart * obj.imaginaryPart) + (tmpObj.imaginaryPart * obj.realPart); 
    return tmpObj; 
} 

問題は、最初の文はtmpObj.realPartを修正することであり、第二の文進みます。

修正は簡単です。tmpObjを割り当ての右側に使用しないでください。

Complex operator*(const Complex&obj) 
{ 
    Complex tmpObj; // your default constructor initialises to (0.0,0.0) 
    tmpObj.realPart = (realPart * obj.realPart) - (imaginaryPart * obj.imaginaryPart); 
    tmpObj.imaginaryPart = (realPart * obj.imaginaryPart) + (imaginaryPart * obj.realPart); 
    return tmpObj; 
}