Problema validazione

[mrL4N]

Ciao, ho un problema con la validazione dei campi select in un form.
Sto utilizzando uno script java che controlla correttamente tutti i campi tranne il select appunto.
Dalle ricerche in rete ho visto che l'argomento e ultra ritrattato, ma ne ho provate di tutte e qualsiasi modifica inserisco, invalida anche il controllo degli altri campi.
Per farla breve il codice che sto utilizzando è questo:

java

<script type="text/javascript">
// <![CDATA[
function controlla(form)
{
 for (i = 0; i < form.length; i++)
 {
 var campo = form.elements[i];
  if (!campo.disabled && !campo.readonly && (campo.type == "text" || 

campo.type == "password" || campo.type == "textarea") && campo.value == "")
 {
 alert("Tutti i campi sono obbligatori!");
 return false;
 }
 }
 return true;
}
// ]]>
</script>

html

<td align="left" valign="top">
          <form action="preventivi.php" method="post" name="form1" onsubmit="if (controlla(this)) return true; return false">
           

Cognome e nome: 

              <input name="nome" type="text" id="nome" size="70" maxlength="70" />
              <label for="nome"></label>
              <label for="nome"></label>
            </p>
            

Email:

              <input name="email" type="text" id="email" size="35" />
          </p>
            

Numero di telefono:

            <input name="telefono" type="text" id="telefono" size="35" /></p>
            

Provincia:

            <select name="provincia" size="1">
              <option selected="selected">   </option>
                  <option>Alessandria</option>
                  <option>Asti</option>
                  <option>Bergamo</option>
              		<option>Biella</option>
                  <option>Brescia</option>
                  <option>Como</option>
                  <option>Cremona</option>
                  <option>Cuneo</option>
                  <option>Lodi</option>
                  <option>Mantova</option>
                  <option>Milano</option>
                  <option>Novara</option>
                  <option>Pavia</option>
                  <option>Piacenza</option>
                  <option>Sondrio</option>
                  <option>Torino</option>
                  <option>Varese</option>
                  <option>Verbano-Cusio-Ossola</option>
                  <option>Vercelli</option>
             </select></p>
            

Comune:
<input name="comune" type="text" size="70" maxlength="70" /></p>
            

Preventivo per:
 
            	<select name="lavoro" size="1">
                <option selected="selected">Seleziona</option>
              <option>Fotovoltaico</option>
              <option>Ristrutturazioni edili</option>
              <option>Impianti elettrici</option>
              <option>Impianti d'allarme</option>
              <option>Impianti di videosorveglianza</option>
              <option>Automazione cancelli/persiane</option>
              <option>Condizionatori</option>
              <option>Installazione caldaie</option>
              <option>Impianti di riscaldamento</option>
              <option>Solare termico</option>
              <option>Traslochi</option>
              <option>Certificazione energetica</option>
              <option>Imbianchino/cartongesso</option>
              <option>Cablaggi strutturati</option>
              <option>Siti web</option>
              <option>Idraulico</option>
              <option>Tariffe telefonia fissa e mobile</option>
              <option>Fabbro</option>
              </select></p>
            

Informazioni aggiuntive:

              <textarea name="informazioni" cols="60" rows="7">Maggiori sono le informazioni inviate, più accurati e precisi potranno essere i preventivi ricevuti</textarea>
            </p>
            

Acconsento al trattamento dei dati personali 
              <input name="privacy" type="checkbox" id="privacy" checked="checked" />
              <label for="privacy"></label>
<label for="privacy"></label>
            </p>
            <input type="submit"  value="Invia i dati per ricevere i preventivi gratuiti"  name="submit" /></form>
              </td>

[br1]

per controllare se un'opzione della select e' stata scelta:

( if campo.type.indexOf("select")>=0 && campo.selectedIndex>=0 )

Legalo tu agli altri controlli... e comincia a pensare ai checkbox e ai radio

[mrL4N]

Credo di aver già provato anche questo e come unico risultato ho ottenuto la mancanza di controllo anche sui campi text...

[br1]

Originariamente inviato da mrL4N
Credo di aver già provato anche questo e come unico risultato ho ottenuto la mancanza di controllo anche sui campi text...

Allora il tuo browser e' strano

Facci vedere come hai "gia' provato"

[mrL4N]

Originariamente inviato da br1
Allora il tuo browser e' strano

Lo so... ...infatti ultimamente sono molto preoccupato

<script type="text/javascript">
// <![CDATA[
function controlla(form)
{
 for (i = 0; i < form.length; i++)
 {
 var campo = form.elements[i];
  if (campo.type.indexOf("select")>=0 && campo.selectedIndex>=0 )
{
 alert("Inserisci il campo provincia!");
 return false;
 else if (!campo.disabled && !campo.readonly && (campo.type == "text" || 

campo.type == "password" || campo.type == "textarea") && campo.value == "")
 {
 alert("Tutti i campi sono obbligatori!");
 return false;
 }
 }
 return true;
}
// ]]>
</script>

[br1]

Indento meglio il tuo codice... noti nulla di strano?

<script type="text/javascript">
// <![CDATA[
function controlla(form) {
  for (i = 0; i < form.length; i++) {
    var campo = form.elements[i];
    if (campo.type.indexOf("select")>=0 && campo.selectedIndex>=0 ) {
      alert("Inserisci il campo provincia!");
      return false;
      else if (!campo.disabled && !campo.readonly && (campo.type == "text" || 
      campo.type == "password" || campo.type == "textarea") && campo.value == "") {
        alert("Tutti i campi sono obbligatori!");
        return false;
      }
    }
    return true;
  }
// ]]>
</script>

[mrL4N]

Originariamente inviato da br1
Indento meglio il tuo codice... noti nulla di strano?

Ma secondo te se conoscevo javascript almeno la metà di quanto la conosci tu, venivo qui a farmi prendere in giro?
Direi che è piuttosto diverso, e probabilmente sarà anche nella forma corretta...
Lo provo subito, grazie

P.S. anche con la tua indentazione il risultato non cambia...se clicco su invia senza inserire nulla, il form invia la mail come se tutto fosse compilato.

[br1]

Originariamente inviato da mrL4N
Ma secondo te se conoscevo javascript almeno la metà di quanto la conosci tu, venivo qui a farmi prendere in giro?
Direi che è piuttosto diverso, e probabilmente sarà anche nella forma corretta...
Lo provo subito, grazie

P.S. anche con la tua indentazione il risultato non cambia...se clicco su invia senza inserire nulla, il form invia la mail come se tutto fosse compilato.

Nessuno ti prende in giro, si cerca solo di fornirti gli spunti perche' ti possa rendere conto degli errori e arrivare ragionando a capire un pelino di piu' la programmazione...
Per quel che riguarda il javascript devi solo stabilire cosa vuoi fare: capirlo meglio per utilizzarlo meglio o farti fare il lavoro da qualcun'altro?

Non e' certo con l'indentazione diversa che uno script funziona o meno, ma una corretta indentazione aiuta a capire dove sono gli errori concettuali e sintattici.

Stavolta te lo scrivo corretto e te lo completo, promettimi di confrontare attentamente la versione funzionante con la precedente e cerca di ragionare sul perche' delle modifiche effettuate.

<script type="text/javascript">
// <![CDATA[
function controlla(form) {
  for (i = 0; i < form.length; i++) {
    var campo = form.elements[i];
    if (campo.type.indexOf("select")>=0 && campo.name=="provincia" && campo.selectedIndex<=0 ) {
      alert("Inserisci il campo provincia!");
      return false;
    } else if (campo.type.indexOf("select")>=0 && campo.name=="lavoro" && campo.selectedIndex<=0 ) {
      alert("Inserisci il campo lavoro!");
      return false;
    } else if (campo.name=="privacy" && !campo.checked ) {
      alert("Devi accettare la privacy!");
      return false;
    } else if (!campo.disabled && !campo.readonly && (campo.type == "text" || 
      campo.type == "password" || campo.type == "textarea") && campo.value == "") {
      alert("Tutti i campi sono obbligatori!");
      return false;
    }
  }
  return true;
}
// ]]>
</script>

Prendilo come un invito allo studio

[mrL4N]

Grazie, ora funziona correttamente...
Nel mio mancava proprio il nome del campo della select da verificare per cui non poteva certo fare quello di cui avevo bisogno.
Non mi è chiaro perchè come l'avevo messo io non controllava più nemmeno i campi testo...