Przenoszenie w C++

Question

Witam

Czy tak powinno działać przenoszenie ? 

Konstruktor Przenoszący:

Useless::Useless(Useless && f)
{
    cout<<endl<<"Konstruktor przenoszacy."<<endl;
    pc=f.pc;
    n=f.n;
    f.n=NULL;
    f.pc=NULL;

    ShowObject();
}

Operator+ :

Useless Useless::operator+(Useless & f)const
{
    Useless temp;

    temp.n=f.n;
    temp.pc=f.pc;    

    f.n=NULL;
    f.pc=NULL;    

    return temp;       
}

Za wszelką odpowiedz dzięki ;)

Answer 1

Kostruktor jest ok. Tylko używaj listy inicjalizacyjnej.

operator+: Biorąc pod uwagę, że to jest operator+, to... w życiu bym się nie spodziewał, że dodawanie będzie działać w ten sposób i na dodatek modyfikować jeden z operandów.
Pomijając to... Nie bardzo rozumiem dlaczego argument jest zwykłą referencją.

Zauważ też, że masz duplikacje kodu. Cały kod operatora+ sprowadza się do:
 

Useless Useless::operator+(Useless & f)const
{
    return Useless(std::move(f));    
}

Comments

Mam coś podobnego w książce "Szkoła programowania C++...", ale trochę bardziej niepotrzebnie skomplikowanie napisany jest ten operator+ dlatego zrobiłem coś prostszego i po teście z std::move() i mojego operatora+ wynik jest ten sam i się dziwie czy napewno  wszystko jest ok :P

---

std::move sprowadza się po prostu do castowania na rvalue reference. Także użyty jest po to, żeby uruchomił się move konstruktor.