Warum wird die Verwendung von len (SEQUENCE) in Bedingungswerten von Pylint als falsch angesehen?

In Anbetracht dieses Code-Snippets:

from os import walk

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

Ich wurde von Pylint mit dieser Meldung bezüglich der Zeile mit der if-Anweisung alarmiert:

[pylint] C1801: Nicht verwenden len(SEQUENCE) als Bedingungswert verwenden

Die Regel C1801 klang auf den ersten Blick für mich nicht sehr vernünftig, und die Definition im Referenzhandbuch erklärt nicht, warum dies ein Problem ist. Tatsächlich nennt es es geradezu eine falsche Verwendung .

len-as-condition (C1801) : Nicht len(SEQUENCE)als Bedingungswert verwenden Wird verwendet, wenn Pylint eine falsche Verwendung von len (Sequenz) innerhalb von Bedingungen feststellt.

Meine Suchversuche haben mir auch keine tiefere Erklärung geliefert. Ich verstehe, dass die Längeneigenschaft einer Sequenz möglicherweise träge ausgewertet wird und __len__so programmiert werden kann, dass sie Nebenwirkungen hat, aber es ist fraglich, ob dies allein für Pylint problematisch genug ist, um eine solche Verwendung als falsch zu bezeichnen. Bevor ich mein Projekt einfach so konfiguriere, dass die Regel ignoriert wird, möchte ich wissen, ob mir etwas in meiner Argumentation fehlt.

Wann ist die Verwendung len(SEQ)als Bedingungswert problematisch? Welche wichtigen Situationen versucht Pylint mit C1801 zu vermeiden?

Wann ist die Verwendung len(SEQ)als Bedingungswert problematisch? Welche wichtigen Situationen versucht Pylint mit C1801 zu vermeiden?

Die Verwendung ist nicht wirklich problematisch len(SEQUENCE)- obwohl sie möglicherweise nicht so effizient ist (siehe chepners Kommentar ). Unabhängig davon überprüft Pylint den Code auf Übereinstimmung mit dem PEP 8- Styleguide , in dem dies angegeben ist

Verwenden Sie für Sequenzen (Zeichenfolgen, Listen, Tupel) die Tatsache, dass leere Sequenzen falsch sind.

Yes: if not seq:
     if seq:

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

Als gelegentlicher Python-Programmierer, der zwischen Sprachen wechselt, würde ich das len(SEQUENCE)Konstrukt als lesbarer und expliziter betrachten („Explizit ist besser als implizit“). Die Verwendung der Tatsache, dass eine leere Sequenz Falsein einem booleschen Kontext ausgewertet wird, wird jedoch als „pythonischer“ angesehen.

Beachten Sie, dass bei Verwendung von NumPy-Arrays tatsächlich die Verwendung von len (seq) erforderlich ist (anstatt nur den Bool-Wert von seq zu überprüfen).

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

führt zu einer Ausnahme: ValueError: Der Wahrheitswert eines Arrays mit mehr als einem Element ist nicht eindeutig. Verwenden Sie a.any () oder a.all ()

Für Code, der sowohl Python-Listen als auch NumPy-Arrays verwendet, ist die Nachricht C1801 daher weniger hilfreich.

Dies war ein Problem bei Pylint und wird nicht mehr len(x) == 0als falsch angesehen.

Sie sollten kein Bare len(x) als Bedingung verwenden. Vergleichenlen(x) mit einem expliziten Wert wie if len(x) == 0von if len(x) > 0ist völlig in Ordnung und wird von PEP 8 nicht verboten.

Aus PEP 8 :

# Correct:
if not seq:
if seq:

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

Beachten Sie, dass das explizite Testen der Länge nicht verboten ist. Das Zen von Python besagt:

Explizit ist besser als implizit.

Bei der Wahl zwischen if not seqund if not len(seq)sind beide implizit, aber das Verhalten ist unterschiedlich. Aber if len(seq) == 0oderif len(seq) > 0 sind explizite Vergleiche und in vielen Zusammenhängen das richtige Verhalten.

In Pylint hat PR 2815 diesen Fehler behoben, der zuerst als Problem 2684 gemeldet wurde . Es wird sich weiterhin beschweren if len(seq), aber es wird sich nicht mehr beschweren if len(seq) > 0. Die PR wurde am 19.03.2019 zusammengeführt. Wenn Sie also Pylint 2.4 (veröffentlicht am 14.09.2019) verwenden, sollte dieses Problem nicht auftreten.

Pylint versagte bei meinem Code und die Recherche führte mich zu diesem Beitrag:

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

Dies war mein Code vor:

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

Dies war nach meinem Code-Fix. Durch die Verwendung von int() attributescheint ich den Pep8 / Pylint zufrieden zu stellen und habe keinen negativen Einfluss auf meinen Code:

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

Mein Fix

Durch das Hinzufügen .__trunc__()zur Sequenz scheint der Bedarf gedeckt zu sein.

Ich sehe keinen Unterschied im Verhalten, aber wenn jemand Einzelheiten weiß, die mir fehlen, lassen Sie es mich bitte wissen.