我不喜欢你的解决方案,原因有很多,包括:元素不应对其在容器中的成员身份负责。应该由容器负责添加和删除。在
通常不希望方法只修改参数。方法通常用于修改实例,可能会对参数产生一些副作用,但我不希望有一个方法只修改参数。在
该方法没有添加任何真正的功能,只是它隐藏了Person是如何插入到列表中的。例如,如果您遵循这种设计,为什么不添加一个从列表中删除Person的方法?在
我发现:person.add_to_list(people)
很难读。
如果您认为您不会更改Person的容器,即始终使用list,那么为什么不直接使用列表呢?它更简单,开销更小,并且使您编写的代码更少:
^{pr2}$
如果您认为可能要更改所使用的容器,那么我认为最好使用add和remove方法编写一个People类(以及您需要的任何其他方法)。幕后People可以使用它想要实现这些方法的任何容器。通过这种方式,您可以获得一个可读、直观和健壮的设计:class People(object):
def __init__(self, people=()):
self._members = list(people)
# add other methods/special methods like __len__ etc.
def add(self, person):
self._members.append(person)
def remove(self, person):
self._members.remove(person)
people = People()
people.add(john)
people.add(jack)
# Change list to set?
class People(object):
def __init__(self, people=()):
self._members = set(people)
def add(self, person):
self._members.add(person)
def remove(self, person):
self._members.remove(person)