主页 / 问题 / 1436137
Accepted
ムkula
Asked:2022-08-04 19:23:21 +0000 UTC2022-08-04 19:23:21 +0000 UTC

启动程序后出现错误

  • 772

主文件

from variants import Variants
from player import Player
bot = Player
alex = Player(Variants.SCISSORS, 'Alex')
print(bot.whoWins(bot, alex))

播放器.py

from variants import Variants


class Player:

    def __init__(self, choise=Variants.ROCK, name='bot'):
        self.choise = choise
        self.name = name

    def whoWins(self, player1, player2):
        if (player1 == player2):
            print("draw")
        if player1 > player2:
            print('player1 wins')
        if player1 < player2:
            print('player2 wins')

变体.py

from enum import Enum


class Variants(Enum):

    ROCK = 1,
    PAPER = 2,
    SCISSORS = 3

错误：

Ошибка: Traceback (most recent call last):
  File "D:\shlak\Zadanie k 2 modulu\main.py", line 8, in <module>
    print(bot.whoWins(bot, alex))
TypeError: Player.whoWins() missing 1 required positional argument: 'player2'
python python-3.x

1 个回答

  1. Best Answer

    代码中有很多问题。

    主要的一点是类可以使用等于/大于/小于进行比较，它们必须具有魔术方法__eq__, __gt__, （理想情况下，您仍然需要为,__lt__创建方法）。<=>=

    我将添加到Variants方法__eq__中，__lt__其余部分是使用装饰器创建的total_ordering。

    在方法中，whoWins我比较的不是类Player，而是他们的选择（字段choice）。

    total_ordering请参阅装饰器的文档

    您可以在 Habré: A Guide to Magic Methods in Python的文章中阅读有关魔术方法的一般信息

    from enum import Enum
from functools import total_ordering


# Добавляем два метода для сравнения - на раверство и на "меньше",
# остальное создатся с помощью декоратора total_ordering

@total_ordering
class Variants(Enum):
    ROCK = 1  # Не должно быть запятой (из-за нее записывается tuple из одного элемента (1,) вместо просто 1)
    PAPER = 2  # Не должно быть запятой
    SCISSORS = 3

    def __eq__(self, other):
        return self.value == other.value
    
    def __lt__(self, other):
        # Если разница числовых значения больше 1, значит это ROCK и SCISSORS
        if abs(self.value - other.value) > 1:  
            # Два "крайних" случая (ROCK и SCISSORS) нужно сравнивать наоборот,
            # Т.е. ROCK должно стать > SCISSORS, хотя 1 < 3
            return self.value > other.value
        else:
            return self.value < other.value


# Проверяем, что операторы сравнения работает правильно
assert Variants.ROCK < Variants.PAPER
assert Variants.PAPER > Variants.ROCK
assert Variants.SCISSORS < Variants.ROCK
assert Variants.ROCK > Variants.SCISSORS
assert Variants.PAPER < Variants.SCISSORS
assert Variants.SCISSORS > Variants.PAPER


class Player:
    def __init__(self, choice=Variants.ROCK, name='bot'):
        self.choice = choice
        self.name = name

    # Добавляем декоратор staticmethod, чтобы можно было вызывать метод от класса, и не указывать аргумент self
    @staticmethod  
    def whoWins(player1, player2):
        # Сравниваем не игроков, а их выбор
        if player1.choice == player2.choice:
            # Если нужно вывести результат функции через print, то результат нужно из нее возвращать через return
            return "draw"
        elif player1.choice > player2.choice:
            return f"{player1.name} wins"  # Возвращаем итог с именем игрока
        else:
            return f"{player2.name} wins"


bot = Player()  # Не было скобок, из-за этого не создавался объект, а просто записывался класс Player
alex = Player(Variants.SCISSORS, 'Alex')
print(Player.whoWins(bot, alex))

    结论：bot wins

    • 1

相关问题