Где я испортил эту программу для отслеживания альянсов фракций?

Question

У меня есть программа, которая моделирует королевства и другие группы (так называемые «фракции» в моем коде).

class Faction: 
    def __init__(self, name, allies=[]): 
     self.name = name 
     self.allies = allies 

    def is_ally_of(self, other_faction): 
     if self in other_faction.allies: 
      return True 
     else: 
      return False 

    def become_ally(self, other_faction, both_ally=True): 
     """ If both_ally is false, this does *not* also 
      add self to other_faction's ally list """ 
     if self.is_ally_of(other_faction): 
      print("They're already allies!") 
     else: 
      self.allies.append(other_faction) 
      if both_ally == True: 
       other_faction.become_ally(self, False) 

RezlaGovt = Faction("Kingdom of Rezla") 
AzosGovt = Faction("Azos Ascendancy") 

Я хочу быть в состоянии назвать фракции become_ally() для добавления фракций в списки союзников, как это:

RezlaGovt.become_ally(AzosGovt) # Now AzosGovt should be in RezlaGovt.allies, 
           # and RezlaGovt in AzosGovt.allies 

Что на самом деле происходит:

RezlaGovt.become_ally(AzosGovt) 
# prints "They're already allies!" 
# now AzosGovt is in the allies list of both AzosGovt and RezlaGovt, 
# but RezlaGovt isn't in any allies list at all. 

Всякий раз, когда я пытаюсь вызвать стареет(), код должен проверить, чтобы убедиться, что они еще не являются союзниками. Это часть, которая не работает. Каждый раз, когда я вызываю start_ally(), он печатает «Они уже союзники!», Независимо от того, являются ли они на самом деле.

Я также пытался использовать if self in other_faction.allies:, но у этой проблемы была такая же проблема.

Я сильно подозреваю, что проблема связана с моим использованием self, но я не знаю, какие условия для Google для получения дополнительной информации.

Answer 1

You can't use mutable arguments as the default argument to a function.

def __init__(self, name, allies=[]): 

Когда используется по умолчанию, это жеlist каждый раз, так что они имеют один и тот же allies; мутирующий один меняет другой, потому что они на самом деле одно и то же.

Изменить на:

def __init__(self, name, allies=None): 
    if allies is None: 
     allies = [] 

В качестве альтернативы, скопируйте allies аргумент безоговорочно (так что вы не обеспокоены ссылкой на его выживание вне класса и получать мутировали под класс):

def __init__(self, name, allies=[]): 
    self.allies = list(allies) # Which also guarantees a tuple argument becomes list 
           # and non-iterable args are rejected 

Answer 2

Измените эту функцию.

def is_ally_of(self, other_faction): 
     if other_faction in self.allies: 
      return True 
     else: 
      return False 

Проверьте свои данные, а не данные переданного объекта.

Также

def __init__(self, name, allies=[]): 

Есть ошибка ждущий, чтобы случиться. Ваш список allies будет статическим списком, общим для всех экземпляров. Вместо того, чтобы использовать

def __init__(self, name, allies=None): 
     self.name = name 
     self.allies = allies or []