No deben usarse comprensiones de listas por sus efectos secundarios. La misión de una comprensión de listas es crear una lista.
En tu caso en cambio la lista que estás creando no tiene sentido porque:
- No la asignas a ninguna variable
- No se sabe qué contiene, pues contiene el resultado de cada uno de los
s.Add()
que haces. Esa función no está definida y no sabemos qué retorna.
En el fondo, tu lista es similar a esta otra que ya he visto en más de una ocasión:
[print(n**2) for n in range(1, 21)]
que quiere imprimir la lista de los primeros 20 cuadrados perfectos. En realidad sí que está imprimiendo eso, pero utilizar una lista de comprehensión para ello es un antipatrón. La función print()
retorna None
y por tanto estamos creando una lista que contiene 20 Nones. A pesar de ello "funciona" debido al efecto secundario del print, que muestra por pantalla su argumento aunque retorne None.
Aunque la comprensión de listas suele ser una solución más Pythónica en muchos casos, ello no significa que haya que reemplazar todos los bucles for
que aparezcan en un programa por una comprensión de listas. Hay casos (como este) en el que no se gana nada, y si acaso se empeora la legibilidad. El ejemplo que acabo de poner estaría mucho mejor así:
for n in range(1, 21):
print(n**2)
Perfectamente legible y evita el desperdicio de memoria y tiempo necesarios para crear una lista temporal con 20 None que de todas formas no se va a almacenar en ningún lugar.
Lo mismo te está ocurriendo a ti con tu comprensión de listas. La implementación inicial en un bucle for
era mejor, pues la lista que estás construyendo con los resultados de s.Add()
requiere tiempo y memoria, y en realidad no quieres esa lista, sólo quieres el "efecto secundario" (que se añadan elementos a s
).
Si acaso, entiendo que quieras reducir la complejidad del código eliminando el bucle anidado (no lo has conseguido porque ese bucle anidado sigue existiendo dentro de la comprensión de listas, más difícil aún de entender). Puedes evitar anidaciones de bucles si haces uso de la función itertools.product
. En este caso sería así:
from itertools import product
for f, c in product(L_Filas, L_Columnas):
if tablaOrigD[f + 1][c + 1]:
s.Add(x[f][c][tablaOrigD[f + 1][c + 1] - 1] == 1)
s.Add()
no tendría que ir fuera de los[]
, rodeando la lista entera devuelta por la list comprehension??