Pylint tarafından koşul değerlerinde len (SEQUENCE) kullanımı neden yanlış kabul edilir?

Bu kod snippet'i göz önüne alındığında:

from os import walk

files = []
for (dirpath, _, filenames) in walk(mydir):
    # more code that modifies files
if len(files) == 0: # <-- C1801
    return None

Pylint tarafından if ifadesi ile ilgili bu mesajla telaşlandım:

[pylint] C1801: Kullanmayın len(SEQUENCE) Koşul değeri olarak

İlk bakışta C1801 kuralı benim için çok makul gelmedi ve referans kılavuzundaki tanım bunun neden bir sorun olduğunu açıklamıyor. Aslında, düpedüz yanlış kullanım olarak adlandırır .

koşul olarak len (C1801) : Koşul değeri olarak kullanmayın len(SEQUENCE)Pylint koşullar içinde len (sekans) yanlış kullanımını tespit ettiğinde kullanılır.

Arama girişimlerim de bana daha derin bir açıklama yapamadı. Bir dizinin length özelliğinin tembel olarak değerlendirilebileceğini ve__len__ yan etkileri olacak şekilde programlanabileceğini anlıyorum, ancak bunun tek başına Pylint'in böyle bir kullanımı yanlış olarak adlandırması için yeterince sorunlu olup olmadığı tartışmalıdır. Bu nedenle, projemi kuralı göz ardı edecek şekilde yapılandırmadan önce, akıl yürütmemde bir şey eksik olup olmadığımı bilmek istiyorum.

len(SEQ)Koşul değeri olarak kullanımı ne zaman sorunludur? Pylint C1801 ile hangi önemli durumlardan kaçınmaya çalışıyor?

Ne zaman kullanılır len(SEQ)Koşul değeri olarak sorunludur? Pylint C1801 ile hangi önemli durumlardan kaçınmaya çalışıyor?

O değil gerçekten kullanımına sorunlu len(SEQUENCE)- bu (bkz verimli olarak olmayabilir ama chepner yorumuna ). Ne olursa olsun, Pylint kodu PEP 8 stil kılavuzuna uygunluk açısından kontrol eder .

Diziler (dizeler, listeler, tuples) için boş dizilerin yanlış olduğu gerçeğini kullanın.

Yes: if not seq:
     if seq:

No:  if len(seq):
     if not len(seq):

Diller arasında flört eden bir Python programcısı olarak, len(SEQUENCE)yapının daha okunaklı ve açık olduğunu düşünürdüm (“Açık, üstü kapalı olmak daha iyidir”). Ancak, boş bir dizinin FalseBoole bağlamında değerlendirildiği gerçeğinin kullanılması daha “Pythonic” olarak değerlendirilir.

NumPy dizilerini kullanırken aslında len (seq) kullanımının gerekli olduğunu unutmayın (sadece seq'nin bool değerini kontrol etmek yerine).

a = numpy.array(range(10))
if a:
    print "a is not empty"

bir istisna ile sonuçlanır: ValueError: Birden fazla öğeye sahip bir dizinin doğruluk değeri belirsiz. A.any () veya a.all () kullanın

Bu nedenle hem Python listelerini hem de NumPy dizilerini kullanan kodlar için C1801 mesajı yardımcı olmaktan daha azdır.

Bu, pylint'te bir konuydu ve artık dikkate almıyor len(x) == 0 yanlış olarak .

Bir çıplakı len(x) koşul olarak kullanmamalısınız . Karşılaştırmak len(x)gibi, açık bir değere karşı if len(x) == 0arasındaif len(x) > 0 tamamen iyi değil PEP 8 yasaklanan olduğunu.

Gönderen PEP 8 :

# Correct:
if not seq:
if seq:

# Wrong:
if len(seq):
if not len(seq):

Not açıkça uzunluğu için test yasaklanmamıştır. Python Zen devletler:

Açık, örtük olmaktan iyidir.

Arasındaki seçim olarak if not seqve if not len(seq)hem örtük ama davranış farklıdır. Ancak if len(seq) == 0veyaif len(seq) > 0 da açık karşılaştırmalar ve birçok bağlamda doğru davranış.

Pylint'te, PR 2815 , ilk olarak 2684 numaralı sorun olarak bildirilen bu hatayı düzeltti . Şikayet etmeye devam edecek if len(seq), ancak artık şikayet etmeyecek if len(seq) > 0. PR 2019-03-19 ile birleştirildi, bu nedenle pylint 2.4 (2019-09-14 yayınlandı) kullanıyorsanız bu sorunu görmemelisiniz.

Pylint kodum için başarısız oldu ve araştırma beni bu gönderiye götürdü:

../filename.py:49:11: C1801: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)
../filename.py:49:34: C1801: Do not use `len(SEQUENCE)` to determine if a sequence is empty (len-as-condition)

Bu önce kodum oldu:

def list_empty_folders(directory):
"""The Module Has Been Build to list empty Mac Folders."""
for (fullpath, dirnames, filenames) in os.walk(directory):
    if len(dirnames) == 0 and len(filenames) == 0:
        print("Exists: {} : Absolute Path: {}".format(
            os.path.exists(fullpath), os.path.abspath(fullpath)))

Bu benim kod düzeltme sonra oldu. Kullanarak int() attribute, ben Pep8 / Pylint memnun gibi görünüyor ve benim kod üzerinde olumsuz bir etkisi gibi görünmüyor:

def list_empty_folders(directory):
"""The Module Has Been Build to list empty Mac Folders."""
for (fullpath, dirnames, filenames) in os.walk(directory):
    if len(dirnames).__trunc__() == 0 and len(filenames).__trunc__() == 0:
        print("Exists: {} : Absolute Path: {}".format(
            os.path.exists(fullpath), os.path.abspath(fullpath)))

Düzeltmem

Diziye ekleyerek .__trunc__()ihtiyacı çözmüş gibi görünüyor.

Davranışta bir fark görmüyorum, ancak kimse özlediğim özellikleri biliyorsa, lütfen bana bildirin.