c++ - 删除缓冲区溢出 []

Question

我试图搜索相同的问题，但没有人帮助我。当我运行程序时，出现“发生缓冲区溢出...”错误。

构造函数:

Player(char* n)
{
   length = strlen(n);
   name = new char[length+1];

   for(unsigned int i(0); i < length; i++)
       name[i] = n[i];

   name[length] = '\0';
}

破坏:

~Player(void)
{
   delete [] name;
}

我的字符串以 NULL 结尾，但没有越界，有什么问题吗？

Answer 1

您发布的代码中没有明显的错误，但尝试通过处理原始指针来管理动态内存几乎不可避免地会导致此类错误。

根据 Rule of Three，可能您没有正确实现或删除复制构造函数和复制赋值运算符.在这种情况下，复制一个 Player 对象会得到两个指向同一个数组的对象；他们都将尝试删除该数组，给出未定义的行为。

最简单的解决方案是使用专为管理字符串而设计的类来管理您的字符串。将 name 的类型更改为 std::string，然后构造函数可以简单地类似于

explicit Player(std::string const & n) : name(n) {}

而且根本不需要声明析构函数(或移动/复制构造函数/赋值运算符)。