Vamos por partes. Lo que recibes es una advertencia, que te la da Spyder, seguramente por que verifica automáticamente tu código contra la PEP 8 -- Style Guide for Python Code que es el documento oficial de estilos de codificación la cual comenta:
Comparisons to singletons like None should always be done with is or
is not, never the equality operators. Also, beware of writing if x
when you really mean if x is not None -- e.g. when testing whether a
variable or argument that defaults to None was set to some other
value. The other value might have a type (such as a container) that
could be false in a boolean context!
Traducción libre:
Las comparaciones entre singletons como "None" se debe hacer siempre con
"is" o "is not", nunca con los operadores de igualdad. También, tenga
cuidado al escribir "if x" cuando realmente quiere decir "if not x" --
por ejemplo, cuando pruebe si una variable o argumento que por defecto
es "None" tiene otro valor. El otro valor podría tener un tipo (por
ejemplo, un contenedor) que podría ser falso en un contexto booleano!
Tu construcción if save != False:
es válida sintácticamente pero no es el estilo "pythonico", la forma recomendada de escribir esta sentencia sería: if not save:
.
El otro problema que tienes es que está reutilizando una misma variable (save
) para guardar en un caso un str
, el directorio que mencionas y caso contrario un bool
, el valor False
si no tienes dicho directorio. No es recomendable hacerlo así, sin embargo si este fuera el caso, lo más sencillo es verificar si el tipo de datos de save
es una cadena, en cuyo caso tenemos seguridad que la variable tiene el directorio, por ejemplo:
if isinstance(save, str):
print("save tiene el directorio")
Pero hay una forma mucho más elegante de resolverlo: en primer lugar, deberemos inicializar la variable save
como None
, y además deberemos no modificar el valor en caso de no tener una carpeta como lo estás haciendo ahora:
save = None
save = "Carpeta" # Proba comentar esta linea
if save is not None:
plt.savefig(save + "/" + title + ".png")
Nota: Otra forma es inicializar save
con una cadena en blanco y preguntar por if save != ""
if save is not False:
if save is not False:
es mejor queif save != False:
.if save:
es mejor todavía.