Skip to main content
3 de 3
adicionou 277 caracteres ao conteúdo
Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil

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`);
}

Coloquei no GitHub para referência futura.

Maniero
  • 484,5mil
  • 94
  • 1,3mil
  • 2,2mil