Question

所以我正在使用传送和通常的老鼠制作蛇游戏。我有一个像这样运行的循环：

while(snake.alive() && miceEaten < micePerLevel)
{
    displayInfo(lives, score, level, micePerLevel - miceEaten);
    //some code
    if(miceEaten())
    {
        //update score...
    }
    //more stuff...
}

上述代码的问题是在分数更新之前调用displayInfo，因此在吃完鼠标后，用户必须等到循环再次运行才能看到他的分数更新。所以我将一行代码移到了函数的底部：

while(snake.alive() && miceEaten < micePerLevel)
{
    //some code
    if(miceEaten())
    {
        //update score...
    }
    //more stuff...
    displayInfo(lives, score, level, micePerLevel - miceEaten);
}

和传送停止工作！只要蛇到达传送，程序就会崩溃。 displayInfo使用以下代码：

stringstream s;
s << "LEVEL " << left << setw(12) << level << "LIVES: " << setw(12) << lives << "MICE LEFT: " << setw(12) << miceLeft
    << "SCORE: " << setw(13) << score;
printLine(0, s.str(), WHITEONBLUE);

printLine只有color_set，mvprintw和refresh()。与传送无关。怪异。

所以我去了蛇功能，蛇从传送中获得了它的下一个位置：

    body.push_back(teleports[overlap(next)]->teleportFrom(dir)); //next is a Location object

teleports[overlap(next)]->teleportFrom(dir)返回蛇被传送到的位置。试图了解它崩溃的原因（也许Teleport在屏幕外返回一些位置？），我在上面的行之前添加了以下3行：

    Location l = teleports[overlap(next)]->teleportFrom(dir);
    mvprintw(1, 0, "(%i, %i)", l.x, l.y);
    refresh();

问题消失了！

不仅如此，我还必须拥有这三条线。如果我发表评论mvprintw(1, 0, "(%i, %i)", l.x, l.y);，或refresh();，或同时注释掉，该程序会在到达传送时崩溃。

关于可能导致此行为的任何想法？

更新：我尝试删除所有警告（主要是关于有符号/无符号数字比较的警告），但到目前为止只剩下1个：

warning: reference to local variable 'other' returned

代码：

Location& Location::operator = (Location other)
{
    if(this == &other)
        return other;
    x = other.x;
    y = other.y;
    return *this;
}

如何解决此警告？

Answer 1

像这样构建你的赋值运算符：
你应该总是返回* this（即使它们是相同的）。但是他们永远不会因为你创建一个本地副本（所以这不是你的错误）。

Location& Location::operator = (Location const& other)
{
    // Does it really matter if you assign to self?
    x = other.x;
    y = other.y;
    return *this;
}

对于这样一个简单的类，标准副本和交换似乎有点过分。

PS。您应该修复所有警告（即使它们像无符号不匹配一样简单）。如果你不修复它们，你就会对它们的效力免疫，并且不会发现真正的问题，因为它被你忽略的警告所包围。所以修复它们（aI总是打开标志，使编译器将所有警告视为错误，以便在有任何警告时代码不会编译）。

实现赋值运算符的正确方法（或最常用的好方法）。是使用复制和交换习语：

// notice the parameter is passed by value (i.e. a copy).
// So the copy part is aromatically taken care of here.
// So now you just need tom implement the swap() part of the idiom.
Location& Location::operator = (Location other)
{
    this->swap(other);
    return *this;
}

void Location::swap(Location& other)
{
    std::swap(x, other.x);
    std::swap(y, other.y);
}

Answer 2

Location& Location::operator = (Location other)
{
    if(this == &other)
        return other;
    x = other.x;
    y = other.y;
    return *this;
}

这会返回一个引用。当函数返回时，other会发生什么？（它死了，你什么也没说。）因为这是你在问题区域周围处理的类，这可能是原因。重新安排周围的代码使堆栈处于某种状态，其中引用死变量“工作”。

将其更改为return *this，或者只是完全取消检查。（在没有分支的情况下分配两个变量可能总是比在现代CPU上添加分支运行得快。）

（您通常也应该通过引用而不是按值来获取参数。）

Answer 3

您是否检查过导致此异常的代码？这里引用Heisenbug现象：

一个常见的例子是在使用优化编译器编译的程序中发生的错误，但在没有优化的情况下编译时不在同一程序中（例如，用于生成调试模式版本）

以下是一些指导原则：

比赛条件？你在使用线程吗？
指针溢出边界某处？
通过valgrind运行代码，以监控内存缓冲区中的任何异常/不稳定更改

另一句话：

类似heisenbug行为的一个常见原因是，在调试模式下执行程序通常会在程序启动之前清除内存，并将变量强制到堆栈位置，而不是将它们保存在寄存器中。这些执行上的差异可以改变涉及越界成员访问的错误或关于内存初始内容的错误假设的影响。另一个原因是调试器通常提供手表或其他用户界面，这些界面会导致执行额外的代码（例如属性访问器），这反过来又会改变程序的状态。另一个原因是核心上的fandango，指针超出界限的效果。在C ++中，许多heisenbugs是由未初始化的变量引起的。

确保关闭开关 - 没有优化，完整的调试信息，清除任何现有版本，重新启动IDE并重新编译....

Answer 4

首先，你的Location :: operator =应该是这样的：

Location& Location::operator = (const Location &other)
{
    if(this == &other)
        return *this;
    x = other.x;
    y = other.y;
    return *this;
}

然而，这可能并不能解释崩溃。堆栈上的错误指针在大多数架构上都不会崩溃（假设x和y是int）。

现在，这是一个mandelbug，而不是heisenbug。你有别人在某处腐蚀记忆。祝你好运。

神秘的heisenbug？

4 个答案: