form con diversi select obbligatori

[lilo]

Ho un lungo form con diversi dati obbligatori, tra cui i seguenti due select:

if(ordini.categoria.selectedIndex > 0) {
            return true;
} else {
    alert('categoria è un campo obbligatorio');
        return false;
}
if(ordini.localita.selectedIndex > 0) {
        return true;
} else {
        alert('località è un campo obbligatorio');
        return false;
} 

Ho già utilizzati questo codice altre volte, nn capisco dove sia l'errore.

Posto anche il codice relativo al form:

 <FORM method="post" name="ordini" onsubmit="return checkForm(this)"> 

da cosa può dipendere?

Grazie
Lilo

[cavicchiandrea]

Prova cosi
document.NomeForm.NomeOggetto.options.selectedInde x
in alcuni browser riferisi al form potrebbe non andare oppure usando DOM
document.getElementById("Id_Oggetto").options.sele ctedIndex

[lilo]

in pratica diverrebbe cosi?

if(ordini.categoria.options.selectedIndex > 0) {
            return true;
} else {
    alert('categoria è un campo obbligatorio');
        return false;
} 

in quanto ho provato, ma nn funziona ancora.

[cavicchiandrea]

if(document.ordini.categoria.options.selectedIndex > 0) {

[lilo]

Salve,

in un'altro form online, mi da lo stesso problema, funziona sempre e solo il primo controllo.

if(document.ordini.provincia.selectedIndex > 0) {
        // an option has been selected
        return true;
} else {
         // no option selected
        alert('provincia è un campo obbligatorio');
        return false;
}
if(document.ordini.comparto.options.selectedIndex > 0)  {
        // an option has been selected
        return true;
} else {
         // no option selected
        alert('ente è un campo obbligatorio');
        return false;
} 

C'è qualcosa di errato nel codice??

[fcaldera]

non c'è nulla di errato.
Entrando nel primo ramo del primo if fai ritornare la funzione con il return. la funzione ritorna appena trova un campo corretto

dovresti scrivere invece

if (!document.ordini.provincia.selectedIndex > 0) {
         // no option selected
        alert('provincia è un campo obbligatorio');
        return false;
}

if (!document.ordini.comparto.selectedIndex > 0) {
         // no option selected
        alert('ente è un campo obbligatorio');
        return false;
}

oppure

var errore = '';
if (!document.ordini.provincia.selectedIndex > 0) errore += 'provincia è un campo obbligatorio\n'
if (!document.ordini.comparto.selectedIndex > 0)  errore += 'ente è un campo obbligatorio\n';

if (errore !== '') {
  alert(errore); return false;
}

[lilo]

Ti ringrazio tantissimo, ho provato il primo metodo e funziona tutto!!

grazie ancora
Lilo