Czy nieskończona pętla to zła praktyka?

Question

Wdrażam grę karcianą w Pythonie, a dla mojej klasy do obsługi graczy, PlayerHandler, niedawno zaimplementowałem __next__, aby po prostu zadzwonić pod numer next_player. Ponieważ rozgrywkę można myśleć w nieskończonej pętli (gracze będą grać dalej, dopóki nie zakończą lub nie wygrają/przegrają), nie ma sensu, aby przerwać iterację. Jednak może wydawać się mylące, jeśli pętla for spowodowała nieskończoną pętlę, więc czy powinienem gdzieś podnieść 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 

Answer 1

Ważne jest, aby twój kod był czytelny. Jeśli się martwisz, dodaj komentarz - właśnie po to są!

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

Powiedziawszy to, może powinieneś przestać, kiedy nie ma już graczy.