JavaScript: Un refactor a un problema de Array

Encontré un ejercicio en la ruta de entrenamiento de JavaScript de freecodecamp. Aquí les dejo el enlace: freecodecamp.org/learn/javascript-algorithm...

Básicamente pide que a partir de un array formado por otros array se genere un array resultante en base al número mayor de cada array. De la siguiente forma:

1// El problema
2[
3  [4, 5, 1, 3],// Extraer el mayor 5
4  [13, 27, 18, 26],// Extraer el mayor 27
5  [32, 35, 37, 39],// Extraer el mayor 39
6  [1000, 1001, 857, 1]// Extraer el mayor 1001
7]
8

1// El resultado
2[5, 27, 39, 1001]// Array formado por los números mayores

Existen varios caminos para resolver el problema, sin embargo mi propósito es explicar cómo a partir de una solución inicial se puede llegar a optimizar el código.

Solución 1:

Vemos que es un array de array, la idea inicial es hacer una primera iteración para recorrer el array general y luego realizar una segunda iteración para el array interior. Esto es algo que no haría pero ¿es legible leer el código?.

1functionlargestOfFour(arr) {
2let result = [];
3for (let i = 0; i < arr.length; i++) {
4let mayor = arr[i][0]// Toma el primer valor de arrayfor (let j = 0; j < arr[i].length; j++) {
5if (arr[i][j] > mayor) {// Compara si existe un número mayor que el primero
6        mayor = arr[i][j];
7      }
8    }
9    result.push(mayor);
10  }
11return result;
12}
13

Solución 2: El refactor

Para obtener el número mayor o menor existe una función matemática en JavaScript. Math.max o Math.min nos permite obtener el número máximo o mínimo respectivamente. Veamos como queda:

1functionlargestOfFour(arr) {
2let result = [];
3for (let i = 0; i < arr.length; i++) {
4let mayor =Math.max(...arr[i])// Buscamos el valor max en el array
5    result.push(mayor);
6  }
7return result;
8}
9

¿Qué tal? ¿mejor, verdad?. Logramos quitar la iteración interna y el código es mas legible. Personalmente usar un for of me resulta más limpio y evita calcular arr.length en la iteración. Reemplazamos for por for of y quitamos la variable mayor ya no es útil.

1functionlargestOfFour(arr) {
2let result = [];
3for (let itemof arr) {
4    result.push(Math.max(...item));
5  }
6return result;
7}
8

Solución 3: El refactor del refactor

Ok, vamos un paso más. Resulta que en lugar de usar for o for of podemos usar map para ir recorriendo los elementos. Con ello evitamos usar la variable result que lo único que hace es almacenar la respuesta. Por defecto map retorna un array de elementos luego de transformar los elementos de la iteración.

1functionlargestOfFour(arr) {
2return arr.map(item =>Math.max(...item));
3}
4