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

aGame 是一个在此类之外定义的数组，因此此代码不起作用。 由于此类中可能有许多其他函数可能会使用 aGame 数组，我应该这样做：

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 是一个在所有地方都可以使用的数组，因此必须在每个类中复制它似乎是不正确的。 我可能有这个错误，我正在慢慢学习 OOP，所以感谢您的帮助。

Answer 1

在我看来，第一个选项是正确的，因为它没有充分的理由使用全局变量。所以选择是在第二个和第三个之间。

决定性的功能是您是否要对多个aGame 值使用相同的Player 实例。如果只有一个值，那么我要么将其传递给构造函数（您的选项 2），要么使用 gnibbler 将其设为类变量的想法。我可能倾向于将它传递给构造函数以便于测试。

如果您希望同一个 Player 实例可用于多个 aGame 值，那么选项 3 可能是实现这一目标的最简洁方法。

Answer 2

每个实例的aGame 是否相同？ 然后你可以像这样把它变成一个类属性

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

或

Class Player:
    ...

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

在课堂内，您仍然可以通过self.aGame访问它

Answer 3

只有第一个选项有效。在第二个示例中，for room in self.aGame['rooms'] 将产生错误，因为没有任何地方 aGame 绑定到 self。如果它是for room in aGame['rooms']，它会起作用，但是你每次调用move() 时都必须不必要地传递aGame。

您也可以将其设为全局变量，但最好让每个 Player 持有一个 aGame 实例。如果您需要更改 aGame 并拥有多个玩家，则应将其定义为全局变量。

另外，只是吹毛求疵，但aGame 不是数组，而是字典。 Python 语言甚至没有数组（尽管一些扩展有）。

Answer 4

我会在全局上使用一些细微的变化：

# 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 成为一个适当的类，在初始化阶段将 Game.instance = Game(...) 分配给某个地方，并获得更多真正的单例模式。