3

La función llamada 'menorMayor' recibe como argumento un arreglo de números llamado 'numeros' y debe devolver un arreglo que contenga el menor número del arreglo 'numeros' en la posición cero y el mayor número del arreglo 'numeros' en la posición 1. Ej: menorMayor([4, 6, 1, 7, 15]) debe retornar [1, 15] ya que 1 es el número más chico (menor) dentro del arreglo [4, 6, 1, 7, 15] y 15 es el número más grande (mayor) dentro del arreglo [4, 6, 1, 7, 15]

Esto es lo que hice, no sé que estoy haciendo mal

function menorMayor(numeros) {
  let min = numeros[0];

  for (var i = 0; i < numeros.length; i++) {
    if (numeros[i] < min) {
      min = numeros[i];
    }
  }
  let max = numeros[0];
  for (var i = 0; i < numeros.length; i++) {
    if (numeros[i] > max) {
      max = numeros[i];
    }
  }
  return max, min;
}

console.log(menorMayor([4, 6, 1, 7, 15]));

7
  • 2
    No sé qué es lo que está fallando, pero lo que devuelves no es un array. Para que sea un array deberías poner return [min, max];
    – Sokkian
    Commented el 9 abr. 2021 a las 17:04
  • 1
    Como añadido a lo que dijo @Sokkian no puedes retornar dos valores, un return te permite solo retornar 1
    – Layer
    Commented el 9 abr. 2021 a las 17:05
  • Obviamente ese return tira un error no? que error tira?
    – gbianchi
    Commented el 9 abr. 2021 a las 17:06
  • No da error, solo no ejecuta el código :c Commented el 9 abr. 2021 a las 17:15
  • 1
    @gbianchi Investigando descubrí que la coma por si sola es un operador y resuelve siempre el último elemento. Entonces no hay error. The more you learn
    – FranAcuna
    Commented el 9 abr. 2021 a las 17:15

2 respuestas 2

6

Como ya te dijo @Riven en comentarios, no puedes devolver 2 valores y return max, min; podría generar error de sintaxis devuelve solo el último valor (gracias por la aclaración @FranAcuna).

Debes corregir encerrando ambos valores entre corchetes para que sea un arreglo válido return [max, min];. Pero no están en el orden correcto de acuerdo al enunciado:

debe devolver un arreglo que contenga el menor número del arreglo 'numeros' en la posición cero y el mayor número del arreglo 'numeros' en la posición 1

No es necesario usar ciclos para obtener los elementos, puedes hacer uso de funciones matemáticas como Math.min() para obtener el menor y Math.max() para el máximo.

El "único detalle" es que no puedes enviar el arreglo numeros como parámetro, tienes que desestructurarlo para que todos los elementos se tomen como parámetros individuales.

function menorMayor(numeros) {
    return [Math.min(...numeros), Math.max(...numeros)];
}

console.log(menorMayor([4, 6, 1, 7, 15]));

Importante: La desestructuración es una característica de introducida a partir de ECMAScript 6. No olvides consultar la tabla de compatibilidad para saber en qué navegadores está disponible.

6
  • Dios, nunca pensé en la posibilidad de hacerlo sin un loop, muchas gracias! Commented el 9 abr. 2021 a las 17:21
  • 1
    @SergioJáureguiÁngel internamente, el lenguaje esta haciendo un loop.. solo que vos no lo escribiste.. llamaste a una funcion que lo hace.. esto solo como aclaracion de que el codigo no es magico...
    – gbianchi
    Commented el 9 abr. 2021 a las 17:54
  • @gbianchi, lamento diferir, Javascript no hace un ciclo, simplemente "desempaca los elementos" para tratarlos de forma individual.
    – Triby
    Commented el 9 abr. 2021 a las 17:57
  • 1
    Un extra a la respuesta seria que el eperador spread("...") fue incluido desde ES6 por lo que en versiones anteriores no funcionaria. Ademas, es compatible con todos los navegadores Commented el 9 abr. 2021 a las 17:59
  • @Triby ok te lo tomo.. pero eso es lo que dice que hace por arriba.. no dice como hace eso ;)
    – gbianchi
    Commented el 9 abr. 2021 a las 18:01
2

Considero que la respuesta de @Triby es la más acertada ya que explica los errores cometidos y simplifica bastante el código.

Ahora!, otra forma de obtener el resultado es ordenando los número en el arreglo de menor a mayor.

Ejemplo:

function menorMayor(numeros) {
  let ordenados = numeros.sort((a, b) => a - b);
  return [ordenados[0], ordenados[numeros.length - 1]];
}

console.log(menorMayor([4, 6, 1, 7, 15]));


Referencias:

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.