Javascript Problema com if e || no JS

[Natã Andrade]

Nesse pen (linha 5 do JS) o ALERT deveria aparecer se a pessoa digitasse um numero menor que 0 ou maior 10, mas não está dando certo.

 

function raffle()
{ 
  unknown = Math.round(Math.random()*10)
  number = prompt('Choose a integer number between 0 and 10');
  if (number < 0 || number > 10){alert('The number got to be between 0 and 10'),location.reload();}
  other = number == unknown;
  if (other){ paragrafoResultado.innerHTML = 'You won'; }
  if (!other){ paragrafoResultado.innerHTML = 'You lost'; }
}
paragrafoResultado.style.backgroundColor='yellow'

 

Link do Pen: https://codepen.io/Arckman/pen/bGeNwoV?editors=0010

[CyberPh4ntom]

Retirei o unknow que estava sendo passado na variavel number e voltou a funcionar

 

function raffle()
{
 unknown=Math.round(Math.random()*10)  
 number=prompt('Choose a integer number between 0 and 10')
 if(number < 0 || number > 10 ){alert('The number got to be between 0 and 10')}  
 if(number){paragrafoResultado.innerHTML='You won'}
  else{paragrafoResultado.innerHTML='You lost'}
}

paragrafoResultado.style.backgroundColor='yellow'

 

[Simon Viegas]

Em 13/10/2020 às 11:39, Arckman disse:

mas não está dando certo

 

Mas o que está ocorrendo de errado?

 

Digitei 20 e funcionou:

 

 

Sobre:

10 horas atrás, CyberPh4ntom disse:

Retirei o unknow que estava sendo passado na variavel number e voltou a funcionar

 

Essa variável não está "sendo passada", mas sim "comparada". Ela contém o valor "sorteado".

 

Sobre:

number == unknown

 

Seria uma expressão lógica... está comparando um valor com o outro, daí isso resulta em true ou false. Os ifs seguintes estão verificando o seu valor, ou seja: se o número informado é igual ao número sorteado ou não.

 

 

Poderia utilizar um operador ternário, exemplo:

function raffle()
{ 
  unknown = Math.round(Math.random()*10)
  number = prompt('Choose a integer number between 0 and 10');
  if (number < 0 || number > 10){alert('The number got to be between 0 and 10'),location.reload();}
  other = number == unknown;
  other ? paragrafoResultado.innerHTML = 'You won' : paragrafoResultado.innerHTML = 'You lost';
}
paragrafoResultado.style.backgroundColor='yellow'

 

Ou ainda:

function raffle()
{ 
  unknown = Math.round(Math.random()*10)
  console.log(unknown)
  number = prompt('Choose a integer number between 0 and 10');
  if (number < 0 || number > 10){alert('The number got to be between 0 and 10'),location.reload();}
  number == unknown ? paragrafoResultado.innerHTML = 'You won' : paragrafoResultado.innerHTML = 'You lost';
}
paragrafoResultado.style.backgroundColor='yellow'

 

Por aí vai.

 

Ah! Para testar o vencedor, seria bom verificar o valor sorteado... daí poderia utilizar um alert para exibir o valor ou um console.log.