El comportamiento es el esperado, sucede que para cada bloque case
debería usar una sentencia break
que indica la salida o interrupción del switch
, si no lo haces se ejecutará la evaluación del siguiente bloque.
Por ejemplo:
const bad = (value) => {
// switch sin usar break
switch (value) {
case 1:
console.log(`Foo: ${value}`);
case -1:
console.log(`Bar: ${value}`);
default:
console.log(`Baz: ${value}`);
break;
}
};
const values = [0, 1, -1];
values.forEach(v => bad(v));
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Como se observa, al no incorporar o incluir la sentencia break
se ejecuta el siguiente bloque de código independientemente de que no coincida con el caso dado. La sentencia switch
busca el caso y empieza a ejecutar desde el primero que consiga dentro de los case
, si no consigue ninguno, ejecuta la sentencia default
.
Si ponemos la sentencia break
el código se ejecuta correctamente:
const good = (value) => {
// switch usando break
switch (value) {
case 1:
console.log(`Foo: ${value}`);
break;
case -1:
console.log(`Bar: ${value}`);
break;
default:
console.log(`Baz: ${value}`);
break;
}
};
const values = [0, 1, -1];
values.forEach(v => good(v));
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Sin embargo vemos que tanto para el caso 1 y -1, el bloque a ejecutar es idéntico, por lo cual podemos escribir:
const better = (value) => {
// switch sin usar break
switch (value) {
case 1:
case -1:
console.log(`Bar: ${value}`);
break;
default:
console.log(`Baz: ${value}`);
break;
}
};
const values = [0, 1, -1];
values.forEach(v => better(v));
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Y si lo quieres hacer un poco más evidente, entonces lo escribes tal como lo muestras en tu pregunta:
const evenBetter = (value) => {
// switch sin usar break
switch (value) {
case 1: case -1:
console.log(`Bar: ${value}`);
break;
default:
console.log(`Baz: ${value}`);
break;
}
};
const values = [0, 1, -1];
values.forEach(v => evenBetter(v));
.as-console-wrapper {
min-height: 100%;
top: 0;
}
¿Porque case 1 || -1
no funciona?
En primer lugar hay que entender que la sentencia switch ... case
hace una comparación del valor que tenga la variable pasada como argumento a switch
con el valor declarado en case
:
switch(value) {
case 1: // si value === 1
...
break;
case -1: // si value === -1
...
break;
default: // si no es ninguno de los anteriores
...
}
Entonces, usar un case
de este tipo: case 1 || -1
siempre evaluará el caso cuando values es 1, porque el operador ||
es de cortocircuito, y devuelve el elemento de la izquierda si el mismo es evaluado a true
, es decir, si el mismo es truthy
.
Eso quiere decir que case 1 || -1:
siempre será igual a case 1:
y nunca será igual a case -1:
Podemos verlo:
const tooBad = (value) => {
// switch sin usar break
switch (value) {
case 1 || -1:
console.log(`Bar: ${value}`);
break;
default:
console.log(`Baz: ${value}`);
break;
}
};
const values = [0, 1, -1];
values.forEach(v => tooBad(v));
console.log(`1 || -1 => ${1 || -1}`);
.as-console-wrapper {
min-height: 100%;
top: 0;
}
Espero que esto aclare tu duda. Agur!!!