Ciao
No, il tuo codice aveva un errore di fondo:
- Il browser caricava la pagina
- Appena ha caricato la pagina (o mentre la sta caricando) prende i valori dei campi e li salva nelle variabili... Lo fa qui, perchè esterni all'evento onClick
- Quando si scatena l'envento onClick, valuta la variabile che ha estratto subito dopo il caricamento della pagina, quindi prima che l'utente inserisca i dati
Nel codice che ti hanno postato come risposta, invece, i valori vengono presi durante l'evento "onClick", quindi dopo che l'utente li ha inseriti.
Poi, giustamente, fanno il parse per convertire la stringa (valore estratto) in numero (valore da usare per il confronto)