Skip to main content
adicionou 67 caracteres ao conteúdo
Fonte Link
Sam
  • 80,8mil
  • 22
  • 73
  • 128

Além do problema em usar o operador || (or) em vez do && (and), onde o && significa que ambas as condições devem ser true, neste caso seria melhor usar um if else e um else (caso não queira usar Math.max() e manter a estrutura de if's), pois não é necessário passar por cada if se você quer que apenas 1 conjunto de condições seja satisfeita.

Além do problema em usar o operador || (or) em vez do && (and), onde o && significa que ambas as condições devem ser true, neste caso seria melhor usar um if else e um else, pois não é necessário passar por cada if se você quer que apenas 1 conjunto de condições seja satisfeita.

Além do problema em usar o operador || (or) em vez do && (and), onde o && significa que ambas as condições devem ser true, neste caso seria melhor usar um if else e um else (caso não queira usar Math.max() e manter a estrutura de if's), pois não é necessário passar por cada if se você quer que apenas 1 conjunto de condições seja satisfeita.

Fonte Link
Sam
  • 80,8mil
  • 22
  • 73
  • 128

Além do problema em usar o operador || (or) em vez do && (and), onde o && significa que ambas as condições devem ser true, neste caso seria melhor usar um if else e um else, pois não é necessário passar por cada if se você quer que apenas 1 conjunto de condições seja satisfeita.

Por exemplo, no primeiro if você já "mata" a questão verificando se o valor1 é maior que os outros dois valores:

if(valor1 >= valor2 && valor1 >= valor3){
  alert("O valor maior é "+valor1)
}else if(valor2 >= valor3){
  alert("O valor maior é "+valor2)
}else{
  alert("O valor maior é "+valor3)
}

Se valor1 for maior ou igual a valor2 e valor3 a verificação irá ignorar os dois outros blocos (else if e else) já encerrando a verificação nas primeiras condições.

Se valor1 for menor que valor2 e maior ou igual a valor3, não satisfará o primeiro if passando pro else if, logo, já é descartado que valor1 é o maior número, então, se valor2 for maior ou igual a valor3, a verificação acaba aí, sendo valor2 o maior número se ele for maior que valor3 ou então, se for igual a valor3, ambos são os valores maiores. Agora, se valor2 for menor que valor3, então irá cair no else porque nenhum dos dois if's foram atendidos, e com isso valor3 é o maior número ou valor1 e valor3 são iguais e são os maiores números.

Testes de mesa:

valor1 = 1
valor2 = 2
valor3 = 3

     if: condição 1 = falso, porque valor1 não é igual nem maior que valor2
                      (nem precisa verificar a segunda condição)
else if: falso, porque 2 não é maior nem igual a valor3, logo, obrigatoriamente
         irá cair no else

> resultado: número maior = 3

------------------------------

valor1 = 3
valor2 = 2
valor3 = 1

     if: condição 1 = verdadeiro, porque valor1 é maior ou igual que valor2
         condição 2 = verdadeiro, porque valor1 é maior ou igual que valor3
else if: ignorado
   else: ignorado

> resultado: número maior = 3

------------------------------

valor1 = 1
valor2 = 2
valor3 = 1

     if: condição 1 = falso, porque valor1 é menor que valor2
                      (nem precisa verificar a segunda condição)
else if: verdadeiro, porque valor2 é maior que valor3
   else: ignorado

> resultado: número maior = 2

------------------------------

valor1 = 1
valor2 = 1
valor3 = 1

     if: condição 1 = verdadeiro, porque valor1 é igual a valor2
         condição 2 = verdadeiro, porque valor1 é igual a valor3
else if: ignorado
   else: ignorado

> resultado: número maior = 1

------------------------------

valor1 = 1
valor2 = 2
valor3 = 2

     if: condição 1 = falso, porque valor1 é menor que valor2
         (nem precisa verificar a segunda condição)
else if: verdadeiro, porque valor2 é igual a valor3
   else: ignorado

> resultado: número maior = 2

Teste:

var valor1 =Number(prompt("Digite hum numero: "))
var valor2 =Number(prompt("Digite hum numero: "))
var valor3 =Number(prompt("Digite hum numero: "))

if(valor1 >= valor2 && valor1 >= valor3){
  alert("O valor maior é "+valor1)
}else if(valor2 >= valor3){
  alert("O valor maior é "+valor2)
}else{
  alert("O valor maior é "+valor3)
}