C++ 在 if 语句中分配 bool 条件值不起作用？

Question

我一直在使用 SDL 库用 C++ 编写游戏代码。今天，在改变我的玩家角色类的工作方式时，我遇到了一个非常令人费解的问题。以下代码构成了我允许玩家发射子弹的逻辑的一部分。控制变量 b_canFire 和 b_shouldFire(我打算重命名它们以使其更有意义)在类的其他地方设置，以允许此函数在用户按下某个键时执行。

bool PlayerChar::DoFiring()
{
    if(b_canFire && b_shouldFire)
    {
        Fire(box.x + 22, box.y); // This fires a bullet
        b_canFire = false; // Does not work
        b_shouldFire = false; // Does not work

        return true;
    }
}

当我使用我的调试器单步执行这段代码时，我发现 b_canFire 和 b_shouldFire 的值没有被 if 语句中的赋值更改为 false。但是，如果我将代码更改为以下内容:

bool PlayerChar::DoFiring()
{
    if((b_canFire) && (b_shouldFire))
    {
        Fire(box.x + 22, box.y); // This fires a bullet
        b_canFire = false; // Works
        b_shouldFire = false; // Works

        return true;
    }
}

或

bool PlayerChar::DoFiring()
{
    if(b_canFire == true && b_shouldFire == true)
    {
        Fire(box.x + 22, box.y); // This fires a bullet
        b_canFire = false; // Works
        b_shouldFire = false; // Works

        return true;
    }
}

突然间作业开始工作了。我也试过在一个空的测试项目中复制这种情况，如下所示:

bool bC = true;
bool bD = true;

if(bC == true && bD == true)
{
    bC = false; // Works
    bD = false; // Works
}


bool bE = true;
bool bF = true;

if(bE && bF)
{
    bE = false; // Works
    bF = false; // Works
}

但是，这两个示例都完全按照它们应有的方式分配了值。显然我在这里遗漏了一些东西，但对于我来说，我看不到它是什么。我已经想出如何解决问题以使我的代码正常工作，但我不知道是什么破坏了第一个示例中的分配，这让我很困扰，因为到目前为止我所了解的有关 C++ 的一切都告诉我它们应该可以正常工作。

这是我使用 C++ 语言的第一个主要项目，我仍在学习中，所以如果有经验丰富的程序员提供任何帮助或建议，那将是非常好的。

谢谢!

编辑:

这里要求的是整个类(class):

#include <list>
#include "SDL_mixer.h"
#include "hiGlobalVars.h"
#include "hiGlobalObjects.h"
#include "hiAssetManager.h"
#include "hiRendering.h"
#include "hiTimer.h"
#include "hiBullet.h"
#include "hiPlayerChar.h"
#include "hiDebugger.h"


using std::list;


PlayerChar::PlayerChar()
{
    moveSpeed = 6;
    moveDir = NONE;
    b_canFire = true;
    b_shouldFire = false;
    box.x = 0;
    box.y = 470;
    box.w = 38;
    box.h = 40;
}


PlayerChar::~PlayerChar()
{

}


void PlayerChar::SetPos(int x)
{
    box.x = x;
}


void PlayerChar::Draw()
{
    BlitSurface(box.x, box.y, assets.ss_playerchar_idle, ss_screen);
}


bool PlayerChar::DoFiring()
{
    if((b_canFire) && (b_shouldFire)) 
    {
        // Fire a bullet
        Fire(box.x + 22, box.y);
        b_canFire = false;
        b_shouldFire = false;

        return true; // fired a bullet
    }
    return false; // did not fire a bullet
}


void PlayerChar::Fire(int x, int y)
{
    // Create a new bullet at the correct location and add it to the global bullet list
    Bullet* bullet = new Bullet();
    bullet->SetPos(x, y);
    bullets.push_back(bullet);

    // Play bullet firing sound
    Mix_PlayChannel(-1, assets.mc_firebullet, 0);
}


void PlayerChar::HandleInput(Uint8* keystates)
{
    if(keystates[SDLK_LEFT] && keystates[SDLK_RIGHT])// Both direction keys
        moveDir = NONE;
    if(keystates[SDLK_LEFT] && !keystates[SDLK_RIGHT]) // Left key and not right key
        moveDir = LEFT; 
    if(!keystates[SDLK_LEFT] && keystates[SDLK_RIGHT]) // Right key and not left key
        moveDir = RIGHT; 
    if(!keystates[SDLK_LEFT] && !keystates[SDLK_RIGHT]) // Neither direction key
        moveDir = NONE;

    if(keystates[SDLK_SPACE]) // Space bar
        b_shouldFire = true;

    if(!keystates[SDLK_SPACE]) // Allow another bullet to be fired after release
        b_canFire = true;
}


void PlayerChar::Move()
{
    if(moveDir == LEFT && box.x > 0) // If not off screen, move
        box.x -= moveSpeed;
    if(moveDir == RIGHT && box.x < (ss_screen->w - box.w)) 
        box.x += moveSpeed;
}

和标题:

#ifndef __hiPlayerChar__
#define __hiPlayerChar__


#include "SDL.h"
#include "SDL_mixer.h"
#include "hiGlobalVars.h"


enum MoveDir
{
    LEFT,
    RIGHT,
    NONE
};


class PlayerChar
{
private:
    Sint16 moveSpeed;
    MoveDir moveDir;
    bool b_canFire;
    bool b_shouldFire;

public:
    // Ctr & Dtr
    PlayerChar();
    ~PlayerChar();

    // Functions
    void SetPos(int x);
    bool DoFiring();
    void Fire(int x, int y);
    void Move();
    void Draw();
    void HandleInput(Uint8* keystates);

    // Size and position
    SDL_Rect box;
};


#endif

Answer 1

您的函数中缺少最后一条返回指令:

bool PlayerChar::DoFiring()
{
    if(b_canFire && b_shouldFire)
    {
        Fire(box.x + 22, box.y); // This fires a bullet
        b_canFire = false; // Does not work
        b_shouldFire = false; // Does not work

        return true;
    }
    return false; // WAS MISSING HERE
}

然后，如果您的 b_canFire、b_shouldFire 中的任何一个为假，您的函数将返回任何 (UB)。