賦值運算符在

Question

我已經使用賦值運算符重載下面的代碼C++超載：

SimpleCircle SimpleCircle::operator=(const SimpleCircle & rhs) 
{ 
    if(this == &rhs) 
     return *this; 
    itsRadius = rhs.getRadius(); 
    return *this; 
} 

我的拷貝構造函數是這樣的：

SimpleCircle::SimpleCircle(const SimpleCircle & rhs) 
{ 
    itsRadius = rhs.getRadius(); 
} 

在上面的操作符重載的代碼，拷貝構造函數被調用因爲有一個新的對象正在創建;所以我用下面的代碼：

SimpleCircle & SimpleCircle::operator=(const SimpleCircle & rhs) 
{ 
    if(this == &rhs) 
     return *this; 
    itsRadius = rhs.getRadius(); 
    return *this; 
} 

其完美的工作，避免拷貝構造函數問題，但有這方面的任何未知的問題（我）？

Answer 1

沒有與賦值運算符的第二個版本沒有問題。實際上，這是賦值運算符的標準方式。

編輯：請注意，我指的是賦值運算符的返回類型，而不是實現本身。正如評論中指出的那樣，執行本身是另一個問題。見here。

Answer 2

二是相當標準。您通常更願意返回賦值運算符的引用，以便按預期的方式解析像a = b = c;這樣的語句。我想不出任何我想從作業中返回副本的情況。

有一點需要注意的是，如果你不需要深拷貝它有時被認爲是最好使用編譯器比滾你自己所產生的隱含拷貝構造函數和賦值操作符。真的取決於你雖然...

編輯：

這裏的一些基本通話費：

SimpleCircle x; // default constructor 
SimpleCircle y(x); // copy constructor 
x = y; // assignment operator 

現在說我們有你的賦值運算符的第一個版本：

SimpleCircle SimpleCircle::operator=(const SimpleCircle & rhs) 
{ 
    if(this == &rhs) 
     return *this; // calls copy constructor SimpleCircle(*this) 
    itsRadius = rhs.getRadius(); // copy member 
    return *this; // calls copy constructor 
} 

它調用複製構造函數並將參考傳遞給this以構造要返回的副本。現在，在第二個例子中，我們避免了拷貝由剛剛返回一個參考this

SimpleCircle & SimpleCircle::operator=(const SimpleCircle & rhs) 
{ 
    if(this == &rhs) 
     return *this; // return reference to this (no copy) 
    itsRadius = rhs.getRadius(); // copy member 
    return *this; // return reference to this (no copy) 
} 

Answer 3

在這種情況下，你幾乎可以肯定是最好跳躍的自我分配的檢查 - 當你只分配一個成員，這似乎是一個簡單的類型（可能是一個雙），它通常更快的做任務不是避開它，所以你最終獲得：

SimpleCircle & SimpleCircle::operator=(const SimpleCircle & rhs) 
{ 
    itsRadius = rhs.getRadius(); // or just `itsRadius = rhs.itsRadius;` 
    return *this; 
} 

我認識許多老人和/或低質量書籍建議檢查自我分配。然而，至少在我的經驗中，如果沒有它，你會變得更好（並且如果操作員依賴它來保證正確性，這幾乎肯定不是例外）。

順便說一句，我會注意的是定義一個圓，你一般需要一箇中心和半徑，當你複製或分配，要複製/分配兩者。

Answer 4

這是正確的方式來使用運營商超載 現在你通過引用 避免價值複製你的對象。

Answer 5

，這可能會有所幫助：

// Operator overloading in C++ 
//assignment operator overloading 
#include<iostream> 
using namespace std; 

class Employee 
{ 
private: 
int idNum; 
double salary; 
public: 
Employee () { 
    idNum = 0, salary = 0.0; 
} 

void setValues (int a, int b); 
void operator= (Employee &emp); 

}; 

void Employee::setValues (int idN , int sal) 
{ 

salary = sal; idNum = idN; 

} 

void Employee::operator = (Employee &emp) // Assignment operator overloading function 
{ 
salary = emp.salary; 
} 

int main () 
{ 

Employee emp1; 
emp1.setValues(10,33); 
Employee emp2; 
emp2 = emp1; // emp2 is calling object using assignment operator 

}