Move constructor und double delete

Question

Anfangs dachte ich, dass move constructor den temporären Objektdestruktor nicht aufrufen würde, aber wenn ich es versuche, ruft er den Destruktor auf. Wenn wir also die Daten von move constructor stehlen, bekomme ich doppelten Löschfehler.

#include <iostream> 
using namespace std; 

class A 
{ 
    public: 
    A() 
    : name("default") 
    { 
     cout<<"i am default\n"; 
     data = new char[20]; 
    } 

    A(A&& t) 
    : name("move") 
    { 
     data = t.data; 
     cout<<"i am move\n"; 
    } 

    ~A() 
    { 
     delete data; 
     cout<<"I am done:"<<name<<endl; 
    } 

    char * data; 
    string name; 
}; 

A getA() 
{ 
    A obj; 
    return obj; 
} 

int main() 
{ 
    A test(std::move(getA())); 
} 

Answer 1

Das ist, weil Sie nicht wirklich sind „stehlen“, du bist nur zu kopieren, und so werden Sie 2 mal den gleichen Zeiger löschen, wie Sie bemerkt.

Um die Daten zu "stehlen", setzen Sie die Originaldaten auf nullptr, da sie nicht mehr zu diesem Objekt gehören.

A(A&& t) 
: name("move") 
{ 
    data = t.data; 
    t.data = nullptr; //'t' doesn't own its data anymore 
    cout<<"i am move\n"; 
} 

Sie könnten auch std::swap (dank @RemyLebeau) verwenden:

A(A&& t) : name("move"), data(nullptr) 
{ 
    std::swap(data, t.data); 
    cout << "i am move\n"; 
}