为什么我在 C++ 中收到指针数组的“未分配指针”错误？

Question

我正在使用一个动态的指针数组。该项目需要此设置。当我执行时，我收到一个错误，即未分配被释放的指针。

更新为添加动态代码。当我尝试访问数组时，函数会抛出错误的访问错误，并在删除动态数组时出现段错误。

class Person
{
private:
string firstName;
string lastName;
int age;
int telNum;
int numAddrs;
//Address* addrs[10];
Address* addrs;

public:
Person(string, string, int, int);
Person();
Person(const Person &p);

~Person();

void setFirstName(string);
void setLastName(string);
void setAge(int);
void setTelNum(int);
void setAddress(Address*);
void setNumAddrs(int);

string getFirstName();
string getLastName();
int getAge();
int getTelNum();
Address* getAddress();
int getNumAddrs();

void displayPerson();
void addAddress(Address &newAddr);

Person& operator =(const Person &RHS);


};

Person::Person()                                //default
{
firstName = "John";
lastName = "Doe";
age = -1;
telNum = -1;

numAddrs = 0;               //current number of addresses for this person

addrs = new Address*[10](); //throws error, won't compile
//new dynamic array of pointers, initialize to NULL

}

Person::Person(string first, string last, int y, int t)
{                                               //constructor to  initialize              Person
firstName = first;
lastName = last;
age = y;
telNum = t;

numAddrs = 0;                   //current number of addresses for this     person

addrs = new Address*[10](); throws error, won't compile
//new dynamic array of pointers, initialize to NULL

}


Person::Person(const Person &p)     //copy constructor
{
this->firstName = p.firstName;
this->lastName = p.lastName;
this->age = p.age;
this->telNum = p.telNum;
this->numAddrs = p.numAddrs;


delete [] addrs;        //delete previous dynamically allocated memory

addrs = new Address*[10](); //throws error, won't compile
//new dynamic array of pointers,     initialize to NULL

for (int i = 0; i<10; i++)    //copy addresses to new object
{
this->addrs[i] = p.addrs[i];
}

}

Answer 1

addrs 不是指针，而是数组。您不需要使用new 为其分配空间，因为它的内存是作为类对象的一部分分配的。它是一个指针数组，所以需要在构造函数中为每个元素分配内存：

for (int i = 0; i < 10; i++) {
    addrs[i] = new Address;
}

然后在析构函数中，你需要删除这些地址中的每一个。

for (int i = 0; i < 10; i++) {
    delete addrs[i];
}

但是，有一个问题。您的赋值运算符将所有 addrs[i] 从 src 复制到目标。所以现在你有两个 Person 对象，它们指向相同的 Address 对象。如果其中任何一个被破坏，它将释放所有Address 对象，然后另一个Person 将在其addrs 数组中包含无效指针。

不清楚为什么你需要一个指针数组而不是一个 Address 对象数组：

Address addrs[10];

那么你就不需要使用new 或delete。

此外，在现代 C++ 中，通常认为使用 std::vector 之类的容器会更好。您还可以使用智能指针类来处理在多个容器之间共享同一个指针。