Asignación de los condicionales

Question

que he visto mucho este tipo de código recientemente:

if ($foo = $bar->getFoo()) 
{ 
    baz($foo); 
} 

¿Esto se considera buena o mala práctica?

Por ejemplo, el IDE Netbeans dar un aviso si se utiliza este tipo de código:

Posible asignación accidental, asignaciones en condiciones deben ser evitados

¿Qué opinas?

Answer 1

Es una herramienta útil que tengo que admitir el uso de vez en cuando para evitar una línea adicional para una tarea. Por un lado, puede ser una mala práctica por algunos porque:

• No es un idioma disponible en otros idiomas comunes
• Es menos legible

Por otro lado:

• La conversión boleana implícita no ocurre en otros idiomas, pero se cuenta ampliamente donde existen. Por el contrario, existen operadores de asignación condicional en Ruby y Javascript (como ejemplos), pero no en PHP. ¿Deberíamos limitar nuestro uso de construcciones idiomáticas solo a aquellas que se encuentran en todos los idiomas similares? Probablemente no.
• ¿Menos legible a quién?

Debo notar que trato de evitarlo porque me resulta menos legible la mayor parte del tiempo, pero es una preferencia puramente personal para mí. Donde lo encuentro útil, lo uso.

Answer 2

Es una forma fácil de colar los errores, pero es una práctica común en PHP. Especialmente durante cosas como recorridos de directorios en los que hace cosas como while (($dir = readdir($handle)) !== FALSE)

Si puede evitarlo. Evítalo.

Answer 3

Si bien esta es una sintaxis válida y los resultados serán los esperados, es un mal hábito. Falta legibilidad, existe el potencial de desarrollar un mal hábito de put = cuando quisiste decir ==, y tus ojos seguirán volviendo a esta línea cuando trates de encontrar errores reales dentro de una aplicación. No usaría este estilo de escritura. En este caso, solo obtenga el valor de retorno y luego verifique el valor de retorno ... o mejor aún, use el manejo de excepciones para evitar atascarse con muchas sentencias if.

Answer 4

Utilizo esta sintaxis todo el tiempo ... Me parece 100% legible, porque estoy tan acostumbrado a ver este tipo de línea. Tener una línea de código adicional para la tarea me parece una pérdida de espacio.