我试图了解如何在逻辑上分离 CRUD 职责,以便遵守单一职责原则 (SRP)。
根据我对 SRP 定义的理解,单一责任不一定是单一行为,而是行为的集合,与其他行为具有明确定义的逻辑边界。
在我的示例中,RestaurantMenu 只是一个集合。我知道有更有效的方法来表示这一点,例如使用字典,但这超出了本示例的意图。我的 RestaurantMenu 没有为其分配任何行为,因为我仍不清楚它定义的任何进一步行为是否会违反 SRP。通过 Manager 对象而不是通过 RestaurantMenu 中的方法实例化和调用单独的 CRUD 对象感觉相当不舒服,因此我决定向这里的观众寻求一些指导。
以下示例是否通过了 SRP 试金石?
class RestaurantMenu(object):
def __init__(self, title, creator, catalog_type, restaurant):
self._title = title
self._creator = creator
self._catalog_type = catalog_type
self._restaurant = restaurant
self._menuitems = dict()
class MenuManager(object):
"""Responsibility
--------------
Coordinates CRUD related activities with a menu
"""
def __init__(self, menu):
self._menu = menu
def add_menu_item(self, item, value):
menu_item_adder = AddMenuItem(self._menu)
menu_item_adder(item, value)
def del_menu_item(self, item):
menu_item_deleter = DelMenuItem(self._menu)
menu_item_deleter(item)
def update_menu_item(self, existing_item, new_info):
menu_item_updater = UpdateMenuItem(self._menu)
menu_item_updater(existing_item, new_info)
def get_menu_items(self):
menu_item_getter = GetMenuItems(self._menu)
menu_item_getter()
class GetMenuItems(object):
def __init__(self, menu):
self._menu = menu
def __call__(self):
print(self._menu._title)
print('='*len(self._menu._title))
for key, value in self._menu._menuitems.items():
print(key, value)
class AddMenuItem(object):
def __init__(self, menu):
self._menu = menu
def __call__(self, item, value):
if item not in self._menu._menuitems:
self._menu._menuitems[item] = value
print('Item added:', item)
else:
print('Item already exists. Please update instead.')
class DelMenuItem(object):
def __init__(self, menu):
self._menu = menu
def __call__(self, item):
popped = self._menu._menuitems.pop(item)
print('Item removed:', popped)
class UpdateMenuItem(object):
def __init__(self, menu):
self._menu = menu
def __call__(self, existing_item, new_info):
self._menu._menuitems.update(existing_item=new_info)
print('Item updated:', existing_item, ' with', new_info)
def main():
mymenu = RestaurantMenu("Joe's Crab Shack 2014 Menu",
"Joe Schmoe",
"Restaurant",
"Joe's Crab Shack")
menumanager = MenuManager(mymenu)
menumanager.add_menu_item('longneck_clams', 7.00)
menumanager.add_menu_item('1 pound lobster', 15.00)
menumanager.add_menu_item('lobster chowder', 9.00)
print('-'*50)
menumanager.get_menu_items()
if __name__ == "__main__":
main()
最佳答案
SRP 合规性的一个可能定义是应该只有 one reason for a class to change .
这使得在抽象的一段代码上调用 SRP 或不调用 SRP 变得非常困难 - 这基本上取决于应用程序中随着时间的推移会一起或单独发展的情况。
一般来说,UI 是可能独立于程序其他部分而发展的主要事物之一。用户总是希望在项目过程中进行少量的显示调整,能够修改呈现逻辑而不用担心破坏系统的其余部分是一件好事。持久性是您可能想要更改的另一件事,无论是由于新的架构决策还是暂时的,具体取决于上下文(例如在测试中交换虚拟持久性对象)。
这就是为什么在大多数实际应用程序中,我倾向于按技术职责而不是按 C/R/U/D 等同一实体上的业务操作来划分类。
如果您仔细观察当前的实现,您会注意到类中的模式。他们都在摆弄 MenuManager
和其中存储的 MenuItems
。它们都打印东西到屏幕上。
如果您想更改数据显示或存储的方式,您基本上必须接触所有这些类。我并不是说对于像这样的小型简单系统来说这是一个严重的缺陷,但在更大的应用程序中它很可能是一个问题。
换句话说,您的示例可以轻松地通过图形界面将菜单更新到 SQL 数据库,通过命令 shell 将菜单插入到平面文件中,并且菜单读取将数据从 XML 文件中收集出来。网络服务。这可能是您在非常特殊的情况下想要做的事情,但不是大多数时候......
关于python - 针对集合的 CRUD 操作的单一职责原则,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/24160856/