Python: лучший способ вызова методов из другого класса?

Question

У меня есть следующий код:

class Player: 
    def __init__(self, username, trip, model): 
     self.username = username 
     self.trip = trip 
     self.hp = 100 


    #### For player moving location/room #### 
    def Move(self, dest): 
     if dest == self.loc: 
      return True 

     # Check destination room is accessible from current room 
     for room in aGame['rooms']: 
      if room['ref'] == self.loc: 
       for acsroom in room['acs']: 
        if acsroom == dest: 
         self.loc = dest 
         return True 
     return False 

агама является массивом, который определен вне этого класса так что этот код не работает. Поскольку, вероятно, будет много других функций этого класса, которые, возможно, используют агаме массив, я должен сделать это:

class Player: 
    def __init__(self, username, trip, model, aGame): 
     self.username = username 
     self.trip = trip 
     self.hp = 100 
     self.aGame = aGame    

    #### For player moving location/room #### 
    def Move(self, dest): 
     if dest == self.loc: 
      return True 

     # Check destination room is accessible from current room 
     for room in self.aGame['rooms']: 
      if room['ref'] == self.loc: 
       for acsroom in room['acs']: 
        if acsroom == dest: 
         self.loc = dest 
         return True 
     return False 

Или было бы лучше сделать это:

class Player: 
    def __init__(self, username, trip, model): 
     self.username = username 
     self.trip = trip 
     self.hp = 100   

    #### For player moving location/room #### 
    def Move(self, dest, aGame): 
     if dest == self.loc: 
      return True 

     # Check destination room is accessible from current room 
     for room in aGame['rooms']: 
      if room['ref'] == self.loc: 
       for acsroom in room['acs']: 
        if acsroom == dest: 
         self.loc = dest 
         return True 
     return False 

Или должен ли я сделать aGame глобальной переменной (если да, то как, обратите внимание, что этот класс находится в другом файле)?

Поскольку aGame - это массив, который используется повсеместно, не представляется правильным делать копии его внутри каждого класса. Возможно, я ошибаюсь, я медленно изучаю ООП, поэтому спасибо за любую помощь.

Answer 1

На мой взгляд, первый вариант сразу же, поскольку он использует глобальные переменные без уважительной причины. Таким образом, выбор между вторым и третьим.

Решающей особенностью является то, что вы захотите использовать один и тот же экземпляр Player для более чем одного значения aGame. Если будет только одно значение, то я либо передал бы его конструктору (ваш вариант 2), либо использовал идею gnibbler о том, чтобы сделать его переменной класса. Я бы предпочел передать его конструктору для удобства тестирования.

Если вы хотите, чтобы один и тот же экземпляр Player использовался с несколькими значениями aGame, тогда вариант 3, вероятно, является самым чистым способом достижения этого.

Answer 2

Только первый выбор будет работать. Во втором примере for room in self.aGame['rooms'] приведет к ошибке, потому что нигде нет привязки к себе. Он будет работать, если он был for room in aGame['rooms'], но тогда вам придется излишне передавать aGame каждый раз, когда вы вызываете move().

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

Кроме того, просто nitpicking, но aGame не является массивом, это словарь. Язык Python даже не имеет массивов (хотя некоторые расширения имеют).

Answer 3

aGame одинаково для каждого экземпляра? Тогда вы можете сделать это атрибут класса либо как этот

class Player: 
    aGame={'rooms':...} 
    ... 

или

Class Player: 
    ... 

Player.aGame={'rooms':...} 

В классе, вы можете получить доступ к нему через self.aGame

Answer 4

Я хотел бы использовать небольшое изменение на a в глобальном масштабе:

# in game.py or whatever 
class Game(object): 
    instance = {} # this is your aGame array 

# in player.py or whatever: 
from game import Game 

class Player(object): 
    # ... 
    def Move(self, dest): 
     # ... 
     for room in Game.instance['rooms']: 
      # ... 

В качестве альтернативы, вы можете сделать игру надлежащей класса, назначьте Game.instance = Game (...) где-то во время фазы инициализации и получите больше реального синглтон-паттерна.