script non funzionante

[theduke]

ho inserito in una pagina jsp uno script javascript che dovrebbe controllare la data inserita e un intero che dovrebbe essere compreso tra 18 e 30, il problema è che quando la pagina viene eseguita il controllo viene fatto solo sulla data e nn sull'intero. Posto il codice così potete dirmi dove sbaglio, premetto che sono agli inizi con javascript, abbiate pietà

function verifica(modulo){
   giorno=modulo.giorno.value;
   mese=modulo.mese.value;
   voto=modulo.voto.value;
   errore=false;
   if((giorno<1) || (giorno>31)) errore=true;
   if((mese<1) || (mese>12)) errore=true;
   if((voto<18) || (voto>30)) errore=true;
   if(errore==true){
       alert("dati errati");
       return false;
   }
}

questo codice viene richiamato dalla form in questo modo: <form action="/servlet" onsubmit="return verifica(this);" method="POST">

dove sbaglio?

[atsap]

Non ho ben capito che significa che il controllo viene effettuato sulla data e non sull'intero. Tu intendi singolarmente su ogni intero?

[br1]

I valori inseriti nei campi del form sono sempre e soltanto stringhe di testo... tu stai tentando di confrontarle con numeri quindi...

Fai:
voto=parseInt(modulo.voto.value);

Inoltre non e' buona cosa chiamare le variabili con lo stesso name usato nei campi del form... IE potrebbe fare confusione nell'identificare quale oggetto prendere in considerazione, se la variabile o il campo, se decide per quest'ultimo avresti un ovvio errore in esecuzione. Quindi meglio:
w_voto=parseInt(modulo.voto.value);

ciao

ps: complimenti per il titolo

[theduke]

grazie per l'aiuto e scusate per il titolo.. ho ancora una domanda, perchè con la data funziona? sono sempre numeri..

[br1]

Originariamente inviato da theduke
grazie per l'aiuto e scusate per il titolo.. ho ancora una domanda, perchè con la data funziona? sono sempre numeri..

Sara'... a me non da' errore se metto come mese il 11111 che alfabeticamente e' minore di "12"

[theduke]

a me invece la data se metto mese 16 si arrabbia.. in ogni caso anche con il parseInt continua ad ignorare il voto, lo salta proprio controlla solo la data

[br1]

Originariamente inviato da theduke
a me invece la data se metto mese 16 si arrabbia.. in ogni caso anche con il parseInt continua ad ignorare il voto, lo salta proprio controlla solo la data

Hai tutti gli elementi e i suggerimenti per risolvere e testare lo script, ma non sei obbligato ad usarli...

Posta tutto il form e lo script come e' attualmente dopo le correzioni, vediamo di darti qualcosa di realmente valido, se vuoi...

[theduke]

ecco qui lo script:

function verifica(modulo){
                w_giorno=parseInt(modulo.giorno.value);
                w_mese=parseInt(modulo.mese.value);
                w_votazione=parseInt(modulo.votazione.value);
                errore=false;
                if ((w_giorno<1) || (w_giorno>31)) errore = true;
                if ((w_mese<1) || (w_mese>12)) errore = true;
                if ((w_votazione<18) || (w_votazione>30)) errore = true;
                if(errore==true){
                    alert("errore nell'inserimento dei dati");
                    return false;
                }
                
            }

e qui c'è il form:

 <form action="/lab_project/servletsweb" onsubmit="return verifica(this);"  method="POST">
            

Matricola: <jsp:getProperty name="docente" property="studenteiscr"/></p>
            

Corso: <jsp:getProperty name="docente" property="appelli_fiss"/></p>
            

Appello del: <input type="text" name="giorno" value="" size="3" />/<input type="text" name="mese" value="" size="3" /> (dd/mm)</p>
            

Votazione: <input type="text" name="votazione" value="" size="3" />/30</p>
            

<input type="submit" value="assegna voto" name="votazione" /></p>
        </form>

[br1]

<input type="submit" value="assegna voto" name="votazione" />

ciao