循环内整数钳位引发程序冻结问题求助

我有一个存储升级所需XP值的数组xpData[]，作为Player类的成员变量，在AddXP()函数中使用。AddXP()里给xp加值的逻辑正常，达到maxExp时也能正确钳位，但用while循环判断xp >= xpData[lvl]来升级时出问题了——当XP超过30级所需值时，程序直接冻结，无报错无崩溃。我需要在保留多等级升级、等级钳位功能的前提下解决这个问题。

相关代码 #

GeneralFunctions.cpp #

int ClampInt(int n, int lower, int upper)
{
    int updatedNum = n;

    if (n <= lower)
        updatedNum = lower;
    else if (n >= upper)
        updatedNum = upper;

    return updatedNum;
}

Player.h #

class Player
{
public:
    Player();

    void AddXP(int amount);

    int GetLevel() const { return lvl; };
    int GetXP() const { return xp; };

private:
    int minLevel;
    int maxLevel;
    int lvl;
    int maxExp;
    int xp;
    int xpData[31] = {
        0,   // zero-based
        10,  // Level 1
        20,  // Level 2
        30,  // Level 3
        40,  // Level 4
        50,  // Level 5
        60,  // Level 6
        70,  // Level 7
        80,  // Level 8
        90,  // Level 9
        100, // Level 10
        110, // Level 11
        120, // Level 12
        130, // Level 13
        140, // Level 14
        150, // Level 15
        160, // Level 16
        170, // Level 17
        180, // Level 18
        190, // Level 19
        200, // Level 20
        210, // Level 21
        220, // Level 22
        230, // Level 23
        240, // Level 24
        250, // Level 25
        260, // Level 26
        270, // Level 27
        280, // Level 28
        290, // Level 29
        300  // Level 30
    };
};

Player.cpp #

Player::Player()
{
    minLevel = 1;
    maxLevel = 30;
    lvl = 1;
    maxExp = 1000000;
    xp = 0;
}

void Player::AddXP(int amount)
{
    if (xp + amount > maxExp)
        xp += (maxExp - xp);
    else
        xp += amount;

    while (xp >= xpData[lvl])
    {
        lvl = ClampInt(lvl + 1, minLevel, maxLevel);
    }
}

问题分析 #

程序冻结的核心原因是无限循环：当等级达到maxLevel（30级）后，ClampInt(lvl + 1, 1, 30)会返回30，lvl不再增长。此时如果xp >= xpData[30]（即300），循环条件xp >= xpData[lvl]永远为真，循环会无终止执行，导致程序卡死。

另外还有隐藏问题：当前逻辑仅提升等级，未处理升级后的剩余XP（比如一次性获得大量XP跳过多个等级时，循环终止条件未考虑等级上限的边界情况）。

解决方案 #

修改AddXP()中的循环逻辑，在等级达到上限后直接终止循环，同时确保每次迭代都有明确的终止条件：

void Player::AddXP(int amount)
{
    // 简化XP钳位逻辑，结果与原代码一致
    if (xp + amount > maxExp)
        xp = maxExp;
    else
        xp += amount;

    // 升级循环：仅在未到满级且XP满足升级条件时执行
    while (lvl < maxLevel && xp >= xpData[lvl])
    {
        lvl++;
        // 可选：若升级需消耗对应等级XP，可添加以下代码
        // xp -= xpData[lvl - 1];
    }

    // 冗余但安全的等级钳位，避免意外越界
    lvl = ClampInt(lvl, minLevel, maxLevel);
}

关键修改点： #

1. 循环条件新增lvl < maxLevel，等级达到30级时直接终止循环，彻底避免无限循环。
2. 简化XP钳位写法，让逻辑更清晰（xp = maxExp和原代码效果完全一致）。
3. 升级时直接lvl++，最后统一做等级钳位（循环条件已限制等级不超过上限，此步骤为冗余安全保障）。
4. 若你的设计中升级需要扣除对应等级的XP，可在循环内添加xp -= xpData[lvl - 1];，根据需求调整。

修改后既保留了多等级连续升级的功能，又解决了等级上限后的循环卡死问题。

内容的提问来源于stack exchange，提问作者William Smith