Python newbie sınıfı tasarım sorusu

Question

Birkaç sınıf tasarlamanın en iyi yolunu bulmaya çalışıyorum. Python (ve genel olarak OOP) için oldukça yeni ve sadece bunu doğru yaptığımdan emin olmak istiyorum. İki sınıfım var: "Kullanıcılar" ve "Kullanıcı". Benim kullanıcılar almak istiyorsanız

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

, ben kullanın:

for u in users.users: 
    print u.email 

"users.users" biraz gereksiz gibi görünüyor. Bunu doğru mu yapıyorum?

Answer 1

senin Users koy:

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

Artık yapabilirsiniz:

for u in users: 
    print u.email 

Docs

Answer 2

Muhtemelen sadece Kullanıcı nesnelerin listesini yerine birden çok kullanıcıyı içeren bir sınıf istiyoruz .

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

Bütün üye Kullanıcı sınıfından değil Kullanıcılar sınıf bulunması gereken Kullanıcı için bağlıyor.

Answer 3

Kullanıcılar ile ilgili yanlış bir şey göremiyorum, ancak bunu yapmanın daha iyi bir yolunu tercih ederseniz, Kullanıcılar'da __iter__'u geçersiz kılabilirsiniz.

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) 

Şimdi bunu yapabilirsiniz:

for u in users: 
    print u.email 

__iter__ özel yöntem nesne yapar davranmasına

Answer 4

ben gerçekten söyleyebilirim bir yineleyici olarak. Users sınıfınızın sadece bir kullanıcı listesi olduğu anlaşılıyor, bu yüzden tüm bir sınıf yerine bir liste hazırlayacağım. İşte öyle yapardım:

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 

Eğer başka bir nedenden dolayı kendi başına bir sınıf olmak Users istiyorsanız, size tanımına bu ekleyebilir (değilse) o list devralan veya olabilir:

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

Bu şekilde, basitçe söyleyebiliriz:

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

Answer 5

hayır "beyaz", "siyah" ve burada, gri sadece tonları var. Sadece bir liste olacaksa özel bir Users sınıfına ihtiyacınız yoktur.

bir başka yolu: Sonra

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) 

, sen user.py yukarıdaki yazdıysanız:

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

Sadece bir örnek

düşündüğünü olsun.