scusa se te lo dico, ma non hai molto chiaro il funzionamento dei radio...
Ma la colpa non è tua, ma di quella cavolo di sintassi ridotta di Javascript.
Se usi le funzioni DOM è molto più semplice gestire gli elementi:
Cambia la funzione con questa
Codice PHP:
<!--
function Email() {
//La funzione getElementsByName ritorna un array di elementi, ma dato che ve ne sono uno solo posso indicare di passare subito il primo (con [0])
var dest_email = document.getElementsByName('dest_email')[0].value;
var mitt_nome = document.getElementsByName('mitt_nome')[0].value;
var mitt_email = document.getElementsByName('mitt_email')[0].value;
var mitt_tel = document.getElementsByName('mitt_tel')[0].value;
var oggetto = document.getElementsByName('oggetto')[0].value;
var messaggio = document.getElementsByName('msg')[0].value;
var mitt_sesso, mitt_eta;
//Qui invece tratti gli elementi radio, dato che sono più di uno devo leggere l'intero array
var radio_sesso = document.getElementsByName('mitt_sesso');
var radio_eta = document.getElementsByName('mitt_età');
//Ti consiglio di non usare mai i caratteri speciali nel codice JS
for(i=0; i < radio_sesso.length; i++){
if(radio_sesso[i].checked)
{
//Se il radio è selezionato
mitt_sesso = radio_sesso[i].value;
}
}
for(i=0; i < radio_eta.length; i++){
if(radio_eta[i].checked)
{
//Se il radio è selezionato
mitt_eta = radio_eta[i].value;
}
}
if ((mitt_nome == "") || (mitt_nome == "undefined")) {
alert("Inserire nome.");
document.modulo.mitt_nome.focus();
}
else if (!mitt_sesso) {
alert("Specificare il sesso.");
}
else if (!mitt_eta) {
alert("Specificare l'età.");
}
else if ((mitt_email.indexOf("@") == (-1)) || (mitt_email == "") || (mitt_email == "undefined")) {
alert("Inserire un indirizzo email valido.");
document.modulo.mitt_email.focus();
}
else if ((mitt_tel == "") || (mitt_tel == "undefined")) {
alert("Inserire numero di telefono.");
document.modulo.mitt_tel.focus();
}
else if ((oggetto == "") || (oggetto == "undefined")) {
alert("Inserire un oggetto.");
document.modulo.oggetto.focus();
}
else if ((messaggio == "") || (messaggio == "undefined")) {
alert("Inserire un messaggio.");
document.modulo.messaggio.focus();
}
else {
a = "mailto:" + dest_email
+ "?Subject=" + oggetto
+ "&Body=Messaggio: " + messaggio
+ "%0aMittente: " + mitt_nome
+ "%0aSesso: " + mitt_sesso
+ "%0aEta': " + mitt_eta
+ "%0atel: " + mitt_tel;
}
}
//-->
Io ti raccomando di non utilizzare mai i caratteri speciali con JS, perché fa veramente fatica a gestirli.
Comunque il focus sull'elemento Radio non ha molto senso, se non l'ha selezionato...
Se hai perplessità sul codice chiedi pure
Rispondi quotando
