为什么我的堆已损坏？

Question

我收到错误 - 堆损坏，无法弄清楚原因。

我的基地：

h：

class Base
{
public :
    Base(char* baseName, char* cityName);
    virtual ~Base();

    list<Vehicle*>::const_iterator GetEndList();
    void PrintAllVehicles(ofstream &ResultFile) const;
    char* GetBaseName() const;
    char* GetLocation() const;
    void InsertNewVehicleToBase(Vehicle* newVehicle);
    list<Vehicle*>::const_iterator FindVehicle(char* id);
    void RemoveVehicle (list<Vehicle*>::const_iterator beg);



 private:
    char* m_name;
    char* m_location;
    list<Vehicle*> m_baseVehicles;

};  

cpp：

Base::Base(char* baseName, char* cityName)
{
    m_name = new char [strlen(baseName)+1];
    strcpy(m_name, baseName);
    m_location = new char [strlen(cityName)+1];
    strcpy(m_location, cityName);
}

Base::~Base()
{
    delete [] m_name;
    delete [] m_location;
    //m_baseVehicles.clear();
}

军队析构函数：

Army::~Army()
{
    list<Base*>::iterator baseIter = m_basesList.begin();
    for (baseIter ; baseIter != m_basesList.end() ; ++baseIter)
        delete (*baseIter);
    m_basesList.clear();
 }  

我做错了什么？

Answer 1

您所显示的代码没有明显的错误，因此错误很可能出在您未显示的代码中。

对我来说，最直接可疑的事情是 Base 类拥有两个指针，并且没有定义复制构造函数或赋值运算符。这意味着，如果您复制 Base 对象，最终将得到两个指向相同数据的 Base 对象，并且当它们析构时，它们将删除它两次，导致堆损坏。

Army 类也可能有这个问题（因为它拥有几个 Base 指针），但是您没有显示该类的定义，因此不清楚是否它是否有复制构造函数和赋值运算符。

最后，您还没有显示 Base 对象的分配位置。它们是否有可能被传递到 Army 对象中，并在 Army 对象之外的某个位置被删除？或者，Army 对象包含的 Base* 可能引用了堆栈上不应删除的对象？

Answer 2

此代码的可见问题：

• 使用 char* 而不是 std::string 需要手动内存管理
• 使用
  STL 容器中的原始指针
  清理代码过于复杂
• 使用 CRT 进行字符串操作使
  C++“代码味道”

Answer 3

给出的代码片段没有任何问题。
但使用这种代码，多次删除的可能性很高，我的意思是两次删除内存块，这会导致堆损坏。

Answer 4

您没有发布有问题的代码部分，因为所有这些看起来都很正常。然而，它有很多 const 正确性问题：

Base(char* baseName, char* cityName);

字符串应该作为 const char* 传递，除非它们被修改。

virtual ~Base();

不知道这是否需要虚拟；看不到它的子类是什么。

list<Vehicle*>::const_iterator GetEndList();

应该是一个 const 方法，因为它是一个 const_iterator： list::const_iterator GetEndList() const;

char* GetBaseName() const;
char* GetLocation() const;

这些应该返回 const char*，因为您的代码未设置为处理正在更改的名称和位置。

list<Vehicle*>::const_iterator FindVehicle(char* id);

同样，应该是一个 const 方法： list::const_iterator FindVehicle(char* id) const;

Base::~Base()
{
    delete [] m_name;
    delete [] m_location;
    //m_baseVehicles.clear();
}

你不需要 m_baseVehicles.clear();因为它无论如何都会在析构函数之后发生。但是，如果其他地方没有引用这些车辆，则需要将其删除，否则会发生泄漏。

军队破坏者：

“破坏者”。其余的陆军在哪里？

Army::~Army()
{
    list<Base*>::iterator baseIter = m_basesList.begin();
    for (baseIter ; baseIter != m_basesList.end() ; ++baseIter)
        delete (*baseIter);
    m_basesList.clear();
 }  

同样，您不需要 m_basesList.clear();。

Answer 5

我没有看到任何问题。正如马特·凯恩所说，它是如何填充的？

Answer 6

堆损坏是由于将比分配给该单元的内存更多的数据复制到已分配的堆单元中。堆单元的开头和结尾包含的数据在被覆盖时将被报告为堆损坏。

您还没有发布所有代码，我在您发布的代码中看不到问题，但我建议使用像 Valgrind 这样的内存工具来帮助您诊断问题。