检测到堆损坏

Question

我的程序的输出唯一不正确的是有一个“！” “钓鱼”之后。我尝试过调试它，但从来没有“！”记忆中。

这是该行的输出

One more: gone down to the fishing! hole

这是 s6 和 s7 的创建

MyString s6("gone ");
MyString s7("fishing");

这是生成语句的行

cout << "One more: " << s6 + "down to the " + s7 + " hole" << endl << endl;

这是 <<运算符重载函数

ostream& operator<<(ostream& leftOp, const MyString& rightOp)
{
leftOp << rightOp.stringArray;

return leftOp;
}

这是 + 运算符重载函数

MyString MyString::operator+(const char* rightOp) const
{
    MyString result; // new object used to store result
    result.stringSize = stringSize + strlen(rightOp);
    // if the string does not fit in the array
    if( result.stringSize > result.stringCap )
    {
        delete[] result.stringArray;
        result.stringCap = ( stringSize + 15 ) & ~15;
        result.stringArray = new char[stringCap + 1];
    }
    strcpy(result.stringArray, stringArray);
    strcat(result.stringArray, rightOp);
    return result;
}

s7 在程序中的其他任何地方都没有调用，所以我认为不再需要显示代码。任何帮助将不胜感激。

Answer 1

很难判断您的代码是否存在问题。
我会看看其他成员。 你遵守了3规则吗？

注意：

我想指出您违反了基本的面向对象规则。

除非有非常充分的理由，否则您不应该摆弄另一个对象的成员。
这里你的对象正在摆弄结果。

代码会更干净

MyString MyString::operator+(const char* rightOp) const
{
    MyString result(*this);   // make a copy of this.
    result += rightOp;        // Let result fiddle with its own members here.
    return result;
}

如果您将其编写为：编辑：

基于下面的评论，

。 三法则

基本上：如果你的对象拥有动态分配的内存（即它调用new/delete) 那么编译器生成的默认版本的方法就不能按你想要的方式工作；并且您应该定义自己的版本：

基本上：

* Copy Constructor
* Assignment Operator
* Destructor

* Not part of rule of 3, but you probably also need a normal constructor.

简单的事情是您可能已经有一个析构函数（否则就不会损坏），并且可以根据复制构造函数编写赋值运算符。因此，您所需要做的就是编写正确版本的复制构造函数，一切都应该可以正常工作。

/*
 * Assignment operator using Copy and swap Idiom.
 * Copy uses copy constructor (here done in pass by value)
 *
 * You then just swap the current content with the copy
 */
MyString& MyString::operator=(MyString rhs)
{
    (*this).swap(rhs);
    return *this;
}

Answer 2

如果将 stringCap 分配给数组，您介意“\0”吗？存储字符串的 char* 必须是大小为 size-of-string + 1 的数组

问题可能出在连接两个 MyString 时。第二个字符串“钓鱼”可能具有太大的 stringSize 值。

而不是 result.stringCap = ( stringSize + 15 ) & ~15; 为什么不简单地写：

result.stringCap = result.stringSize + 1;

Answer 3

if( result.stringSize > result.stringCap ) 应为 if( result.stringSize >= result.stringCap ) 以允许 null 终止。您还需要重置 result.stringSize。

Answer 4

正常情况下你没有分配stringArray，太大的情况下你没有重置stringSize，strncpy更安全：

MyString MyString::operator+(const char* rightOp) const
{
    MyString result; // new object used to store result
    result.stringSize = stringSize + strlen(rightOp);
    // if the string does not fit in the array
    if( result.stringSize > result.stringCap )
    {
        delete[] result.stringArray;
        result.stringCap = ( stringSize + 15 ) & ~15;
        // don't allocate here
        result.stringSize = stringCap;
    }

    // always allocate for the new string
    result.stringArray = new char[result.stringSize + 1];

    strncpy(result.stringArray, stringArray, result.stringSize);
    strncat(result.stringArray, rightOp, strlen(rightOp) + 1);
    return result;
}

编辑：好的，忽略那。我对 MyString 类做了一些假设，但事实显然并非如此。所以，我现在可以指出的是：

if( result.stringSize > result.stringCap )
{
    delete[] result.stringArray;
    result.stringCap = ( stringSize + 15 ) & ~15;
    result.stringArray = new char[stringCap + 1]; // WRONG stringCap!
    result.stringArray = new char[result.stringCap + 1]; // right stringCap
}

您可以尝试简化事情，以缩小问题范围：

cout << "One more: " << s6 + "down to the " << s7 + " hole" << endl << endl;

也许问题出在 operator+(const MyString&) 而不是 operator+(const char *）？