Я пишу класс для вставки пользователей в базу данных, и прежде чем я стану слишком далеким в, я просто хочу удостовериться, что мой подход 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()
Действительно ли это - правильный подход? Я должен объявить переменные класса вершина? Я должен использовать _ перед всеми переменными класса для создания их частными?
Спасибо за то, что выручили.
В целом это правильно, AFAIK, но вы можете очистить его с помощью свойств .
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()
Также есть удобный синтаксис на основе декораторов, это тоже объясняется в документации.
использование одного _ не делает ваши атрибуты закрытыми: это соглашение, говорящее, что это внутренний атрибут, и при нормальных обстоятельствах к нему не должен обращаться внешний код. С вашим кодом это также означает, что пароль и имя доступны только для чтения.
Я настоятельно рекомендую использовать инициализатор для вашего класса, который будет инициализировать ваши атрибуты, даже если используется значение по умолчанию, например None: это упростит задачу в методе фиксации, где вам не придется проверять наличие Атрибуты _name и _password (с hasattr).
Используйте Pylint в своем коде.
Другие уже указывали на это: избегайте использования сеттеров и геттеров и используйте простой доступ к атрибутам, если вам не нужно выполнять дополнительную логику при получении / установке атрибута. Если вам действительно нужна эта дополнительная логика, используйте характеристики.
Если у вас есть много параметров для передачи инициализатору экземпляра, рассмотрите возможность использования отдельного объекта или словаря, содержащего все параметры:
>>> 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'
P.S. В вашем случае вам не нужно объявлять свои атрибуты на уровне класса. Обычно это делается, если вы хотите использовать одно и то же значение для всех экземпляров класса:
>>> 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'
В этом коде нет переменных класса, только атрибуты экземпляра. И используйте свойства вместо методов доступа. И создайте атрибуты экземпляра в инициализаторе, предпочтительно из значений, переданных в:
class User(object):
def __init__(self, name, password='!!'):
self.name = name
self.password = password
...