Ciao... ti invio qualche annotazione
- verifica il link... fa riferimento ad un file locale:
homepage
- Il controllo sulle password, richiamato in quel punto, non permette la verifica nel caso non venga inserito nemmono un valore, e segnala l'errore prima di introdurre la conferma;
- il loop sui campi del form non tiene conto della natura dei radio: per essi dovrebbe essere diverso - al posto del loop e' preferibile verificare tutti i campi singolarmente;
- e' errato il value dei campi radio: riporterebbero lo stesso valore in entrambe le scelte - non li controllo: comunque uno sara' selezionato;
- ho modificato i nomi dei campi nel form e delle variabili nello script per evitare conflitti e incompatibilita', nota anche cosa usavi nel form e nello script per anni ed email;
- ho preferito eseguire tutti i controlli e visualizzare un unico messaggio... piuttosto che fermarmi alla prima segnalazione e dover controllare gli altri campi in seguito;
- ho semplificato la pagina... ma per sole esigenze di leggibilita' nel forum
ps: benvenuta nel forumcodice:<SCRIPT type=text/javascript> function convalida() { var messaggio = ""; f = document.fmDynamic; // controllo passsword wpassword = f.fpassword.value; wconferma = f.conferma.value; if (wpassword != wconferma || wpassword=="") { messaggio += "Il campo Password ed il campo Conferma Password non corrispondono!\n"; } // controllo eta' wanni=f.anni.value; if (isNaN(wanni) || wanni==""){ messaggio += "Inserisci un'età valida\n"; } // controllo email (un po' troppo spartano... fai una ricerca per reperirne uno migliore) wemail=f.email.value; if (wemail.indexOf("@")==-1) { messaggio += "Inserisci un'email valida\n"; } // cognome e nome pippo=f.cognome.value; if (pippo=="") { messaggio += "Inserisci il cognome\n"; } pippo=f.nome.value; if (pippo=="") { messaggio += "Inserisci il nome\n"; } // finiti i controlli, visualizzo l'eventuale messaggio if (messaggio!="") { alert(messaggio); return false; } } </SCRIPT> ... <FORM name=fmDynamic onsubmit="return convalida();" action=""> nome: <INPUT name=nome> cognome: <INPUT name=cognome> maschio: <INPUT type=radio CHECKED value="M" name=radiobutton> femmina: <INPUT type=radio value="F" name=radiobutton> anni: <INPUT type=txt size=10 name=anni> password: <INPUT type=password name=fpassword> conferma: <INPUT type=password name=conferma> e-mail: <INPUT name=email> <INPUT type=submit value=invia name=Submit> </FORM>![]()


Rispondi quotando