c ++有奇怪的问题

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.isABerserkeror Enemy3.hasHadLeftLegCutOffThreeInchesBelowTheKnee，但这只是个人喜好）。

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

换句话说，如果线程 1 正在执行推送操作，而线程 2 遇到该if语句，则线程 2 也将开始推送值，因为线程 1 尚未设置dEnemy为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)给你自纪元以来的秒数，因此，直到那一秒结束之前，你很可能会执行该内容if 数千次（或更多次）。