私自身の演算子の定義に問題があります...

Question

演算子の定義方法を見てみるのは面倒です。次のコードは、「候補関数が見つかりません」というエラーを表示しています。

根本原因以外の批判も歓迎です。ありがとう！

#include <iostream> 

using std::cout; using std::cin; using std::endl; 

using std::string; 

class SomeClass { 
public: 
    SomeClass(int newNum, string newString) { num=newNum; str = newString; } 

    SomeClass& operator=(const SomeClass& rh) { 
     string newVal(rh.getStr()); 
     str = newVal; 
    } 

    void setStr(string newString) { str = newString; } 
    const string getStr() { return str; } 
    string toString() { return str+str; } 

private: 
    string str; 
    int num; 
}; 

int main() { 
    SomeClass a(5, "five"); 
    SomeClass b(3, "three"); 

    cout << a.toString() << endl << b.toString() << endl; 

    a=b; 

    cout << a.toString() << endl << b.toString() << endl; 
} 

Answer 1

const string getStr() { return str; } 

は

const string& getStr() const { return str; } 

する必要がありますそうしないと、プライベート、パブリックおよび保護された可視性がクラスレベルであることを

SomeClass& operator=(const SomeClass& rh) 

Answer 2

すべてはあなたがあなたにもこれを含める必要が

#include <string> 

を含めていませんでしたことを除いて、これまでに正常に見えます。

ああ、私はまた、あなたが関数から何かを返していないことを見た：

SomeClass& operator=(const SomeClass& rh) { 
     string newVal(rh.getStr()); 
     str = newVal; 
     return *this; //DO THIS AS WELL 
    } 

そしてまた、rhはこの機能ではconstオブジェクトであり、それを使用して、あなたが非でgetStr()を呼んでいます問題を引き起こしている-const関数ので、修正はこれです：

SomeClass& operator=(const SomeClass& rh) { 
     str = rh.str; //no need to create a local (temporary) variable! 
     return *this; //DO THIS AS WELL 
    } 

私はそのよりよい解決策を考える：

const string getStr() const { return str; } 
//     ^^^^^ make the function const! 

そして、代わりに次のように、あなたはあなたのoperator=を書いたことができます！

Answer 3

ノートのconstのパラメータに非const関数を呼び出すことはできません、インスタンスレベルではありません。したがって、getStr()機能は必要ありません。あなたは書くことができます：

SomeClass& operator=(const SomeClass& rh) { 
    this->str = rh.str; 
    return *this; 
} 

Answer 4

をあなただけのconstオブジェクトが、constメンバ関数を呼び出すことができるだけで、あなたがこれを行うにしようとしていたので、これは

const string getStr() { return str; } 

const string getStr() const { return str; } //the 2nd const make getStr() a const member function 
         ^^^^ 

に変更する必要があります。

SomeClass& operator=(const SomeClass& rh) { //here you declared rh to be const 
     //so in order to call getStr() from rh, you need declare getStr function to be const 
     string newVal(rh.getStr()); 
     str = newVal; 
     return *this; 
    } 

Answer 5

あなたのクラスは単純に次のようになります：

class SomeClass { 
public: 
    SomeClass(int newNum, string newString):num(newNum), str(newString) {/* Empty */} 
    SomeClass& operator=(const SomeClass& rh) { 
    str=rh.str; 
    return *this; 
    } 
    string toString() { return str+str; } 
private: 
    string str; 
    int num; 
};