non capisco perché questo script sia sbagliato

[awd]

  function Modulo2() {
    //Effettua il controllo sul campo "Nominativo"
    if (document.getElementById('Newsletter').Nominativo.value.replace(/^\s+/,"").replace(/\s+$/,"").length == 0){
      alert('Il campo "Nominativo" è obbligatorio.');
      document.getElementById('Newsletter').Nominativo.focus();
      return false;
      }
    //Effettua il controllo sul campo "Email"
    if (document.getElementById('Newsletter').Email.value.length == 0){
      alert('Il campo "Email" è obbligatorio.');
      document.getElementById('Newsletter').Email.focus();
      return false;
      }
      else {
        re=/\s+$|^\s+/g;
        document.getElementById('Newsletter').Email.value = document.getElementById('Newsletter').Email.value.replace(re,"");

        Filtro = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/;
        if (!Filtro.test(document.getElementById('Newsletter').Email.value)){
          alert('Inserire un indirizzo Email corretto.');
          document.getElementById('Newsletter').Email.select();
          return false;
          }
        }
    //INVIA IL MODULO
    else {
      document.getElementById('Newsletter').action = "sendnews.php";
      document.getElementById('Newsletter').submit();
      }
  }

la soluzione la so già, basta spostare una graffa, ma non capisco, a me sembra giusto così!

[willybit]

Ciao awd,

un else lo puoi mettere solo dopo un if... tu lo hai messo dopo un altro else.
Ti consiglio di identare meglio il codice per non sbagliarti

function Modulo2() {
	//Effettua il controllo sul campo "Nominativo"
	if (document.getElementById('Newsletter').Nominativo.value.replace(/^\s+/,"").replace(/\s+$/,"").length == 0){
		alert('Il campo "Nominativo" è obbligatorio.');
		document.getElementById('Newsletter').Nominativo.focus();
		return false;
	}
	//Effettua il controllo sul campo "Email"
	if (document.getElementById('Newsletter').Email.value.length == 0){
		alert('Il campo "Email" è obbligatorio.');
		document.getElementById('Newsletter').Email.focus();
		return false;
	}
	else {
		re=/\s+$|^\s+/g;
		document.getElementById('Newsletter').Email.value = document.getElementById('Newsletter').Email.value.replace(re,"");
		Filtro = /^([a-zA-Z0-9_\.\-])+\@(([a-zA-Z0-9\-]{2,})+\.)+([a-zA-Z0-9]{2,})+$/;
		if (!Filtro.test(document.getElementById('Newsletter').Email.value)){
			alert('Inserire un indirizzo Email corretto.');
			document.getElementById('Newsletter').Email.select();
			return false;
		}
		else {//INVIA IL MODULO
			document.getElementById('Newsletter').action = "sendnews.php";
			document.getElementById('Newsletter').submit();
		}
	}
}

[awd]

ma io pensavo che l'else facesse riferimento all'if precedente