Da JS a ECMAscript corretto

[coniglio]

Ciao a tutti.
Prendiamo come esempio questo datatissimo JS che ho utilizzato per fare il controllo sui campi presente nella pagina
http://www.mastra-srl.com/contatti.asp

eccolo

<script type="text/javascript">
// This Part Checks the Checkout Form Fields;

function CheckForm(){;
var okay = true;
var atsign=document.form2email.email.value;
if ((atsign.indexOf('@') == -1) && (document.form2email.telefono.value=='Telefono')){;
// alert (atsign + ' Non è un indirizzo di e-mail valido!');
 alert ('Per permetterci di contattarla deve inserire il suo indirizzo email oppure il telefono');
 okay = false;
 document.form2email.email.focus();
};
if ((document.form2email.accettazione.checked==false) && (okay == true) ) {
	    alert("Acconsentire al trattamento dei dati");
	    okay = false;
document.form2email.accettazione.focus();
};
if ((document.form2email.nome.value=='Nome') && (okay == true)){;
 alert ('Inserire il Nome di riferimento prima di inviare');
 okay = false;
 document.form2email.nome.focus();
};
return okay;
};
</script>

Funziona ma non è nulla di valido per il W3C logicamente.
Come lo dovrei modificare per renderlo valido?(a parte le lettere maiuscole)
Grazie

[cavicchiandrea]

Originariamente inviato da coniglio
Ciao a tutti.
Prendiamo come esempio questo datatissimo JS che ho utilizzato per fare il controllo sui campi presente nella pagina
http://www.mastra-srl.com/contatti.asp

eccolo

<script type="text/javascript">
// This Part Checks the Checkout Form Fields;

function CheckForm(){;
var okay = true;
var atsign=document.form2email.email.value;
if ((atsign.indexOf('@') == -1) && (document.form2email.telefono.value=='Telefono')){;
// alert (atsign + ' Non è un indirizzo di e-mail valido!');
 alert ('Per permetterci di contattarla deve inserire il suo indirizzo email oppure il telefono');
 okay = false;
 document.form2email.email.focus();
};
if ((document.form2email.accettazione.checked==false) && (okay == true) ) {
	    alert("Acconsentire al trattamento dei dati");
	    okay = false;
document.form2email.accettazione.focus();
};
if ((document.form2email.nome.value=='Nome') && (okay == true)){;
 alert ('Inserire il Nome di riferimento prima di inviare');
 okay = false;
 document.form2email.nome.focus();
};
return okay;
};
</script>

Funziona ma non è nulla di valido per il W3C logicamente.
Come lo dovrei modificare per renderlo valido?(a parte le lettere maiuscole)
Grazie

prova ad cambiare questa riga

<script type="text/javascript">

con questa:

<script language="javascript" type="text/javascript">

X lo script prova così:

function CheckForm(){
var atsign=document.form2email.email.value;
if (atsign.indexOf('@') == -1 && document.form2email.telefono.value=='Telefono'){
// alert (atsign + ' Non è un indirizzo di e-mail valido!');
alert ('Per permetterci di contattarla deve inserire il suo indirizzo email oppure il telefono');
document.form2email.email.focus();
return false;
}
if (!document.form2email.accettazione.checked){
alert("Acconsentire al trattamento dei dati");
document.form2email.accettazione.focus();
return false;
}
if (document.form2email.nome.value=='Nome') {
 alert ('Inserire il Nome di riferimento prima di inviare');
 document.form2email.nome.focus();
return false;
}
}
</script>

[Mich_]

Prima di modificare il JS, occorre mettere a posto il codice HTML.

Se vuoi passare a XHTML, non puoi usare l'attributo name in molti tag, che va sostituito con id

A quel punto il
document.NOMEFORM
va sostituito con
document.getElementById('ID_FORM').

Invece all'interno del form (nei campi) il name e` valido.

In XHTML non esiste l'attributo language: va eliminato e lasciato solo l'attributo type

[coniglio]

Originariamente inviato da Mich_
Prima di modificare il JS, occorre mettere a posto il codice HTML.

Se vuoi passare a XHTML, non puoi usare l'attributo name in molti tag, che va sostituito con id

A quel punto il
document.NOMEFORM
va sostituito con
document.getElementById('ID_FORM').

Invece all'interno del form (nei campi) il name e` valido.

In XHTML non esiste l'attributo language: va eliminato e lasciato solo l'attributo type

A ok grazie del consiglio, ma io il name l'ho usato solo nei campi del form....
se non fosse per il JS la pagina è validata, ed anche il resto del sito, compresi gli swf.
Per l'attributo language lo so infatti l'avevo levato.

[coniglio]

Originariamente inviato da cavicchiandrea
X lo script prova così:

Grazie ora provo

[Mich_]

Originariamente inviato da coniglio
ma io il name l'ho usato solo nei campi del form....

Non credo, dato che usavi la sintassi:
document.NOMEFORM.NOMECAMPO
che presuppone che nel tag sia definito:
<form name="NOMEFORM" ...>
oppure che non funzioni in browser diversi da IE.

Nota che in XHTML Strict, lo script va inserito in un file a parte, oppure dentro commenti CDATA

[coniglio]

Si è vero non avevo pensato al name nel tag form.
Strano che il validatore non mi dia errore