¿Por qué Pylint considera incorrecto el uso de len (SECUENCIA) en los valores de condición?

Teniendo en cuenta este fragmento 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

Pylint me alarmó con este mensaje con respecto a la línea con la declaración if:

[pylint] C1801: No utilizar len(SEQUENCE)como valor de condición

La regla C1801, a primera vista, no me pareció muy razonable, y la definición en la guía de referencia no explica por qué esto es un problema. De hecho, lo llama francamente un uso incorrecto .

len-as-condition (C1801) : no utilizar len(SEQUENCE)como valor de condición Se utiliza cuando Pylint detecta el uso incorrecto de len (secuencia) dentro de las condiciones.

Mis intentos de búsqueda tampoco han podido proporcionarme una explicación más profunda. Entiendo que la propiedad de longitud de una secuencia puede evaluarse perezosamente, y que __len__se puede programar para que tenga efectos secundarios, pero es cuestionable si eso solo es lo suficientemente problemático como para que Pylint considere incorrecto dicho uso. Por lo tanto, antes de simplemente configurar mi proyecto para ignorar la regla, me gustaría saber si me falta algo en mi razonamiento.

¿Cuándo es len(SEQ)problemático el uso de como valor de condición? ¿Qué situaciones principales intenta evitar Pylint con C1801?

¿Cuándo es len(SEQ)problemático el uso de como valor de condición? ¿Qué situaciones principales intenta evitar Pylint con C1801?

No es realmente problemático de usar len(SEQUENCE), aunque puede que no sea tan eficiente (vea el comentario de chepner ). Independientemente, Pylint verifica el código para el cumplimiento de la guía de estilo PEP 8 que establece que

Para secuencias (cadenas, listas, tuplas), use el hecho de que las secuencias vacías son falsas.

Yes: if not seq:
     if seq:

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

Como programador ocasional de Python, que pasa de un idioma a otro, consideraría que la len(SEQUENCE)construcción es más legible y explícita ("explícito es mejor que implícito"). Sin embargo, usar el hecho de que una secuencia vacía se evalúa Falseen un contexto booleano se considera más "Pitónico".

Tenga en cuenta que, de hecho, se requiere el uso de len (seq) (en lugar de simplemente verificar el valor bool de seq) cuando se usan matrices NumPy.

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

da como resultado una excepción: ValueError: el valor de verdad de una matriz con más de un elemento es ambiguo. Utilice a.any () o a.all ()

Y, por lo tanto, para el código que usa listas Python y matrices NumPy, el mensaje C1801 no es útil.

Este fue un problema en pylint, y ya no se considera len(x) == 0incorrecto.

No debe usar un desnudo len(x) como condición. Comparar len(x)con un valor explícito, como if len(x) == 0de, if len(x) > 0está totalmente bien y no está prohibido por PEP 8.

De PEP 8 :

# Correct:
if not seq:
if seq:

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

Tenga en cuenta que no está prohibido probar explícitamente la longitud . El Zen de Python dice:

Explícito es mejor que implícito.

En la elección entre if not seqy if not len(seq), ambos son implícitos pero el comportamiento es diferente. Pero if len(seq) == 0o if len(seq) > 0son comparaciones explícitas y en muchos contextos el comportamiento correcto.

En pylint, PR 2815 ha solucionado este error, primero informado como el problema 2684 . Continuará quejándose if len(seq), pero ya no se quejará if len(seq) > 0. El PR se fusionó el 19/03/2019, por lo que si está utilizando pylint 2.4 (lanzado el 14/09/2019), no debería ver este problema.

Pylint estaba fallando en mi código y la investigación me llevó a esta publicación:

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

Esto fue después de la reparación de mi código. Al usar elint() attribute , parece que he satisfecho el Pep8 / Pylint y no parece tener un impacto negativo en mi 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)))

Mi arreglo

Añadiendo .__trunc__() a la secuencia parece haber resuelto la necesidad.

No veo una diferencia en el comportamiento, pero si alguien conoce los detalles que me faltan, hágamelo saber.