缺少不应缺少的参数（Python）？

Question

我目前正在编写一个简单的“战斗算法”（如果这是它的术语的话），它最终会变得越来越复杂，但现在最需要计算的是造成的伤害、当前的健康状况点数，还有躲闪的机会。这是代码本身：

#paradisium combat algorithm 1.0
import random
#ranged speed
player1_attack=5
#damage to enemy is attack-defense points
player1_defence=2
player1_speed=9
#higher speed higher chance to land a hit
player1_agility=9
#higher agility, higher chance of dodge
player1_HP=50
#two-handed defense
player2_attack=9
player2_defence=9
#blocking raises defense by doubling it
player2_speed=5
player2_agility=2
player2_HP=100
def player1attack(player1_speed,player1_attack,player2_agility,player2_defence,player2_HP,damage):
    dogeChance=player2_agility-player1_speed
    if random.randint(0,100)>dogeChance:
         damage=player1_attack-player2_defence
         player2_HP-=damage
         return (player2_HP, damage)
while player1_HP>0 or player2_HP>0:
    askMove=input("Attack(1) Brace(2)")
    if askMove=="1":
         damage=0
         player1attack(player1_speed,player1_attack,player2_agility,player2_defence,player2_HP,damage)
         damage,player2_HP=player1attack(damage,player2_HP)
         print("Player 2 recieved",damage,"damage!")
         print("Player 2's HP is at",player2_HP+"!")}

如您所见，我只是在开头列出了所有统计变量。该程序最终将能够获取两个人正在战斗的统计数据，并将其放入算法中。 该算法背后的当前数学是，只有在您未能躲避攻击时才会对人的生命值造成伤害，躲避机会（是的，我故意将其命名为 dogeChance）是通过从防御者的敏捷中减去攻击者的速度来计算的（在这种情况下玩家 2 的敏捷减去玩家 1 的速度，因为这是我们所得到的）。 伤害的计算方法类似，用防御减去攻击来计算，剩下的数字就是对敌人造成的伤害。 我遇到的问题是，虽然所有需要的变量都列在程序的开头，但我收到了这个错误：

Traceback (most recent call last):
  File "C:/Users/Thomas/Desktop/combatAlgorithmParadisium.py", line 30, in <module>
    damage,player2_HP=player1attack(damage,player2_HP)
TypeError: player1attack() missing 4 required positional arguments: 'player2_agility', 'player2_defence', 'player2_HP', and 'damage'

我从中得到的是该函数没有接受它需要的参数，所以我不知道我是否犯了新手错误或什么。如您所见，这种设计非常简单。我在一般编程方面已经有一段时间了，所以如果我不能理解你的解释，我提前道歉。我还更改了变量名称，使其更易于阅读和识别。

感谢生锈的 Manibus

Answer 1

首先，我建议使用对象类来使这段代码更具可读性，并且最终更易于使用和维护。

这是我尝试将您的代码翻译成“更经典”的东西：

#!python
## !/usr/bin/env python3
import random

class Player (object):
    def __init__(self, attack, defence, speed, agility, hitpoints):
        self.attack    = attack
        self.defence   = defence
        self.speed     = speed
        self.agility   = agility
        self.hitpoints = hitpoints

    def is_alive(self):
        return self.hitpoints > 0

    def Attack(self, target):
        dodgeChance = target.agility - self.speed
        damage = 0
        roll = random.randint(0,100)
        if roll > dodgeChance:
             damage = abs(self.attack - target.defence)
        target.hitpoints -= damage
        return (target.hitpoints, damage, dodgeChance, roll)


if __name__ == '__main__':
    player1 = Player(5, 2, 9, 9, 50)
    player2 = Player(9, 9, 5, 2, 100)

    while player1.is_alive() or player2.is_alive():
        damage = dodge = roll = 0
        askMove = input("Attack(1) Brace(2)")
        if askMove.startswith("1"):
             damage, x, dodge, roll = player1.Attack(player2)
        player2.Attack(player1)
        print("Player 2 recieved", damage, "damage!")
        print("Player 2's HP is at %s!" % player2.hitpoints)
        print("\t\tDodge: %s\tRoll: %s!" % (dodge, roll))
        print("\tPlayer 1 HP: %s\n" % (player1.hitpoints))

顺便说一句，在旧版本的 Python 中，input() 返回基于一行输入（来自标准输入）的 Python 求值 的结果。因此输入 1[Enter] 会产生整数 1 而不是字符串：“1” --- 在您的代码中，askMove == "1" 条件将永远不会满足（在 Python 2.x 下） . （注意，我粘贴的代码中重复的 #! 行是为了区分我正在运行的代码副本和使 SO 的语法突出显示高兴的 shebang 行。

此外，我对您代码的初始翻译似​​乎永远运行（几乎？所有情况）。我们必须取伤害结果的绝对值，以确保在此模拟中生命值始终在下降。只要任何一个玩家还活着就保持while循环的选择很奇怪......因为死去的玩家继续战斗而活着的玩家（应该获胜）继续与僵尸战斗，直到他也死了。

（不完全是我推荐的游戏机制）。

另外请原谅使用.attack 作为字段名和.Attack() 作为方法。我想在我的代码翻译中保持尽可能接近你的术语。在我自己的代码中，我可能会将字段/属性命名为“.attack_bonus”，甚至只是“.bonus”，具体取决于我打算在未来支持多少其他属性。

Answer 2

 player1attack(player1_speed,player1_attack,player2_agility,player2_defence,player2_HP,damage)
 damage,player2_HP=player1attack(damage,player2_HP)

player1attack 有六个参数。在上面的代码中，您在第一行使用了正确数量的参数。在第二行，您没有传递足够的参数。您传递了所需六个中的两个，因此出现missing 4 错误。无论如何，我不确定您为什么要调用该函数两次。我说，只需使用任何一个中的最佳质量（第一行中的“正确数量的参数”，第二行中的“将结果分配给某些变量”）合并这些行。

damage, player2_HP = player1attack(player1_speed, player1_attack, player2_agility, player2_defence, player2_HP, damage)

Answer 3

凯文所说的。此外，damage 和 player2_HP 在行上被分配了错误的值

damage,player2_HP=player1attack(damage,player2_HP)

因为您的函数设计为首先返回 player2_HP，然后返回 damage。

最后，在当前状态下，您的算法使玩家 2 在每次命中时获得更多 HP，因为您减去了一个负整数，因此得到了一个正数。我建议进行以下编辑以避免这种情况：

damage=player1_attack-player2_defence if player1_attack-player2_defence > 0 else 0

但是，这仍然不能令人满意，因为这意味着玩家 2 将永远不会受到任何伤害，因为他们的防守太强了。也许您不应该将整个防御点减去攻击点，而应该只减去一部分。

（编辑：我也刚刚注意到您的算法使用了否定的 dogeChance）

（我想将所有这些作为附带评论发布，但我还没有评论权限）