对象如何在不违反依赖性倒置原则的情况下进行通信?

时间:2015-04-07 13:59:49

标签: python oop dependency-inversion

我正在构建一个路径规划器,它将帮助人们计划通过RPG控制台游戏的路径。

我想创建一个表格,显示整个舞台的每一步。我实际上implemented a working version of this,但它看起来很糟糕的OOP设计;它违反了各种原则,我相信它甚至不是合法的OOP。 问题显然是Table是上帝阶级。

由于这个原因,我决定重写它,同时努力记住正确的OOP原则。我想将Table分解为多个类。

我的问题是我需要各种对象来互相交流。但是,我的解决方案是始终使用组合。这打破了依赖原则以及单一责任原则。

以下是存储玩家步骤的主表:

class PathTable(object):
    ''' A path table. '''

    def __init__(self):
    # table is a list of dicts, representing rows
        self._table = []

    @property
    def table(self):
        return self._table

    def addStep(self, step):
        ''' Adds a step to the table. '''
        self._table.append(step)

    def rmStep(self):
        ''' Removes the last step from the table. '''
        try:
            del self._table[-1]
        except:
            raise IndexError('Tried to remove item from an empty table.')

现在,我创建了一个负责接受和验证用户输入的InputManager

class InputManager(object):
    ''' Responsible for managing user input. '''
    def __init__(self):
        pass
    def addFight(self, position):
        ''' Add a 'fight' at table[position]. '''
        # table._table[position]['input'] = 'fight'
        # Need to somehow edit a particular row in the Table.

但是,现在我不知道如何访问PathTable._table[position]。不破坏各种OO设计原则。

这令人沮丧,因为InputManager的整个工作是访问PathTable。但我无法使用合成将InputManager置于PathTable内,因为这是糟糕的设计。

实现这一目标的干净方法是什么?

我是初学者,我正在努力学习。

1 个答案:

答案 0 :(得分:1)

首先添加支持,以便在PathTable课程中编辑步骤的行:

class PathTable(object):
    def __init__(self):
        self._table = []

    ## NB : This defeats the whole point of making `_table` private
    #@property
    #def table(self):
    #    return self._table

    def add_step(self, step):
        ''' Adds a step to the table. '''
        self._table.append(step)

    def rm_step(self):
        ''' Removes the last step from the table. '''
        return self._table.pop()

    def update_step(self, position, key, value):
        self._table[position][key] = value

然后将PathTable个实例传递给您的InputManager

class InputManager(object):
    ''' Responsible for managing user input. '''
    def __init__(self, path_table):
        self._path_table = path_table

    def add_fight(self, position):
        ''' Add a 'fight' at table[position]. '''
        self._path_table.update_step(position, 'input', 'fight')