Hola Sharik, angsanchez.
Varias cositas adicionales:
1.- En JavaScript no es necesario que declares las variables, aunque yo también suelo hacerlo. Sin embargo, un string no hay necesidad de usarlo como objeto.
2.- El prefijo this siempre se refiere al elemento HTML donde esta ubicado el código. Si lo usas en un evento de un botón, por ejemplo, el this se refiere al botón en si, pero cuando lo usas en una función JavaScript pierde significado al menos al leer el código (además no se realmente que valor toma).
3.- Tu función debería retornar "true" si la verificación es exitosa.
Vistas ese par de cosillas, tu código se podría escribir mas claramente asi:
Código PHP:
<script language="JavaScript">
function validar(){
ingreso = document.principal.NombreDeUsuario.value;
if (ingreso == ""){
alert("Debe introducir un nombre de usuario.");
return false;
} else {
ingreso = document.principal.Password.value;
if (ingreso == ""){
alert("Debe ingresar una contraseña.");
return false;
}
}
return true;
}
</script>
O con mayor claridad si usas un with:
Código PHP:
<script language="JavaScript">
function validar(){
with (document.principal){
if (NombreDeUsuario.value == ""){
alert("Debe introducir un nombre de usuario.");
return false;
} else {
if (Password.value == ""){
alert("Debe ingresar una contraseña.");
return false;
}
}
}
return true;
}
</script>
Saludos.