Pregunta de diseño de clase para principiantes de Python

Question

Estoy tratando de encontrar la mejor manera de diseñar un par de clases. Soy bastante nuevo en Python (y OOP en general) y solo quiero asegurarme de estar haciendo esto bien. Tengo dos clases: "Usuarios" y "Usuario".

class User(object): 
    def __init__(self): 
     pass 

class Users(object): 
    def __init__(self): 
     self.users = [] 

    def add(self, user_id, email): 
     u = User() 
     u.user_id = user_id 
     u.email = email 
     self.users.append(u) 

users = Users() 
users.add(user_id = 1, email = 'bob@example.com') 

Si quiero recuperar mis usuarios, utilizo:

for u in users.users: 
    print u.email 

"users.users" parece ser un poco redundante. ¿Lo estoy haciendo bien?

Answer 1

Ponga esto en su Users:

def __iter__(self): 
    return iter(self.users) 

Ahora usted puede:

for u in users: 
    print u.email 

Docs

Answer 2

Es probable que desea es una lista de los objetos de usuario en lugar de una clase que contiene múltiples usuarios .

class User(object): 
    def __init__(self, user_id, email): 
     self.user_id = user_id 
     self.email = email 

users = [] 
users.append(User(user_id = 1, email = 'bob@example.com')) 

Todos los atributos de miembro para el usuario deberían residir en la clase de usuario, no en la clase de usuarios.

Answer 3

No veo nada malo con usuarios.usuarios, pero si prefiere una manera más agradable de hacerlo, puede anular __iter__ en Usuarios.

class Users(object): 
    def __init__(self): 
     self.users = [] 

    def add(self, user_id, email): 
     u = User() 
     u.user_id = user_id 
     u.email = email 
     self.users.append(u) 

    def __iter__(self): 
     return iter(self.users) 

Ahora usted puede hacer esto:

for u in users: 
    print u.email 

El método especial __iter__ hace que su objeto se comportan como un iterador

Answer 4

no diría realmente. Su clase Users parece ser solo una lista de usuarios, así que simplemente la convertiría en una lista en lugar de una clase completa. Esto es lo que yo haría:

class User(object): 
    def __init__(self, user_id=None, email=None): 
     self.user_id, self.email = user_id, email 

users = [] 
users.append(User(user_id = 1, email = 'bob@example.com')) 

for u in users: 
    print u.email 

Si desea Users a ser una clase propia por alguna otra razón, usted podría tener que herede de list, o (si no) se podría añadir estos a la definición:

class Users(object): 
    # rest of code 
    def __iter__(self): 
     return iter(self.users) 

esta manera, se puede decir simplemente:

users = Users() 
... 
for u in users: 
    print u.email 

Answer 5

no hay "negro" y "blanco" aquí, sólo matices de gris. No necesita una clase especial Users si solo va a ser una lista.

Otra forma:

class User: 
    all_users = [] 

    def __init__(self, id, email): 
     self.id = id # No need to call it user_id - it's a User object, after all! 
     self.email = email 
     self.all_users.append(self) #automatically add to list of all users 

    def __str__(self): 
     return '%s(%s)' % (self.id, self.email) 

Entonces, si ha escrito lo anterior en user.py:

 
>>> from user import * 
>>> bob = User('bob', 'bob@test.com') 
>>> alice = User('alice', 'alice@test.com') 
>>> for u in User.all_users: 
...  print u 
... 
bob(bob@test.com) 
alice(alice@test.com) 
>>> 

Sólo un ejemplo para hacerle pensar.