有人可以告诉我下面的代码（C++）有什么问题吗？

Question

除非我没有为类定义析构函数，否则下面的代码可以正常工作。 但是当我定义析构函数时它会产生错误的结果。在析构函数部分，我释放了在构造函数中创建的内存。我认为这是我的工作。但在这种情况下，这样做会使我的程序以不需要的输出运行。我对这个计划感到非常沮丧。请帮帮我。我无法弄清楚代码有什么问题。一切对我来说似乎都还好。 在这里，我制作了一个列表类型的对象数组。然后我在构造函数的帮助下分别为对象赋值。但似乎这些值没有得到适当的初始化。但我认为初始化过程没有犯任何错误。然后，当我使用成员函数显示打印对象数据时，它无法产生所需的结果。尽管它正确打印了 int 和 float 数据成员。它无法正确打印 char* 数据成员。

#include <iostream>
#include <cstring>
#include <iomanip>
using namespace std;

class list
{
private:
    char*  name;
    int code;
    double cost;
public:
    //constructor
    list()
    {

    }
    list(const char* name,int code,double cost)
    {
        int len=strlen(name);
        this->name=new char[len+1];
        strcpy(this->name,name);
        this->code=code;
        this->cost=cost;

    }
    //mem fun
    void display(void)
    {
        cout.precision(2);
        cout.setf(ios::fixed);
        cout << setw(20) << this->name << setw(10) << this->code << setw      (12)    << cost << endl;
    }
    //destructor
    ~list()
    {
        delete[] this->name;
    }
};

int main()
{
    list item[3];

    item[0]=list("Turbo C++",1001,250.95);
    item[1]=list("C Primer",905,95.70);
    item[2]=list("C++ Book",105,30);

    cout << setw(20) << "NAME" << setw(10) << "CODE" << setw(12) << "COST"   <<     endl;

    for(int i=0;i<3;i++)
        item[i].display();

    return 0;
}

Answer 1

您正在复制所有列表项，因为您有一个对象数组（不是指针） - 因为它只是一个“浅拷贝”，所以您有两个实例具有相同的 name 指针，因此是“双重释放”消息.

有几种方法可以解决它：

• 您可以定义一个operator= 来生成重复的字符串。那 工作，但你仍然有不必要的副本。
• 您可以在main 中有一个指针数组（而不是实例）。
• 您可以使用set() 方法在现有实例中设置值（例如item[0].set("Turbo C++", 1001, 250.95);）