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 |