我正在构建一个路径规划器,它将帮助人们计划通过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
内,因为这是糟糕的设计。
实现这一目标的干净方法是什么?
我是初学者,我正在努力学习。
答案 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')