缺少不应该遗漏的参数(Python)?

时间:2014-10-03 19:47:35

标签: python algorithm

我目前正在编写一个简单的“战斗算法”(如果这就是它的术语),它最终会被构建得越来越复杂,但是现在它需要计算的最多就是损坏,目前的健康状况点和躲闪的机会。这是代码本身:

#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+"!")}

如您所见,我只是在开头列出了所有的统计变量。该程序最终将能够接收两个人正在战斗的统计数据,并将其纳入算法。 这个算法背后的当前数学是,只有当你未能躲避攻击时才会对人的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'

我从中获得的是,该功能没有接受它需要的参数,所以我不知道我是在犯一个菜鸟错误还是什么。如您所见,这种设计非常简单。我已经在初级编程一般了一段时间,所以如果我无法理解你的解释,我会提前道歉。我还更改了变量名称,以便更容易阅读和识别。

谢谢,Rusty Manibus

3 个答案:

答案 0 :(得分:1)

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

player1attack需要六个参数。在上面的代码中,您在第一行使用正确数量的参数。在第二行,您没有传递足够的参数。您正在传递所需的六个中的两个,因此missing 4错误。无论如何,我不确定你为什么要两次调用该函数。我说,只使用其中任何一个的最佳品质(&#34;正确数量的参数&#34;来自第一行,&#34;将结果分配给一些变量&#34;来自秒)。

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

答案 1 :(得分:1)

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

这是我尝试将您的代码转换为“classier:”

的内容
#!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”,具体取决于我打算在未来支持多少其他属性。

答案 2 :(得分:0)

凯文说。另外,damageplayer2_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)

(我希望将所有这些作为旁注发布,但我还没有评论特权)