Expresiones en JavaScript ternario del operador y de JSLint

Question

poco recibí un comentario en uno de mis blogs sobre JSLint preguntando por qué JSLint arrojó un error con el siguiente: generada

s === "test" ? MyFunc() : MyFunc2(); 

El error fue:

"Esperaba una asignación o función llamada y en su lugar vio una expresión."

Claramente JSLint está esperando una misión aquí, somthing más como:

var y = (s === "test") ? MyFunc() : MyFunc2(); 

Pero, yo no veo el problema con el primer ejemplo. ¿Es realmente el caso que los operadores ternarios solo deberían usarse para asignaciones?

No pude ver nada en JSLint.com, ni aparecía nada en el libro JavaScript: The Good Parts. Y, el mismo error también se informa en el fork de la comunidad JSHint.

¿Alguien?

Answer 1

Es una expresión. Es equivalente a la escritura

0 === 1;

Estás escribiendo una expresión que tiene efectos secundarios inmediatos y que es considerado malo.

Generalmente, las expresiones son declaraciones inútiles que no tienen ningún efecto secundario. Se considera una mejor forma simplemente hacer

if (s === "test") { 
    MyFunc(); 
} else { 
    MyFunc2(); 
} 

Aparte de eso, es una sintaxis perfectamente sólida. Personalmente, estoy de acuerdo en que escribir un ternario breve como alternativa a un if es malo y es mejor que solo lo utilices para la tarea.

Otros expresión abreviada que han sido (ab) usado para escueto-dad

someCondition && doMagic(magic); 
someCondition || doMagic(magic); 

Una vez más éstos formen malos son considerados si hay utilizarse sólo como expresiones porque el uso de estos sólo oscurece la lógica de distancia y hacer que sea más difícil mantener el código.

JSHint tiene una opción expr para esto. Ver ticket

de reproducción:

/*jshint 
    expr: true 
*/ 

var s, MyFunc, MyFunc2; 
s === "test" ? MyFunc() : MyFunc2(); 
0 === 1; 

pasará