1

Tengo una variable save para almacenar el directorio que indica dónde guardar una grafica o, en caso contrario y por defecto, tiene el valor False.

Después, la siguiente sentencia:

if save != False:
    plt.savefig(save + "/" + title + ".png")

De modo que si save tiene un directorio, se cumple la condición y se guarda la imagen en el lugar indicado y si no, no se guarda la imagen y la ejecución del programa sigue su curso.

Pero me aparece advertencia (uso Spider):

comparison to False should be 'if cond is not False' or 'if cond:'

¿Hay alguna forma mejor de de resolver el problema?

7
  • if save is not False: Commented el 5 ene. 2018 a las 11:04
  • ¿Puedes desarrollar un poco la respuesta?
    – Zhisi
    Commented el 5 ene. 2018 a las 11:07
  • if save is not False: es mejor que if save != False:. Commented el 5 ene. 2018 a las 11:07
  • Si explicas por qué sería de agradecer.
    – Zhisi
    Commented el 5 ene. 2018 a las 11:08
  • 1
    if save: es mejor todavía. Commented el 5 ene. 2018 a las 11:08

1 respuesta 1

1

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 != ""

4
  • ¿La forma "elegante" que comentas es lo mismo que he hecho yo pero cambiando False por None y != por is not?
    – Zhisi
    Commented el 5 ene. 2018 a las 14:30
  • Si, pero por que digo que es más elegante, lo que estás verificando en castellano sería ¿save tiene un valor? esta pregunta es mucho más clara con if save is not None: y evitamos una mala práctica que es cambiar el tipo de una variable que es lo que ahora estás haciendo al hacer que sea una cadena en un caso o un booleano en otro. Commented el 5 ene. 2018 a las 19:58
  • Respecto al estilo de codificación PEP 8: 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!
    – Zhisi
    Commented el 8 ene. 2018 a las 8:34
  • Gracias @Zhisi, agrego tu comentario a la respuesta. Saludos. Commented el 8 ene. 2018 a las 15:09

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.