Örnek değişkenleri Python'da bir sınıfta Yok olarak bildirmek iyi bir uygulama mıdır?

Aşağıdaki sınıfı göz önünde bulundurun:

class Person:
    def __init__(self, name, age):
        self.name = name
        self.age = age

İş arkadaşlarım bunu şöyle tanımlamaya meyillidir:

class Person:
    name = None
    age = None

    def __init__(self, name, age):
        self.name = name
        self.age = age

Bunun temel nedeni, seçtikleri editörün otomatik tamamlama özelliklerini göstermesidir.

Şahsen, ikincisini beğenmedim, çünkü bir sınıfın bu özelliklere sahip olduğu bir anlam ifade etmiyor None.

Hangisi daha iyi uygulama olacaktır ve hangi sebeplerden dolayı?

"Bu, senin yaptığını düşündüğünü yapmaz" kuralı altındaki ikinci kötü uygulama olarak adlandırıyorum.

İş arkadaşınızın pozisyonu şu şekilde yeniden yazılabilir: "Hiçbir zaman erişilemeyen, ancak çeşitli sınıfın ad alan tablolarında ( __dict__) yer alan, IDE'nin yapması için yer kaplayan bir grup sınıf-statik yarı-global değişken yaratacağım. bir şey."

1. Kodunuzun anlaşılmasını kolaylaştırın

Kod yazılandan çok daha sık okunur. Kod sağlayıcınızın görevini kolaylaştırın (gelecek yıl kendiniz de olabilir).

Sert kurallar hakkında hiçbir fikrim yok, ancak gelecekteki herhangi bir durumun açıkça açıkça ilan edilmesini tercih ederim . Bir ile gürültüyle çarpmak AttributeErroryeterince kötü. Bir örnek niteliğinin yaşam döngüsünü açıkça görmemek, daha kötüdür. Atanan arama dizilerini geri yüklemek için gereken zihin jimnastiğinin miktarı, atanan özelliğe neden olan kolayca önemsiz hale gelebilir ve hatalara yol açabilir.

Bu yüzden genellikle sadece yapıcıdaki her şeyi tanımlamakla kalmaz, aynı zamanda değişken niteliklerin sayısını minimumda tutmaya çalışırım.

2. Sınıf ve örnek seviye üyelerini karıştırmayın.

classBildirimin içinde tanımladığınız her şey sınıfa aittir ve sınıfın tüm örnekleri tarafından paylaşılır. Örneğin, bir sınıf içindeki bir işlevi tanımladığınızda, tüm örnekler için aynı olan bir yöntem haline gelir. Aynısı veri üyeleri için de geçerlidir. Bu tamamen tanımladığınız örnek özelliklerin aksinedir __init__.

Sınıf düzeyinde veri üyeleri en çok sabit olarak kullanışlıdır:

class Missile(object):
  MAX_SPEED = 100  # all missiles accelerate up to this speed
  ACCELERATION = 5  # rate of acceleration per game frame

  def move(self):
    self.speed += self.ACCELERATION
    if self.speed > self.MAX_SPEED:
      self.speed = self.MAX_SPEED
    # ...

Şahsen ben üyeleri __ init __ () metodunda tanımlarım. Onları sınıf bölümünde tanımlamayı hiç düşünmedim. Ama her zaman yaptığım şey: __ init__ yöntemindeki tüm üyeleri, __ init__ yönteminde gerekmeyenleri bile init ediyorum.

Örnek:

class Person:
    def __init__(self, name, age):
        self._name = name
        self._age = age
        self._selected = None

   def setSelected(self, value):
        self._selected = value

Tüm üyeleri tek bir yerde tanımlamanın önemli olduğunu düşünüyorum. Kodu daha okunaklı hale getirir. __ init __ () içinde mi yoksa dışında mı olduğu da o kadar önemli değil. Ancak bir ekibin aynı kodlama stiline az çok karar vermesi önemlidir.

Oh, fark edersiniz ki "_" ön ekini üye değişkenlerine ekledim.

Bu kötü bir uygulamadır. Bu değerlere ihtiyacınız yoktur, kodu bozarlar ve hatalara neden olabilirler.

Düşünmek:

>>> class WithNone:
...   x = None
...   y = None
...   def __init__(self, x, y):
...     self.x = x
...     self.y = y
... 
>>> class InitOnly:
...   def __init__(self, x, y):
...     self.x = x
...     self.y = y
... 
>>> wn = WithNone(1,2)
>>> wn.x
1
>>> WithNone.x #Note that it returns none, no error
>>> io = InitOnly(1,2)
>>> InitOnly.x
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: class InitOnly has no attribute 'x'

"Biraz dokümanlar gibi, sonra" ile gideceğim ve bu zararsız, her zaman olduğu gibiNone , ya da her zaman değişmeyen, diğer değerler arasında dar bir değer olduğunu ilan edeceğim.

Atavism ve statik olarak yazılan dillere aşırı bağlanma kokuyor. Ve kod olarak iyi değil. Ancak dokümantasyonda kalan küçük bir amacı vardır.

Beklenen adların ne olduğunu belgelemektedir, bu nedenle kodu biriyle kullanırsak ve birimizin 'kullanıcı adı' ve diğer kullanıcı_adı varsa, insanlara ayırdığımız ve aynı değişkenleri kullanmadığımıza dair bir ipucu vardır.

Bir ilke olarak tam başlatmaya zorlamak aynı şeyi daha Pythonic bir şekilde gerçekleştirir, ancak içinde gerçek bir kod varsa __init__, bu kullanılan değişkenleri belgelemek için daha net bir yer sağlar.

Açıkçası, buradaki BÜYÜK sorun, insanları Nonekötü olabilecek dışındaki değerlerle başlatmaya teşvik ediyor :

class X:
    v = {}
x = X()
x.v[1] = 2

küresel bir iz bırakır ve x için bir örnek oluşturmaz.

Fakat bu, Python'da bir bütün olarak bu uygulamadan çok bir tuhaflık ve bu konuda zaten paranoyak olmalıyız.