是一个无限循环的坏练习?

时间:2014-04-17 00:28:41

标签: python loops iterator

我正在使用Python实现纸牌游戏,而我的班级要处理玩家PlayerHandler,我最近实施了__next__来简单地调用next_player。因为游戏玩法可以被认为是无限循环(玩家将继续玩,直到他们退出或赢/输),它停止迭代是没有意义的。但是,如果for循环导致无限循环,那么它可能会让人感到困惑,所以我应该在某处提升StopIteration吗?

class PlayerHandler():
    """Holds and handles players and order"""
    def __init__(self, players=None, order=1):
        if players is None:
            self.players = []
        else:
            self.players = list(players)
        self.current_player = None
        self.next_player()
        self.order = order

    def get_player(self, interval):
        assert type(interval) == int, "Key must be an integer!"
        if not interval and self.players:
            return self.current_player
        elif self.players:
            step = interval * self.order
            index = self.players.index(self.current_player) + step
            while index >= len(self.players):
                index -= len(self.players)
            while index <= -(len(self.players)):
                index += len(self.players)
            return self.players[index]
        else:
            raise KeyError("No players in the list!")

    def next_player(self):
        """Sets the current player to the next player to play"""
        if not self.current_player:
            if self.players:
                self.current_player = self.players[0]
            else:
                self.current_player = None
        else:
            self.current_player = self.get_player(1)

    def __iter__(self):
        return self

    def __next__(self):
        self.next_player()
        return self.current_player

1 个答案:

答案 0 :(得分:4)

重要的是您的代码是可读的。如果您担心,请添加评论 - 这就是他们的目标!

# Endless loop
for p in players:
    # Do game things...

话虽如此,也许你应该在没有更多玩家时停止迭代。