PROBLEMA
Uno de los problemas que veo en tu implementación es que asumes muy pronto que el primer elemento sea el menor y el segundo el mayor:
var menor = numeros[0];
var mayor = numeros[1];
Otro problema es que tienes una sentencia return
dentro de tu bucle for
:
for(var i = 1; i < numeros.length; i++) {
if( ...) { ... }
else { ... }
return ... // <- esto finaliza la función justo en la primera iteración.
}
SOLUCIÓN
Primero vamos a corregir la asunción errónea de que el primer elemento es el menor y el segundo es el mayor. Dado que no sabes a ciencia cierta qué número será mayor y qué número será menor.
Por lo tanto, puedes decir que el primer elemento es el mayor y a la vez es el menor, y al recorrer el Array
actualizarás según sea necesario.
Por ejemplo:
function mayorMenor(numeros) {
var menor = numeros[0];
var mayor = numeros[0];
...
}
Ahora podemos recorrer el Array
desde el primer elemento, tal como lo estabas intentando hacer, pero evitaremos usar la sentencia return
dentro del ciclo for
:
function mayorMenor(numeros) {
var menor = numeros[0];
var mayor = numeros[0];
for (var i = 1; i < numeros.length; i++) {
if (numeros[i] > mayor) {
mayor = numeros[i];
}
if(numeros[i] < menor) {
menor = numeros[i];
}
}
return [menor, mayor];
}
Como puedes observar, no estoy usando una estructura if ... else
, ya que cuando esté recorriendo el Array
sólo me interesa actualizar el caso cuando el valor sea mayor o cuando sea menor, y por lo tanto si no se cumple una condición se verificará la siguiente. ¿Sabes que pasa si no se cumple ninguna de las 2 condiciones?
Al final el resultado será el siguiente:
function mayorMenor(numeros) {
var menor = numeros[0];
var mayor = numeros[0];
for (var i = 1; i < numeros.length; i++) {
if (numeros[i] > mayor) {
mayor = numeros[i];
}
if(numeros[i] < menor) {
menor = numeros[i];
}
}
return [menor, mayor];
}
let listaDeNumeros = [
[9, 17, 6, 2, 4],
[17, 9, 6, 2, 4],
[1],
[10, 0],
[0, 0, 0, 0, 0]
]
listaDeNumeros.forEach(numeros => {
console.log(mayorMenor(numeros));
});
.as-console-wrapper{
min-height: 100%;
top: 0;
}
Extra
En Javascript disponemos de las funciones Math.min()
y Math.max()
, que devuelven el elemento menor y mayor de una lista de números. Supongo que no te han permitido usarlas, pero no está demás conocer su existencia. El código podría quedar simplificado así:
function mayorMenor(numeros) {
return [Math.min(...numeros), Math.max(...numeros)];
}
let listaDeNumeros = [
[9, 17, 6, 2, 4],
[17, 9, 6, 2, 4],
[1],
[10, 0],
[0, 0, 0, 0, 0]
]
listaDeNumeros.forEach(numeros => {
console.log(mayorMenor(numeros));
});
.as-console-wrapper{
min-height: 100%;
top: 0;
}
En el código anterior he utilizado la Sintáxis Spread, para convertir el Array
de números en una lista de argumentos que son pasados a las funciones min()
y max()
:
Math.min(...numeros)
Espero que esto te ayude a resolver el problema.
for
termina en la primera iteración, ya que tienes una sentenciareturn
dentro del mismo. Eso como primera observación.