Por que o uso de len (SEQUENCE) em valores de condição é considerado incorreto pelo Pylint?

Considerando este trecho de código:

from os import walk

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

Fiquei alarmado com Pylint com esta mensagem sobre a linha com a instrução if:

[pylint] C1801: Não use len(SEQUENCE)como valor de condição

A regra C1801, à primeira vista, não me pareceu muito razoável, e a definição no guia de referência não explica por que isso é um problema. De fato, ele chama de uso incorreto .

len como condição (C1801) : não use len(SEQUENCE)como valor de condição Usado quando o Pylint detecta o uso incorreto de len (sequência) dentro das condições.

Minhas tentativas de pesquisa também falharam em me fornecer uma explicação mais profunda. Entendo que a propriedade de comprimento de uma sequência pode ser avaliada preguiçosamente e que __len__pode ser programada para ter efeitos colaterais, mas é questionável se isso é problemático o suficiente para que Pylint chame esse uso de incorreto. Portanto, antes de simplesmente configurar meu projeto para ignorar a regra, gostaria de saber se estou perdendo alguma coisa no meu raciocínio.

Quando é len(SEQ)problemático o uso como valor de condição? Que situações principais a Pylint está tentando evitar com o C1801?

Quando é len(SEQ)problemático o uso como valor de condição? Que situações principais a Pylint está tentando evitar com o C1801?

Não é realmente problemático de usar len(SEQUENCE)- embora possa não ser tão eficiente (veja o comentário de chepner ). Independentemente disso, a Pylint verifica o código quanto à conformidade com o guia de estilo do PEP 8, que afirma que

Para seqüências (strings, listas, tuplas), use o fato de que sequências vazias são falsas.

Yes: if not seq:
     if seq:

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

Como programador ocasional em Python, que alterna entre linguagens, consideraria a len(SEQUENCE)construção mais legível e explícita (“Explícito é melhor que implícito”). No entanto, o uso do fato de que uma sequência vazia é avaliada Falseem um contexto booleano é considerado mais "pitônico".

Observe que o uso de len (seq) é de fato necessário (em vez de apenas verificar o valor bool de seq) ao usar matrizes NumPy.

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

resulta em uma exceção: ValueError: o valor verdadeiro de uma matriz com mais de um elemento é ambíguo. Use a.any () ou a.all ()

E, portanto, para o código que usa listas Python e matrizes NumPy, a mensagem C1801 é menos que útil.

Esse foi um problema no pylint e não é mais considerado len(x) == 0incorreto.

Você não deve usar o bare len(x) como condição. A comparação len(x)com um valor explícito, como if len(x) == 0de, if len(x) > 0é totalmente adequada e não é proibida pelo PEP 8.

Do PEP 8 :

# Correct:
if not seq:
if seq:

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

Observe que testar explicitamente o comprimento não é proibido. O Zen do Python afirma:

Explícito é melhor que implícito.

Na escolha entre if not seqe if not len(seq), ambos estão implícitos, mas o comportamento é diferente. Mas if len(seq) == 0ou if len(seq) > 0são comparações explícitas e, em muitos contextos, o comportamento correto.

Em pylint, o PR 2815 corrigiu esse bug, relatado pela primeira vez como problema 2684 . Ele continuará reclamando if len(seq), mas não reclamará mais if len(seq) > 0. O PR foi mesclado 19/03/2019, portanto, se você estiver usando o pylint 2.4 (lançado em 2019-09-14), não deverá encontrar esse problema.

Pylint estava falhando no meu código e a pesquisa me levou a este post:

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

Este foi o meu código antes:

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

Isso foi depois da minha correção de código. Ao usar o int() attribute, eu pareço ter satisfeito o Pep8 / Pylint e não parece ter um impacto negativo no meu código:

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

My Fix

Ao adicionar .__trunc__()à sequência, parece ter resolvido a necessidade.

Não vejo diferença no comportamento, mas se alguém souber de detalhes que estão faltando, entre em contato.