Estas son las condiciones que aplicó, el puntaje actual debe ser mayor que el max y menor que 20 .

 if (score[i] > max && score[i] <= 20) { max = score[i]; }

Este es un ejemplo de la cuarta primera iteración.

• Iteración 1: input: 50 > max: 50
• Iteración 2: input: 70 > max: 50
• Iteración 3: input: 60 > max = 50
• Iteración 4: input: 20 > max = 50

En la cuarta iteración, el max no se asigna como 20 porque no es mayor que el máximo actual , que es 50.

Aquí está la solución simple:

1. Añadir score.sort(); para ordenar la matriz antes de comenzar a hacer un bucle.
2. Defina el min con el último índice de la matriz.

 'use strict'; let scors = [50, 70, 60, 20, 15, 14, 12, 10, 9, 8, -5, 4, 16, 2, 17.5, 12.75]; function minMax(score) { score.sort(); let max = score[0]; let min = score[score.length - 1]; for (let i = 0; i < score.length; i++) { if (score[i] < min && score[i] >= 0) { min = score[i]; } // This part now works! if (score[i] > max && score[i] <= 20) { max = score[i]; } } console.log(max); console.log(min); } minMax(scors);

Para el rango de 0 a 20, puede limitar el mínimo y el máximo, lo que eliminaría la necesidad de ordenar la matriz.

Complejidad temporal O(N).

 'use strict'; let scors = [50, 70, 60, 20, 15, 14, 12, 10, 9, 8, -5, 4, 16, 2, 17.5, 12.75]; function minMax(score) { let max = -1; let min = 21; for (let i = 0; i < score.length; i++) { if (score[i] < min && score[i] >= 0) { min = score[i]; } // This part now works! if (score[i] > max && score[i] <= 20) { max = score[i]; } } console.log(max); console.log(min); } minMax(scors);