環路

Question

我用下面的類拷貝構造函數調用：

#include <list> 

class A { 
public: 
    A(); 
    A(const A&); 
    ~A(); 
    list<A> createCopies(); 

private: 
    int *cells; 
}; 

A::A() { 
    this->cells = new int[5]; 
    for(unsigned int i = 0; i < 5; i++) { 
     this->cells[i] = 0; 
    } 
} 

A::(const A &oldA) { 
    this->cells = new int[5]; 
    for(unsigned int i = 0; i < 5; i++) { 
     this->cells[i] = oldA.cases[i]; 
    } 
} 

A::~A() { 
    delete[] this->cells; 
} 

list<A> A::createCopies() { 
    list<A> listA; 

    for(unsigned int i = 0; i < 3; i++) { 
     A newA = A(*this); // or A newA(*this)? 
     A.cells[i] = 1; 
     listA.push_back(newA); 
    } 

    return listA; 
} 

然而，編譯器似乎總是推同一個對象在列表中。我認爲對拷貝構造函數的調用會阻止...

我錯了嗎？我是否需要使用創建對象的方式（並因此在某處管理其銷燬的痛苦）？

Answer 1

您遇到不確定的行爲，因爲你使用std::array雖然忘記

return listA; 

你最好。如果你選擇保持這種方法，你還需要一個賦值操作符。

我假設你的實際成員是int* cells;。