C++のオーバーロード代入演算子

Question

私はLocationというクラスを持っており、そのメンバー変数にCArrayを追加する必要がありました。この変更により、代入演算子のオーバーロードが必要になりました。

変更を加える前にコピーされていたこのクラスタイプのすべての変数をコピーし、個々のメンバ変数を個別にコピーせずにCArrayをコピーする追加コードを追加する方法はありますか？

Location& Location::operator=(const Location &rhs) 
{ 
    // Only do assignment if RHS is a different object from this. 
    if (this != &rhs) 
    { 
     //Copy CArray 
     m_LocationsToSkip.Copy(rhs.m_LocationsToSkip); 

     //Copy rest of member variables 
     //I'd prefer not to do the following 
     var1 = rhs.var1; 
     var2 = rhs.var2; 
     //etc 
    } 

    return *this; 
} 

Answer 1

ありがとうございます。 operator=自体をオーバーロードするタイプを使用してください。そのため、代わりにそのクラスを含むクラスで行う必要はありません。 MFCコードを書くときでも、私はまだMFCのコレクションと文字列クラスの代わりにstd::vector、std::stringなどを使用します。時々あなたはCStringを使用してかなり詰まっていますが、最後にstd::vectorの代わりにCArrayを使用したときは思い出せません。

Answer 2

いいえ、できません。 これを行う最善の方法は、実際のコードを生成するスクリプトを使用することです。

Answer 3

これは、通常、「コピーとスワップイディオム」と呼ばれるもので行われます。メンバ値と、最も重要なことには外部データへのポインタを交換するコピーコンストラクタとswap()メソッドを実装します。これで、割り当て担当者は次のようになります。

C& C::operator=(const C& c) { 
    C tmp(c); 
    this->swap(tmp); 
    return *this; 
} 

これで自己割り当てのガードは必要ありません。

Answer 4

はい。私が通常行うことは、コピー可能でないものを除いて、クラス内のメンバー構造体にすべてを入れることです。このように：

class Location 
{ 
    struct Members 
    { 
     int var1, var2; 
    }; 

    Members m; 
    CArray m_LocationsToSkip; 

public: 
    Location& operator=(Location const& rhs); 
}; 

Location& Location::operator=(const Location &rhs) 
{ 
    // Only do assignment if RHS is a different object from this. 
    if (this != &rhs) 
    { 
     //Copy CArray 
     m_LocationsToSkip.Copy(rhs.m_LocationsToSkip); 

     //Copy rest of member variables 
     m = rhs.m; //will use Members automatically generated operator= 
        //which should do the correct thing because you only put 
        //normally copyable members in m 
    } 

    return *this; 
} 

は、私が最初にここにこのことについて投稿：https://stackoverflow.com/questions/469696/what-is-your-most-useful-c-c-utility/1609496#1609496