C++函数重载问题：双参damage函数未正确修改玩家生命值

本人刚接触C++，有Java基础。现有一段计算玩家受伤害后生命值的代码，玩家处于脆弱状态时，spellFireDamage未能降低玩家生命值，但spellPoisonDamage可以。已知函数重载，但双参数的damage函数未正常工作，单参数版本却能运行。查到可能和传引用有关，但不太理解，求最简单清晰的修复方法及原因说明。

代码如下：

#include <iostream>

using namespace std;

int damage(int spellsDamage) {
    return(spellsDamage*3+5);
}

int damage(int health, int spellsDamage) {
    return (health - (spellsDamage*3+5));
}

int main(){
    int spellFireDamage = 3;
    int spellPoisonDamage = 2;
    int playerHealth = 100;
    bool playerVulnerable = true;
    

    //do something if vulnerable is true
    if (playerVulnerable) {
        damage(playerHealth, spellFireDamage);
        cout << "player health after fire attack" << endl;
        cout << playerHealth << endl;
        playerHealth = playerHealth - damage(spellPoisonDamage);
        cout << "player health after poison attack" << endl;
        cout << playerHealth << endl;
    }
    else {
        cout << "bruh" << endl;
    }
    

    return 0;
}

原因分析 #

• 值传递导致副本无效：C++默认采用值传递，调用damage(playerHealth, spellFireDamage)时，函数内的health是playerHealth的临时副本。函数计算出的扣除后生命值只是这个副本的结果，不会影响外部的playerHealth变量。而单参数版本是返回伤害值，你通过playerHealth = playerHealth - damage(spellPoisonDamage)主动把结果赋值给原变量，所以能生效。
• 和重载无关：你的函数重载逻辑是对的，两个damage函数参数数量不同，编译器能正确区分调用，问题不在重载本身。

两种简单修复方法 #

方法一：接收返回值赋值（无需修改函数定义） #

双参数damage已经算出了扣除伤害后的生命值，只需要将返回值赋值给playerHealth即可：

// 替换原调用语句
playerHealth = damage(playerHealth, spellFireDamage);

修改后运行，火攻击后playerHealth会正确更新为100 - (3*3+5) = 86。

方法二：使用引用传递（让函数直接修改原变量） #

如果希望函数直接修改外部的playerHealth，可以把函数第一个参数改为引用类型（在参数类型前加&），这样函数操作的就是原变量本身：

1. 修改函数定义：

// 改为void类型，因为不需要返回值，直接修改原变量
void damage(int& health, int spellsDamage) {
    health -= (spellsDamage*3+5);
}

2. 调用方式不变：

damage(playerHealth, spellFireDamage);

此时函数会直接修改playerHealth的值，无需手动赋值。

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