正确的方式来设置对象的实例变量

Question

我正在写一个类用户插入到数据库中，而之前，我太远，我只是想确保我的OO方法是干净的：

class User(object): 

    def setName(self,name): 

     #Do sanity checks on name 
     self._name = name 

    def setPassword(self,password): 

     #Check password length > 6 characters 
     #Encrypt to md5 
     self._password = password 

    def commit(self): 

     #Commit to database 

>>u = User() 
>>u.setName('Jason Martinez') 
>>u.setPassword('linebreak') 
>>u.commit() 

这是正确的做法吗？ 我应该向上声明类变量吗？ 我应该在所有类变量前面使用_来使它们变为私有的吗？

感谢您的帮助。

Answer 1

这通常是正确的，据我所知，但你可以用properties清理。

class User(object): 

    def _setName(self, name=None): 
     self._name = name 

    def _getName(self): 
     return self._name 

    def _setPassword(self, password): 
     self._password = password 

    def _getPassword(self): 
     return self._password 

    def commit(self): 
     pass 

    name = property(_getName, _setName) 
    password = property(_getPassword, _setPassword) 

>>u = User() 
>>u.name = 'Jason Martinez' 
>>u.password = 'linebreak' 
>>u.commit() 

还有一个方便的基于装饰器的语法，文档也解释了这一点。

Answer 2

该代码中没有类变量，只有实例属性。并使用properties而不是访问器。而且确实创造了在初始化实例的属性，最好在传递的值：

class User(object): 
    def __init__(self, name, password='!!'): 
    self.name = name 
    self.password = password 

    ... 

Answer 3

使用单个_不会使您的属性变为私有：它是一种约定，它告诉它是内部属性，并且不应在正常情况下由外部代码访问。使用您的代码也意味着密码和名称是只读的。

我强烈建议为您的类使用初始化程序，它将初始化您的属性，即使是默认值（如None）：它会使事情变得更容易，您不必检查是否存在_name和_password属性（使用hasattr）。

在您的代码上使用Pylint。

Answer 4

其他人已经指出：如果在获取/设置属性时不需要执行额外的逻辑，请避免使用setter和getter并使用简单的属性访问。 如果您确实需要额外的逻辑，请使用属性。

如果有许多参数传递给一个实例初始化考虑使用单独的对象或辞典保持所有参数：

>>> class User(object): 
...  def __init__(self, params): 
...   self.__dict__.update(params) 
... 

>>> params = { 
...  'username': 'john', 
...  'password': 'linebreak', 
...  } 
>>> user = User(params) 
>>> user.username 
'john' 
>>> user.password 
'linebreak' 

附：在你的情况下，你不需要在课堂级别声明你的属性。 通常情况下，如果要在所有类实例中共享相同的值，请执行以下操作：

>>> class User(object): 
...  type = 'superuser' 
... 
>>> user = User() 
>>> user2 = User() 
>>> 
>>> user.type 
'superuser' 
>>> user2.type 
'superuser' 
>>> 
>>> user2.type = 'instance superuser' 
>>> 
>>> user.type 
'superuser' 
>>> user2.type 
'instance superuser' 
>>> User.type 
'superuser'