Chi mi spiega pechè qeusto ciclo non viene considerato?

[dops]

ciao, devo controllare che in una textarea ci sia scritto qualcosa e che in un controllo select venga selezionata una voce..
ho fatto così:

function controller(){
if(document.form1.modcat.value=="" ){
alert('Devi selezionare una categoria prima di modificarla')
return false;
}
for (i=0; i<cmpidcat.length; i++) { 
if(!cmpidcat.options[i].selected){
alert('Attenzione');
return false;
} 
} 

return true;
}

Sembra controllare la textarea vuota, ma non sembra considerare il controllo sulla select..

[Mich_]

Forse vuoi una cosa di questo tipo?

function controller(){
  if(document.form1.modcat.value=="" ){
    alert('Devi selezionare una categoria prima di modificarla')
    return false;
  }

  if(document.form1.cmpidcat.selectedIndex == 0) {
    alert('Attenzione');
    return false;
  }

return true;
}

[dops]

no niente non và... cmq c'ero arrivati al fatto che non avevo messo document.nomeform

[dops]

cmq richiamo la funzione con:
return controller();
nel submit della form.

[dops]

ah ecco:

  if(document.form1.cmpidcat.selectedIndex == -1) {

l'indice 0 esiste

[Mich_]

L'indice 0 esiste, ma esiste anche una opzione selezionata di default.

La cosa piu` pulita (e valida in qualsiasi browser) e` mettere l'opzione default con il value vuoto. Puoi puoi testare il suo indice (facile se e` al primo posto), oppure puoi testare il suo value:

if(document.form1.cmpidcat.options[document.form1.cmpidcat.selectedIndex ].value == "") {

Dentro le virgoeltte ci devi mettere una stringa uguale a quella del value dell'opzione "nulla"

[dops]

Originariamente inviato da Mich_
L'indice 0 esiste, ma esiste anche una opzione selezionata di default.

La cosa piu` pulita (e valida in qualsiasi browser) e` mettere l'opzione default con il value vuoto. Puoi puoi testare il suo indice (facile se e` al primo posto), oppure puoi testare il suo value:

if(document.form1.cmpidcat.options[document.form1.cmpidcat.selectedIndex ].value == "") {

Dentro le virgoeltte ci devi mettere una stringa uguale a quella del value dell'opzione "nulla"

ok grazie mille