kopyalama ve takas tekniği atama operatörü işlevi içinde kopya kurucu kullanır

Question

Ben Madde 11 benim atama operatörü içinde "kopyalama ve takas" tekniğini kullanmaktır öneren olduğu "Scott Meyers tarafından Etkili C++", okuyordu:

Widget& Widget::operator=(const Widget &rhs) 
{ 
    Widget temp(rhs); // Copy constructor 
    swap(temp); //Swap with *this 
    return *this; 
} 

Ama Item 12'de yazılıdır:

Bu kopya atama operatörü kopya kurucu çağırmak için hiçbir mantıklı.

Öğe 11 ve Öğe 12'nin çelişkili olduğunu düşünüyorum. Yanlış anlıyor muyum?

Answer 1

Tam tersini yapıyorum. Atama operatörüne "derin" bir kopya yapmak için aşırı yük veriyorum (yani, dinamik belleği dikkate aldığınız ve yeni kopyaları buna göre başlatacağınız anlamına gelir). Ardından kopya oluşturucudaki atama işlecini kullanın. Metninize yorum yapamayacağım çünkü o kitabım yok.

Widget::Widget(const Widget& existing) : dynamicThingie(0) 
{ 
    *this = existing; 
} 


Widget& Widget::operator=(const Widget& existing) 
{ 
    a = existing.a; 
    b = existing.b; 

    delete dynamicThingie; 
    dynamicThingie = new Thingie(); 
    memcpy(dynamicThingie, existing.dynamicThingie, lengthOfDynamicThingue); 

    return *this; 
} 

Answer 2

"Etkili C++ tarafından Scott Meyers" in 2 atıfta yer aldığı 2 farklı yönü ele alınmıştır.

• Öğe 11'deki kod parçasında Scott, atama işlecini gerçekleştirmenin yollarından birini gösterir.
• Öğe 12'de, atıf yaptığınız atıf, atama operatörü ve kopya yapıcısı arasında kod çoğaltmasından kaçınılmasıyla ilgilidir.

iki kopyalama fonksiyonları genellikle benzer organları olacak ve bu bir fonksiyon diğer çağırmasını sağlayarak kod tekrarını önlemek için denemek için sizi baştan olabilir: alıntının üstünde bir paragraf diyor. Madde 12 bu kısmı benim anlayış

şu şekildedir: Aşağıda gibi bir şey yazmaya çalışırsanız (kopyalama atama operatörü görüşmesi kopya yapıcı olması) o zaman yanlış olur:

PriorityCustomer::PriorityCustomer(const PriorityCustomer& rhs) 
: Customer(rhs), // invoke base class copy ctor 
    priority(rhs.priority) 
{ 
    logCall("PriorityCustomer copy constructor"); 
} 

PriorityCustomer& 
PriorityCustomer::operator=(const PriorityCustomer& rhs) 
{ 
    logCall("PriorityCustomer copy assignment operator"); 
    this->PriorityCustomer(rhs); // This line is wrong!!! 
    return *this; 
}