Question

我目前正在尝试使用纯python编写基于文本的冒险。因此，我有一个Room类，看起来有点像这样（缩短）：

class Room1(Room):

    def __init__(self):

        Room.__init__(self)
        self.init_objects()
        self.description = """Looks like a living room. There's a lamp, a 
                              cupboard, and
                              a door to your right. On the floor, there's a carpet."""
        self.door_state = 1
        self.carpet_state = 0
        self.images = [(text-based images)]

    def init_objects(self):

        self.objects = {"door" : Door1(),
                        "frontdoor" : FrontDoor(),
                        "carpet" : Carpet(),
                        "lamp" : Lamp1(),
                        "clock" : Clock(),
                        "escritoire" : Escritoire(),
                        "booklet" : Booklet(),
                        "screws" : Screws(),
                        "trapdoor" : TrapDoor()}

    def update_image(self):

        IMG_MAPPER = {(0, 0, 0) : 0,
                      (0, 0, 1) : 1,
                      (1, 0, 0) : 2,
                      (1, 0, 1) : 3,
                      (1, 1, 1) : 4,
                      (1, 1, 0) : 5,
                      (0, 1, 1) : 6,
                      (0, 1, 0) : 7}
        key = (self.objects["door"].state, self.objects["carpet"].state)
        self.img = img_mapper[key]

我的问题在于Room的update_image（）方法。我需要一个映射器存储在那里，根据对象的状态（打开/关闭）找出正确的图像，如果我把这个映射器放在方法的开头，这个dict是由python读取和构造的每次调用该方法时，是吗？那么我是否应该将此映射器dict存储为实例变量，如self.img_mapper_dict = {（0,0,0）：0，...}？

关于那个人的任何想法也许？

Answer 1

我认为您遇到的问题是img_mapper会在每次调用 update_image()时重置为指定的状态。这是因为成员函数有自己的作用域，因此当它完成时会删除img_mapper，并且必须在下次调用更新函数时重新创建它。存储为成员变量可以解决这个问题。

Answer 2

您认为当前代码的结构方式是正确的，IMAGE_MAPPER方法每次调用update_image()都会被分配。

现在，由于这是一个不随时间变化的静态映射，从功能的角度来看这不是问题 - 它不像你重置一个应该在调用update_image()之间跟踪的值

在性能方面，在这种特殊情况下，缺点也绝对可以忽略不计。

但从纯逻辑角度来看，将此映射设为class attribute可能是有道理的：

class Room1(Room):

    IMG_MAPPING = {(0, 0, 0) : 0,
                   (0, 0, 1) : 1,
                   (1, 0, 0) : 2,
                   (1, 0, 1) : 3,
                   (1, 1, 1) : 4,
                   (1, 1, 0) : 5,
                   (0, 1, 1) : 6,
                   (0, 1, 0) : 7}

    def __init__(self):
        # ...

    def update_image(self):
        # ...
        self.img = Room1.IMG_MAPPING[key]

我建议使用class属性而不是实例属性（或实例成员），因为对于Room1的所有实例，该映射将保持不变（对吧？我假设{{1}的每个实例1}}将具有完全相同的布局，只是不同的门状态等。）。所以它不依赖于实例的任何状态，而是它的类的属性。

另请注意，class属性的访问方式不同：Room1而不是Room1.IMG_MAPPING。这也反映了它不依赖于实例的事实。

Answer 3

我看到的第一个问题是你的IMG_MAPPER dict有三元组密钥。但是你通过提供两元组来访问它。

关于python对象及其方法

3 个答案: