Un mensaje de alerta

Jondalar · #1 (**permalink**) 01/02/2009, 11:35

hola, tengo un problema con codigo:

lo tengo preparado para k acepte o no a unos usuarios, pero cuando pongo

else {
return false;
alert ('Contraseña incorrecta')
}

me pone el mensaje cuando la contraseña esta mal pero tambien cuando la pongo bien (aunque en este caso me redirige a la pagina destino igualmente)

Código HTML:

<HTML>
<HEAD>
<title></title>
<script LANGUAGE="JavaScript" >
function Login(){
var done=0;
var usuario=document.login.usuario.value;
var password=document.login.password.value;
if (usuario=="1" && password=="1") {
window.location="1.html";
}
if (usuario=="2" && password=="2") {
window.location="2.html";
}
if (usuario=="3" && password=="3") {
window.location="3.html";
}
if (usuario=="4" && password=="4") {
window.location="4.html";
}
if (usuario=="administrador" && password=="administrador") { 
window.location="administrador.html";
}
else {
return false;
alert ('Contraseña incorrecta')
}
}
</script>
</head>
<BODY>

<FORM name=login>
Usuario:<input type=text name=usuario><br>
Clave:<input type=PASSWORD name=password><input type=button value="Entrar" onClick="Login()">
</FORM>

</BODY>
</HTML>

alguien podria decirme k es lo k pongo mal?

gracias
salu2

quimfv · #2 (**permalink**) 01/02/2009, 13:37

Código:

function Login(){
var done=0;
var usuario=document.login.usuario.value;
var password=document.login.password.value;
var correcto=false;
if (usuario=="1" && password=="1") {
window.location="1.html";
correcto=true;
}else if (usuario=="2" && password=="2") {
window.location="2.html";
correcto=true;
}else if (usuario=="3" && password=="3") {
window.location="3.html";
correcto=true;
}else if (usuario=="4" && password=="4") {
window.location="4.html";
correcto=true;
}else if (usuario=="administrador" && password=="administrador") { 
window.location="administrador.html";
correcto=true;
}

if (!correcto) {
return false;
alert ('Contraseña incorrecta o usuario!!!')
}
}

Tienes claro que con ese login podrá entrar todo el mundo, con solo mirar el codigo de la pagina?

Quim

Jondalar · #3 (**permalink**) 06/02/2009, 11:10

no si eso lo tengo claro pero creo k si lo enlazo a un archivo .js y ese archivo lo meto dentro de una carpeta sera mas dificil encontra eso no?

Jondalar · #4 (**permalink**) 06/02/2009, 11:15

acabo de probar ese codigo me acabas de poner y lo k me pasa aora es k al poner una contraseañ correcta se va a la pagina correcta pero si pongo una contraseña incorrecta no ocurre absolutamente nada

gracias

David · #5 (**permalink**) 06/02/2009, 11:20

Tu sistema de seguridad está mal planteado, ya que yo puedo acceder directamente a la página administrador.html y listo, además, como se dijo, se puede ver en el código aunque lo pongas en un *.js en otro directorio.

El sistema de seguridad deberías hacerlo del lado del servidor.

Jondalar · #6 (**permalink**) 06/02/2009, 11:23

bueno luego probare pero ese no es mi problema

mi problema lo he planteado anteriormente

gracias por la informacion igualmente

David · #7 (**permalink**) 06/02/2009, 11:26

Es que fíjate que pones un return false antes del alert, lo cual hace que la función devuelva falso y todo el código posterior (en este caso el mensaje) es ignorado por el intérprete.

Jondalar · #8 (**permalink**) 06/02/2009, 11:29

acabo de probarlo y aora me fuciona perfectament

asi k ese ere el problema, k problem mas tonto!!

muchas gracias

mariocervantes · #9 (**permalink**) 01/04/2009, 01:08

Ya quite el return false y sigue mostrando el mensaje de alerta cuando entra este bien o mal la contraseña!! espero que alguien me pueda ayudar!!

Lo deje asi:
if (usuario=="usuario3" && password=="usuario3") {
window.location="usuario3.html";
}
if (usuario=="usuario4" && password=="usuario4") {
window.location="usuario4.html";
}
if (usuario=="administrador" && password=="administrador") {
window.location="administrador.html";
}
else {alert ("Por favor verifique sus datos")}
}
</script>

Muchas gracias de antemano

Panino5001 · #10 (**permalink**) 01/04/2009, 01:13

No pierdas tiempo. Eso es impresentable en muchos sentidos.