Teste o que é mais fácil primeiro, o que é valor único, então faça isso com o 1 e o 20, depois o que está dentro dessa faixa é o que sobrou, não precisa testar explicitamente, ele é o exceto, então ele já cai no else
se não for qualquer um dos números que testou diferente.
E tome o cuidado para todas condições estarem vinculadas, o else if
serve para isso, a execução vai cair apenas em um dos if
ou else
, é uma seleção do mais adequado, o primeiro que ele entrar nem tenta outros, o else
sozinho garante que entrará ali se não entrar em nenhum if
anterior. Ele é como se fosse um else if (true)
.
Precisa ver se não não pode dar outro valores, eu imagino que não, já que o código indicava isso.
var roll1 = 3;
if (roll1 == 1) {
console.log(`DESASTRE`);
} else if (roll1 == 20) {
console.log(`NORMAL`);
} else {
console.log(`FALHOU`);
}
Coloquei no GitHub para referência futura.
Se puder ter outros valores aí tem que mudar:
var roll1 = 3;
if (roll1 == 1) {
console.log(`DESASTRE`);
} else if (roll1 == 20) {
console.log(`NORMAL`);
} else if (roll1 > 1 && roll1 < 20) {
console.log(`FALHOU`);
} else {
console.log(`Deu número inválido`);
}