Question

我有一个函数可以在向量中创建和插入一些数字。

    if(Enemy2.dEnemy==true)
    {
        pt.y=4;
        pt.x=90;
        pt2.y=4;
        pt2.x=125;
        for(int i=0; i<6; i++)
        {
            Enemy2.vS1Enemy.push_back(pt);
            Enemy2.vS2Enemy.push_back(pt2);
            y-=70;
            pt.y=y;
            pt2.y=y;
        }
        Enemy2.dEnemy=false;
        Enemy3.cEnemy=0;
    }

它应该在两个向量中插入6个数字，唯一的问题是它不 - 它实际上插入更多。

除非Enemy2.dEnemy == true，否则我认为该代码段不会运行，并且它永远不会成为现实。

代码段第一次运行时，Enemy2.dEnemy设置为false，不应该再次运行。

除非创建窗口，否则我不会将Enemy2.dEnemy设置为true。

如果我在片段中的任何位置插入断点，程序将正常工作 - 它将在两个向量中仅插入6个数字。

任何想法在这里有什么问题？

<小时/> 好的，所以我做了一些调试我发现Enemy2.dEnemy=false;由于某种原因被跳过了我试着这样做，看看是不是。

 if(Enemy2.dEnemy)
    {
        pt.y=4;
        pt.x=90;
        pt2.y=4;
        pt2.x=125;
        for(int i=0; i<6; i++)
        {
            Enemy2.vS1Enemy.push_back(pt);
            Enemy2.vS2Enemy.push_back(pt2);
            y-=70;
            pt.y=y;
            pt2.y=y;
        }
        TCHAR s[244];
        Enemy2.dEnemy=false;
        if(Enemy2.dEnemy)
        {
          MessageBox(hWnd, _T("0"), _T(""), MB_OK);
        }
        else
        {
            MessageBox(hWnd, _T("1"), _T(""), MB_OK);
        }
        Enemy3.cEnemy=0;
    }

弹出消息框说1，我的代码工作正常。似乎Enemy2.dEnemy=false;没有时间运行; /
blahblahblahblahblahblahblahblahblahblahblahblahblahblahblahblah！好吧，我发现导致插入超过6个数字的真正问题在哪里。这是我作为Enemy2.dEnemy=true;

的地方

if(Enemy2.e1)
{
Enemy2.now=time(NULL);
Enemy2.tEnemy=Enemy2.now+4;
Enemy2.e1=false;
}
if(Enemy2.tEnemy==time(NULL))
{
check=1;
Enemy2.aEnemy=0;
Enemy2.dEnemy=true;
}

问题似乎是第二次运行不止一次，这很奇怪！

Answer 1

首先要做的事情是：摆脱那个可恶的if (Enemy2.dEnemy == true) - 它应该是：

if (Enemy2.dEnemy)

（我也更喜欢将我的布尔语命名为Enemy2.isABerserker或Enemy3.hasHadLeftLegCutOffThreeInchesBelowTheKnee这样的可读段，但这只是个人偏好。

除此之外，我唯一可以建议的是线程问题。该代码本身没有任何问题，但是有一个窗口，其中两个线程可以进入if语句并且都开始将值推送到向量中。

换句话说，如果线程1在线程2遇到if语句时正在执行推送，则线程2也将开始推送值，因为线程1尚未将dEnemy设置为{{1 }}。并且不要认为您可以将分配移动到true块的顶部 - 这将减少但不删除窗口。

我的建议是在有超过六个条目的情况下打印出向量的内容，这可能会给出发生了什么的线索（如果你愿意，可以在这里发布输出）。

重新启动以下第二个if正在运行两次的更新：

if

如果此代码在同一秒内执行两次（并且不超出可能范围），则第二个if(Enemy2.e1) { Enemy2.now=time(NULL); Enemy2.tEnemy=Enemy2.now+4; Enemy2.e1=false; } if(Enemy2.tEnemy==time(NULL)) { check=1; Enemy2.aEnemy=0; Enemy2.dEnemy=true; }语句将运行两次。

那是因为if给你自纪元以来的秒数，直到那一秒结束，你可能正在执行time(NULL) 数千的内容次（或更多）。

Answer 2

如果在输入断点或诊断输出消息时此问题消失，那么这是一个强有力的线索，即问题是未定义的行为，这通常是由取消引用未初始化的指针或不小心使用const_cast引起的。

问题的原因可能与您正在查看的代码无关。它是在其他地方引起的，恰好出现在这里。这就像有人被摔倒的砖块击中：明显的症状是一名男子躺在人行道上无意识，但真正的问题与男子或人行道无关，这是几个故事。

如果要查找错误原因，请删除诊断程序，直到问题再次出现，然后开始删除其他所有内容。修剪掉所有其他代码。每当错误停止时，请备份直到它再次启动;如果您没有看到错误原因，请在其他地方开始修剪。最终这个bug无处可藏。

c ++有奇怪的问题

2 个答案: