Python:类 "init"中的 If-else；这有用吗？

Question

我是另一个编程新手，正在经历“艰难地学习 Python”，遇到了一些我不完全理解的事情。

在开始之前，我会说虽然我真的很喜欢他的类(class)计划，但一个不幸的副作用似乎是我不了解编程世界中涉及的很多技术话题。我花了两天时间才弄清楚什么是“实例化”，即使是现在我也只是觉得我明白了，呵呵。因此，希望我的查询的答案还没有以更具技术性的描述方式被问到，如果有的话，我很抱歉是多余的。

无论如何，我在这里要做的是完成一个文本冒险，使用类来描述每个房间。我实际上正在做的是将我的旧作品导入这种更新的、更“OOP”的做事方式——转换一个基本上有 980 行函数调用的程序。

所以之前我在每个功能中都有这些“房间”，它们会检查你是否达到了特定的游戏目标来向你描述房间。示例:

def room_one():
    if "flashlight" in pack:
        print "You see an exit to the NORTH and a bear to the EAST."
    elif "flashlight" not in pack:
        print "It's too dark to see."

无聊的房间，但恰当的描述。所以在尝试使用类来做到这一点时，我似乎碰壁了。我将向您展示我用来定义游戏“运行者”和房间类别的代码。

class Runner(object):
    def __init___(self):
        self.goals = [] #an array for game achievements (talked_to_king, etc)
        self.pack = [] #an array for game items (flask, key, etc)

    def play(self, currentroom, gamestate):

        while True:

            print "\n--------"
            print currentroom.desc #this line prints the room description, could
            cmd = raw_input("> ")  #it also be part of the problem?
            gamestate.state = currentroom.actions(cmd)

            if gamestate.state != "SAME":
                currentroom = gamestate.state

            else:
                pass

class Room(object):
    def __init__(self):
        self.desc = "null" #short room description
        self.lookdesc = "super null" #longer room description for the LOOK command


    def actions(self, cmd):
        if 'help' in cmd:
            print """
            'look' -- see your surroundings, including exits.
            'get' -- to pick up an item (if it can be picked up).
            'inventory' -- displays your collected items
            'throw' -- some objects can be thrown
            'talk' -- speak to someone
            Other commands, or objects to interact with, will tend to
            show up IN UPPERCASE. (but still type them in lowercase!)
            Cardinal directions (north, south, east, west) for movement.
            """
            return "SAME"
        elif 'inv' in cmd:
            print "--Inventory--\n"
            for items in game.pack:
                print items
            print "-------------\n"
            return "SAME"
        elif 'look' in cmd:
            print self.lookdesc
            return "SAME"
        elif 'goals' in cmd:
            for goal in game.goals:
                print goal
            return "SAME"
        else:
            print "That won't work here."
            return "SAME"


class Office(Room):

    def __init__(self):

        super(Office, self).__init__()
        # here's the part that doesn't work
        if 'principal' in game.goals:
            self.desc = "You're in the principal's office, and there's a zombie in here!"
        else:
            self.desc = "You're in the principal's office."

        self.lookdesc = "The walls in here are dingy from decades of smoking.\nA door to the WEST leads back to the foyer."



    def actions(self, cmd):

        if "west" in cmd and not "principal" in game.goals:

            print "Buck up and talk to the old prune."

            return "SAME"

        elif "talk" in cmd:

            print "You call to the principal."
            game.goals.append('principal')
            next = raw_input("--Mr. Friiiiinkseseeees...--")
            print "But OH MY DAMN he's actually a zombie now."
            next = raw_input("--Aww weak--")
            return "SAME"

        return super(Office, self).actions(cmd)

为了检查以确保附加了“目标”数组，我确实在 Room 父类中放置了一个简单的目标检查函数，并且确实附加了 game.goals。但是我的 if-else 语句似乎实际上并没有做任何事情；无论 goals 数组中有什么，当我回到房间时，我总是得到 else 的 desc。

公平地说，将 if-else 放在房间 init 中就像是在黑暗中开枪，事实上，我很惊讶我没有收到一条错误消息，告诉我那不是那些地方!但它似乎并不关心 game.goals 数组的内容这一事实告诉我，这不是执行此操作的最佳方法。我应该如何实现？

作为一个子问题 - 对于这样的程序，如果游戏状态变化足以使房间的内容发生显着改变，是否认为改变房间的整个其他类更好？更重要的是维护“ map ”并说 RoomThree(Room) 总是 RoomThree 无论发生什么游戏状态变化，或者当游戏回到 RoomThree 时这个地方已经被毁坏、抢劫、着火，充满了吸血鬼和生菜的味道，那应该是完全不同的 RoomThree 吗？我有点希望不要，因为让每个房间“成为那个房间”并让 game.goals 和 game.pack 之类的内容更改类的实例似乎更合理只是在满足某些条件时创建一个新类。

总之，这是冗长的。 tl;dr - 我这样做很奇怪吗？？

再次编辑:按照这里的人的建议清理主要问题的代码以满足新代码，还添加了父类“Room”以查看这是否不是问题的一部分。仍然没有解决方案，但我相信代码的质量已经提高了:) Whee

最终编辑(我希望):啊哈，斯通的回答再次很有帮助。由于我将“相同”返回给我的运行者，因此它使用的是旧的 Office 实例。我需要创建一个全新的实例来让它识别变化；通过返回“Office()”而不是“SAME”，我完成了这个。感谢大家不仅帮助解决了手头的问题，还让代码更易于阅读，并帮助我改变了对冗长的 if-else 语句的想法。哇:D

Answer 1

当您有 100 个房间时会发生什么？你的 play 函数会有 100 个 if 语句，只是为了设置你所在的房间吗？ (再设置一遍，基本上)

当游戏状态发生变化时，直接将游戏状态设置为您想要的任何状态可能会好得多。

编辑:我刚刚重新阅读了您的内容，现在问题似乎很明显，除非您遗漏了重要代码。

您唯一一次将委托(delegate)人添加到目标是在类 Office 的成员函数中。因此，当您着手将主体添加到目标时，为时已晚，Office 已经初始化并且不会重新初始化，除非您创建一个新的 Office 实例。添加该目标后，您需要更新房间的描述。