E731 bir lambda ifadesi atamaz, bir def kullanın

Lambda ifadeleri kullandığımda bu pep8 uyarısını alıyorum. Lambda ifadeleri önerilmez mi? Değilse neden?

Karşılaştığınız PEP-8'deki öneri :

Lambda ifadesini doğrudan bir ada bağlayan atama ifadesi yerine her zaman bir def ifadesi kullanın.

Evet:

def f(x): return 2*x 

Hayır:

f = lambda x: 2*x 

İlk form, sonuçta ortaya çıkan işlev nesnesinin adının, genel '<lambda>' yerine özellikle 'f' olduğu anlamına gelir. Bu, genel olarak izlemeler ve dize gösterimleri için daha kullanışlıdır. Atama ifadesinin kullanımı, bir lambda ifadesinin açık bir def ifadesi üzerinden sunabileceği tek avantajı ortadan kaldırır (yani, daha büyük bir ifadenin içine gömülebilir)

Adlara lambdas atamak temel olarak sadece işlevselliğini çoğaltır def- ve genel olarak, karışıklığı önlemek ve netliği artırmak için tek bir yol yapmak en iyisidir.

Lambda için yasal kullanım durumu, bir işlevi atamadan kullanmak istediğiniz yerdir, örneğin:

sorted(players, key=lambda player: player.rank)

Genel olarak, bunu yapmamaya yönelik ana argüman, defifadelerin daha fazla kod satırına neden olacağıdır. Buna benim ana cevabım: evet, ve bu iyi. Kod golf yapmadığınız sürece, çizgi sayısını en aza indirmek yapmanız gereken bir şey değildir: kısaca netleşin.

İşte hikaye, iki kez kullandığım basit bir lambda fonksiyonum vardı.

a = map(lambda x : x + offset, simple_list)
b = map(lambda x : x + offset, another_simple_list)

Bu sadece temsil için, bunun birkaç farklı versiyonu ile karşılaştım.

Şimdi, şeyleri KURU tutmak için, bu ortak lambda'yı tekrar kullanmaya başlarım.

f = lambda x : x + offset
a = map(f, simple_list)
b = map(f, another_simple_list)

Bu noktada benim kod kalite denetleyicisi lambda adlı bir işlev olmaktan şikayet ediyor, bu yüzden bir işleve dönüştürmek.

def f(x):
    return x + offset
a = map(f, simple_list)
b = map(f, another_simple_list)

Şimdi denetleyici, bir işlevin önce ve sonra bir satır boşluk ile sınırlandırılması gerektiğinden şikayet ediyor.

def f(x):
    return x + offset

a = map(f, simple_list)
b = map(f, another_simple_list)

Burada, okunabilirlikte bir artış ve pitonik olarak bir artış olmadan orijinal 2 satır yerine 6 kod satırımız var. Bu noktada kod denetleyicisi fonksiyonun docstrings'e sahip olmamasından şikayet eder.

Bence bu kural mantıklı olduğunda kaçınılmalı ve kırılmalıdır, kararınızı kullanın.

Lattyware kesinlikle doğru: Temelde PEP-8 gibi şeylerden kaçınmanızı istiyor

f = lambda x: 2 * x

ve bunun yerine kullan

def f(x):
    return 2 * x

Ancak, son bir hata raporunda (Ağustos 2014) ele alındığı gibi, aşağıdaki gibi ifadeler artık uyumludur:

a.f = lambda x: 2 * x
a["f"] = lambda x: 2 * x

PEP-8 denetleyicim bunu henüz doğru bir şekilde uygulamadığından şimdilik E731'i kapattım.

Ayrıca def (ined) işlevini kullanmanın bile imkansız olduğu bir durumla karşılaştım.

class SomeClass(object):
  # pep-8 does not allow this
  f = lambda x: x + 1  # NOQA

  def not_reachable(self, x):
    return x + 1

  @staticmethod
  def also_not_reachable(x):
    return x + 1

  @classmethod
  def also_not_reachable(cls, x):
    return x + 1

  some_mapping = {
      'object1': {'name': "Object 1", 'func': f},
      'object2': {'name': "Object 2", 'func': some_other_func},
  }

Bu durumda, gerçekten sınıfa ait bir harita yapmak istedim. Eşlemedeki bazı nesnelerin aynı işleve ihtiyacı vardı. Adlandırılmış bir işlevi sınıfın dışına koymak mantıksız olurdu. Sınıf gövdesi içinden bir yönteme (statik yöntem, sınıf yöntemi veya normal) başvurmak için bir yol bulamadım. Kod çalıştırıldığında SomeClass henüz mevcut değil. Yani sınıftan bahsetmek de mümkün değil.