[javascript]: il controllo dei campi non avviene

[Bartjsp]

Questo pezzo di codice dovrebbe fare il controllo dei campi, ma in realtà viene evitato, la form va avanti senza passare da javascript, ma solo se metto il metodo .focus().

<script language="javascript">
<!--
	function checkControl()
	{
		var login = document.getElementById('idutente').value
		var pswd = document.getElementById('password').value
		
		if ((login == "") || (login == "undefined"))
		{
			alert('Inserire la login.');
                        login.focus();
			return false;
		} else if ((pswd == "") || (pswd == "undefined")) {
			alert('Inserire la password.');
                        password.focus();
			return false;
		}
		return true;
	}
//->
</script>

questa invece è la form:

      <table border="0" align="center" width="300">
      <form onSubmit="return checkControl();" action="checkLogin.php" name="areariservata" method="POST">
       <tr>
        <td class="login">ID-User</td>
       </tr>
       <tr>
        <td align="right"><input class="box_login" type="text" name="idutente" value="" id="1"/></td>
       </tr>
       <tr>
        <td class="login">Password</td>
       </tr>
       <tr>
        <td align="right"><input class="box_login" type="password" name="password" value="" id="2"/></td>
       </tr>
       <tr>
        <td align="right" vaglin="middle"><input type="submit" name="login" value="Entra"/></td>
       </tr>
       </form>
      </table>

Sto uscendo pazzo, non riesco a capire dove sia l'errore.

[cavicchiandrea]

Originariamente inviato da Bartjsp
Questo pezzo di codice dovrebbe fare il controllo dei campi, ma in realtà viene evitato, la form va avanti senza passare da javascript, ma solo se metto il metodo .focus().

<script language="javascript">
<!--
	function checkControl()
	{
		var login = document.getElementById('idutente').value
		var pswd = document.getElementById('password').value
		
		if ((login == "") || (login == "undefined"))
		{
			alert('Inserire la login.');
                        login.focus();
			return false;
		} else if ((pswd == "") || (pswd == "undefined")) {
			alert('Inserire la password.');
                        password.focus();
			return false;
		}
		return true;
	}
//->
</script>

Sto uscendo pazzo, non riesco a capire dove sia l'errore.

L'errore sta qui:
var login = document.getElementById('idutente').value
se gli assegni il value non puoi poi passargli il focus
login.focus();
puoi risolvere cosi:

var login = document.getElementById('idutente')
	
		if ((login.value == "") || (login.value == "undefined"))
		{
			alert('Inserire la login.');
                        login.focus();
			return false;
		}

[Bartjsp]

Hai perfettamente ragione, ho fatto un'errore da principiante.

Grazie.