Die richtige Art und Weise Variablen Objektinstanz zu setzen

Question

Ich schreibe eine Klasse Benutzer in eine Datenbank einzufügen, und bevor ich zu weit in, ich möchte nur sicherstellen, dass meine OO Ansatz ist sauber:

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() 

Ist das der richtige Ansatz? Soll ich Klassenvariablen nach oben deklarieren? Sollte ich _ vor allen Klassenvariablen verwenden, um sie privat zu machen?

Vielen Dank für Ihre Hilfe.

Answer 1

Es ist grundsätzlich richtig, AFAIK, aber man konnte es mit properties aufzuräumen.

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() 

Es gibt auch eine bequeme dekoratorbasierte Syntax, die Dokumentation erklärt das auch.

Answer 2

In diesem Code gibt es keine Klassenvariablen, nur Instanzattribute. Und verwenden Sie properties anstelle von Accessoren. Und tun erstellen die Instanz in der Initialisierungsliste Attribute, vorzugsweise von Werten in geben:

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

    ... 

Answer 3

mit einem einzigen _ macht nicht Ihre Attribute privat: Es ist eine Konvention, die besagt, dass es ein internes Attribut ist und nicht unter normalen Umständen durch externen Code zugegriffen werden sollte. Mit Ihrem Code bedeutet es auch, dass Passwort und Name nur gelesen werden können.

Ich empfehle dringend, einen Initialisierer für Ihre Klasse zu verwenden, der Ihre Attribute initialisiert, auch wenn sie auf einen Standardwert wie None gesetzt ist: Es erleichtert die Commit-Methode, bei der Sie nicht auf Existenz prüfen müssen die Attribute _name und _password (mit hasattr).

Verwenden Sie Pyylint auf Ihrem Code.

Answer 4

Andere haben bereits darauf hingewiesen: Vermeiden Sie die Verwendung von Setter und Getter und verwenden Sie einfachen Attributzugriff, wenn Sie beim Abrufen/Festlegen Ihres Attributs keine zusätzliche Logik ausführen müssen. Wenn Sie diese zusätzliche Logik benötigen, dann verwenden Sie Eigenschaften.

Wenn Sie viele Parameter zu einer Instanz initializer passieren prüfen, mit separatem Objekt oder Wörterbuch hält alle Parameter:

>>> 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. In diesem Fall müssen Sie Ihre Attribute nicht auf Klassenebene deklarieren. Dies ist normalerweise der Fall, wenn Sie für alle Klasseninstanzen den gleichen Wert verwenden möchten:

>>> 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'