Skip to main content

Línea de tiempo para Advertencia en sentencia condicional

Licencia actual CC BY-SA 3.0

7 eventos
cuándo alternar formato qué por licencia comentario
el 8 ene. 2018 a las 15:09 comentario añadido Patricio Moracho Gracias @Zhisi, agrego tu comentario a la respuesta. Saludos.
el 8 ene. 2018 a las 15:09 historial editado Patricio Moracho CC BY-SA 3.0
se añadieron 856 caracteres en el cuerpo
el 8 ene. 2018 a las 8:34 comentario añadido Zhisi 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!
el 8 ene. 2018 a las 7:45 votar aceptar Zhisi
el 5 ene. 2018 a las 19:58 comentario añadido Patricio Moracho 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.
el 5 ene. 2018 a las 14:30 comentario añadido Zhisi ¿La forma "elegante" que comentas es lo mismo que he hecho yo pero cambiando False por None y != por is not?
el 5 ene. 2018 a las 14:11 historial respuesta Patricio Moracho CC BY-SA 3.0