3

Por que está errado?

var n1 = parseFloat(prompt("Digite um número:"));
var n2 = parseFloat(prompt("Digite um número:"));
var n3 = parseFloat(prompt("Digite um número:"));
function maiorDosTres(n1, n2, n3) {
  if ( n1 > n2 > n3) {
    alert( "O maior número é: " + n1 + " e o menor é: " + n3);
  } else if ( n1 > n3 > n2) {
    alert( "O maior número é: " + n1 + " e o menor é: " + n2);
  } else if ( n2 > n1 > n3) {
    alert( "O maior número é: " + n2 + " e o menor é: " + n3);
  } else if ( n2 > n3 > n1) {
    alert( "O maior número é: " + n2 + " e o menor é: " + n1);
  } else if ( n3 > n1 > n2) {
    alert( "O maior número é: " + n3 + " e o menor é: " + n2);
  } else if ( n3 > n2 > n1) {
    alert( "O maior número é: " + n3 + " e o menor é: " + n1);
  }
}
maiorDosTres(n1, n2, n3);
5
  • 1
    Já implementaram o Math.max(args) e o Math.min(args) Commented 29/08/2015 às 17:15
  • 1
    Eu não posso usar funções. E o problema é que quando eu executo, em vez de mostrar a string com o maior e menor número, mostra undefined.
    – Bernardo
    Commented 29/08/2015 às 17:19
  • Tem os operadores && e || que podem ajudar vc a resolver isso. Falta algo na suas comparações :P
    – rray
    Commented 29/08/2015 às 17:22
  • Obrigado. Ajudou muito.
    – Bernardo
    Commented 29/08/2015 às 17:27
  • Dê uma olhada no tour. Você pode aceitar uma resposta se ela resolveu seu problema. Você pode votar em todos os posts do site também. Alguma lhe ajudou mais? Precisa que algo seja melhorado?
    – Maniero
    Commented 4/03/2016 às 18:58

6 Respostas 6

11

Está errado porque você não pode comparar mais que dois operandos ao mesmo tempo. Isto não existe:

if ( n1 > n2 > n3) {

O correto seria:

if ( n1 > n2 && n2 > n3) {

Coloquei no GitHub para referência futura.

Desta forma você tem três expressões com dois operandos cada: n1 > n2, n2 > n3 e a operação relacional && que tem como operandos os resultados da expressões anteriores.

O mesmo vale para os demais. Acho que já pegou o jeito.

Tem maneiras melhores de fazer isto. Usar Math.max() e Math.min() seria uma delas. Não vou colocar porque me parece que você está fazendo isto para fins de estudo e não quer a melhor solução.

10

Você pode utilizar as funções Math.max() e Math.min(). As duas funções recebem todos os valores para comparação como parâmetros. Apesar da a pergunta ser o motivo de estar errado, essa forma deixará o seu código mais enxuto.

var n1 = 10;
var n2 = 5;
var n3 = 7;

var max = Math.max(n1, n2, n3);
var min = Math.min(n1, n2, n3);

var pre = document.body.appendChild(document.createElement('pre'));
pre.textContent = 'dos valores ' + n1 + ', ' + n2 + ' e ' + n3;
pre.textContent += ' o máximo é ' + max + ' e o mínimo é ' + min;

Se os dados estiverem em um array utilize apply():

var numeros = [4,6,2,7,9,5];
Math.max.apply(Math, numeros);
1
  • 3
    Math.max() e Math.min() é a maneira semanticamente correta/nativa para comparar valores. +1
    – Sergio
    Commented 31/08/2015 às 15:56
1

Grave os valores em um array e use o "for" para este caso, a função fica mais flexível e serve para qualquer número de argumentos:

var n = [1, 2, 3];


numero=n(i)

for (i = 0; i < n.length; i++) { 
    if (n(i) >= numero) { maior = n(i); }
    if (n(i) <= numero) { menor = n(i); }
}
alert( "O maior número é: " + maior + " e o menor é: " + menor);
1
  • Poderia explicar seu código? Fiquei curioso sobre o funcionamento. Commented 31/08/2015 às 17:34
1

Por que não ordenar os números?

function maiorDosTres() {
    var a = Array.prototype.sort.call(arguments);
    alert( "O maior número é: " + a[a.length - 1] + " e o menor é: " + a[0]);
}

Fiddle

-3

function maxEMim(n1, n2, n3){

var max = Math.max(n1,n2,n3);
var mim = Math.min(n1,n2,n3);

console.log(`Maior: ${max} e o menor: ${mim}`);

}

maxEMim(10,5,0);

-3

let n1 = 1; let n2 = 5; let n3 = 7;

let max = Math.max(n1,n2,n3); let mim = Math.min(n1,n2,n3);

console.log(Maior: ${max} e o menor: ${mim});

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .