C++回合制计分对比时total1值异常导致胜负判断错误求助

问题现象 #

开发C++回合制计分程序时，total1、total2两个分值的比较逻辑运行不符合预期：即使玩家1得分更高，程序也会错误判定玩家2获胜。通过cout打印变量取值排查发现，total1的存储值被损坏，实际取值与total2一致，最终导致比较结果错误。初步怀疑问题出在玩家2回合计算逻辑执行时，total1的存储值未被正确保留。

变量与规则说明 #

• 变量i代表当前回合序号：奇数取值对应玩家1回合，偶数取值对应玩家2回合
• 变量total1：存储玩家1的累计得分
• 变量total2：存储玩家2的累计得分

现有核心实现代码 #

if (i == 1 || i == 3 || i == 5 || i == 7 || i == 9)
{
    cout << endl << "Your mark for this round is " << total1;
}

if (i == 2 || i == 4 || i == 6 || i == 8 || i == 10)
{
    cout << endl << "Your mark for this round is " << total2;
}

if (i == 2 || i == 4 || i == 6 || i == 8 || i == 10)
{
    if (total1 > total2)
    {
        cout << "player 1 wins";
        cout << endl << "Total 1: " << total1 << " while total 2: " << total2;
    }
    else
    {
        cout << endl << "player 2 wins";
        cout << endl << "Total 1: " << total1 << " while total 2: " << total2;
    }
}

问题根因分析 #

你贴出的这段打印、比较逻辑本身只做值读取，不会修改total1和total2的取值，total1值被覆盖的问题必然出现在未贴出的得分累计计算段，常见触发场景如下：

• 变量引用错误：玩家2回合计算单轮得分时，误将得分累加/赋值到total1上，直接覆盖了total1原有的累计值，这也是出现total1取值和total2一致现象的最常见原因
• 作用域错误：若total1定义在玩家1回合的判断代码块内部，属于块级局部变量，块外代码访问的total1实际是同作用域下其他未正确初始化的变量，内存空间会被后续操作随意覆盖
• 内存越界：如果两个变量为相邻定义的数组元素、或通过指针操作得分时指针偏移计算错误，写入total2的操作会越界篡改total1所在的内存空间。

另外你贴出的代码本身存在逻辑缺陷，和值损坏问题无关，但会导致胜负判定不符合预期：
当前代码在每一个偶数回合（i=2/4/6/8/10）都会触发一次胜负判断，而非所有回合结束后再判定最终结果，会出现赛程未完成就提前输出胜负的问题。

修复建议 #

1. 定位值覆盖位置：在每一处对total1、total2做赋值、累加操作的代码前后加打印，定位到哪一步执行后total1的值被异常修改，重点检查玩家2回合的得分计算逻辑，确认得分累加的目标变量为total2
2. 调整胜负判断触发时机：将胜负比较逻辑移到全部10个回合执行完成后再运行，不要在每一个偶数回合都触发判定
3. 简化奇偶判断逻辑：无需硬枚举1-10的回合数值，直接用取模运算判断奇偶即可，减少硬编码出错概率：

// 玩家1奇数回合逻辑
if (i % 2 == 1) {
    cout << endl << "Your mark for this round is " << total1;
} 
// 玩家2偶数回合逻辑
else {
    cout << endl << "Your mark for this round is " << total2;
}

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