Почему использование len (SEQUENCE) в значениях условий считается неправильным Pylint?

Учитывая этот фрагмент кода:

from os import walk

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

Пилинт встревожил меня этим сообщением относительно строки с оператором if:

[pylint] C1801: не использовать в len(SEQUENCE)качестве значения условия

Правило C1801, на первый взгляд, показалось мне не очень разумным, и определение в справочном руководстве не объясняет, почему это проблема. Фактически, это прямо называет это неправильным использованием .

len-as-condition (C1801) : Не использовать в len(SEQUENCE)качестве значения условия. Используется, когда Pylint обнаруживает неправильное использование len (последовательности) внутри условий.

Мои попытки поиска также не дали мне более глубокого объяснения. Я понимаю, что свойство длины последовательности может быть лениво оценено, и это __len__может быть запрограммировано, чтобы иметь побочные эффекты, но сомнительно, является ли это само по себе достаточно проблематичным для Pylint, чтобы назвать такое использование неправильным. Следовательно, прежде чем я просто настрою свой проект на игнорирование правила, я хотел бы знать, что я что-то упускаю в своих рассуждениях.

Когда использование len(SEQ)в качестве значения условия проблематично? Каких основных ситуаций Pylint пытается избежать с помощью C1801?

Когда использование len(SEQ)в качестве значения условия проблематично? Каких основных ситуаций Pylint пытается избежать с помощью C1801?

Это не очень проблематично в использовании len(SEQUENCE)- хотя это может быть не так эффективно (см . Комментарий Чепнера ). В любом случае, Pylint проверяет код на соответствие руководству по стилю PEP 8, в котором говорится, что

Для последовательностей (строк, списков, кортежей) используйте тот факт, что пустые последовательности являются ложными.

Yes: if not seq:
     if seq:

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

Будучи случайным программистом на Python, который переключается между языками, я считаю len(SEQUENCE)конструкцию более читаемой и явной («Явное лучше, чем неявное»). Однако использование того факта, что пустая последовательность оценивается Falseв логическом контексте, считается более «питоническим».

Обратите внимание, что на самом деле требуется использование len (seq) (вместо простой проверки значения bool seq) при использовании массивов NumPy.

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

приводит к исключению: ValueError: Значение истинности массива с более чем одним элементом является неоднозначным. Используйте a.any () или a.all ()

И, следовательно, для кода, который использует как списки Python, так и массивы NumPy, сообщение C1801 менее чем полезно.

Это была проблема в Pylint, и она больше не считается len(x) == 0неправильной.

Вы не должны использовать голое len(x) как условие. Сравнение len(x)с явным значением, таким как if len(x) == 0of if len(x) > 0, совершенно нормально и не запрещено PEP 8.

Из ППК 8 :

# Correct:
if not seq:
if seq:

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

Обратите внимание, что явное тестирование длины не запрещено. Дзен Python гласит:

Явное лучше, чем неявное.

В выборе между if not seqи if not len(seq), оба являются неявными, но поведение отличается. Но if len(seq) == 0или if len(seq) > 0явные сравнения и во многих контекстах правильное поведение.

В pylint PR 2815 исправил эту ошибку, впервые сообщенную как проблема 2684 . Он будет продолжать жаловаться if len(seq), но больше не будет жаловаться if len(seq) > 0. PR был объединен 2019-03-19, поэтому, если вы используете Pylint 2.4 (выпущен 2019-09-14), вы не должны видеть эту проблему.

Pylint терпел неудачу для моего кода, и исследование привело меня к этому сообщению:

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

Это был мой код раньше:

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

Это было после исправления моего кода. Используя int() attribute, я, кажется, удовлетворил Pep8 / Pylint и, похоже, не оказывает негативного влияния на мой код:

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

Мое исправление

Добавляя .__trunc__()к последовательности, кажется, что она исчерпала потребность.

Я не вижу различий в поведении, но если кто-то знает особенности, которые мне не хватает, пожалуйста, дайте мне знать.