Fico sto sito! Mi è comparsa la notifica oggi su Twitter e sono venuto a vedere :)

In realtà mi stimola più che altro l'aspetto di problem solving più che la "pulizia" del codice. Però grazie per le osservazioni ;) in realtà cerco sempre di rubare con gli occhi e di imparare cose nuove.

Hola ciccio! Ultimamente sto studiando a fondo la teoria del JS quindi ti faccio 3 osservazioni così faccio lo sborone e ti insegno qualcosa che ho imparato :D
1 - dovresti imparare a dichiarare le variabili anche nel FOR LOOP per una questione di scope della variabile, var i=0, o meglio ancora let i=0;
2 - al posto di quegli IF statements annidati ne avrei fatto uno solo con operatore && con le due condizioni insieme.
3 - la variabile "undefined" è vuota proprio come "null" ma c'è una differenza di significato del termine. Undefined è una parola che ci dice in fase di debugging che la variabile è stata dichiarata ma mai assegnata, (è lo stato in cui sono le variabili in automatico quando le crei) quindi per convenzione non si assegna mai come hai fatto tu max = undefined. Per ottenere come risultato che "max" restituisca undefined dovresti dichiararla e poi assegnargli un valore solo nel momento in cui ne trovi uno idoneo. ;)