デストラクタはどこにありますか？

Question

私はC++を学んでおり、クラスでオブジェクトを使いたいと思っています。私のプログラムの出力は、私が予想していたので、データデストラクタが2回呼び出されることを私に混乱させます。私のオブジェクトは、 "データデータ（3）;"コードブロックの最後でデストラクタを呼び出す必要があり、またrectsデストラクタが呼び出されると、rect内のデータのデストラクタも呼び出される必要があります。私が期待したように、コードブロックが終了してから2回、「データデストラクタ」を印刷します。しかし、それはまた、私のデータオブジェクトが作成された後、それを印刷します。なぜこれが起こり、データ内の割り当てられたメモリを正しく削除することができますか？

class Data { 
private: 
    int* data; 
    int size; 
public: 
    Data() { 
     size = 10; 
     data = new int[size]; 
    } 
    Data(int size) 
     : size(size) { 
     data = new int[size]; 
    } 
    virtual ~Data() { 
     // delete data; 
     std::cout << "data destruktor" << std::endl; 
    } 
    void init(int val) { 
     for (int i = 0; i < size; i++) 
      data[i] = val; 
    } 
    int& operator[](int index) { 
     return data[index]; 
    } 
    int getSize() const { 
     return size; 
    } 
}; 

class Rect { 
private: 
    Data data; 
    int width, height; 
public: 
    Rect() {} 
    Rect(int width, int height, Data data, int val = 0) 
     : width(width), height(height), data(data) { 
     data.init(val); 
    } 
    int getWidth() const { 
     return width; 
    } 
    int getHeight() const { 
     return height; 
    } 
    int getArea() const { 
     return width * height; 
    } 
    Data& getData() { 
     return data; 
    } 
}; 

int main() { 

    { 
     Data data(3); 
     std::cout << "data created" << std::endl; 
     Rect rect(5, 4, data, 9); 
     std::cout << "end of code block" << std::endl; 
    } 

    std::cout << "end of program" << std::endl; 

    system("pause"); 
    return 0; 
} 

出力：

data created 
data destruktor 
end of code block 
data destruktor 
data destruktor 
end of program 

Answer 1

あなたはコンストラクタの呼び出し後に破壊し、そのため、あなたdesctructor呼び出しでされている、のRectのc'torにデータのコピーを渡しているためです。

Answer 2

Rect(int width, int height, Data data, int val = 0) 

現在値によってDataを渡しているので、あなたは、このメソッドを呼び出したときに新しいものが作成され、失われます。それは次のようになります。

Rect(int width, int height, Data &data, int val = 0) 

または可能性もconst Data &data：Rectはすでに、独自のものを持っているときにDataを渡している理由は明らかではありません。たぶん、あなたはそれがコピー割り当てること、またはコピー構築し、それをshoudl：

Rect(int width, int height, const Data &data, int val = 0) 
: data(data) 
// ... 

NBの@ P0Wで指摘したようにあなたが実際にdelete[] data;として、delete data;ラインが必要です。 Data::Data(const Data&)のコピーコンストラクタも同様です。