[SOLUCIONADO] no me funciona return en javascript

fandurc · #1 (**permalink**) 08/02/2013, 11:39

Hola a todos. He hecho una funcion en javascript para enviar un formulario, y cuando quiero interrumpirla usando un "return 0;" luego de una sentencia dentro de un if no me funciona y me ejecuta igual las sentencias que siguen. Esta es la funcion:

Código:

function enviar () {
    var username = document.getElementById("username").value;
    var password = document.getElementById("password").value;
    if ((username == "") && (password == "")) {        
        alert("Debe ingresar su Nombre de Usuario y su Contraseña para acceder");
        document.getElementById("username").focus();
        return 0;        
    } else {
        if (username == "") {
            alert("Debe ingresar su Nombre de Usuario para acceder");
            document.getElementById("username").focus();
            return 0;
        } else {
            if (password == "") {
                alert("Debe ingresar su Contraseña para ingresar");
                document.getElementById("password").focus();
                return 0;
            } else {                 
                document.fingreso.username.value = trim(username);
                document.forms["fingreso"].submit();
                return 1;                
            }
        }
    }        
}

En este caso si no he ingresado ni username y ni password me envia igual el formulario, ignorando mi return 0; . Ayuda xfaaaaaaa

emprear · #2 (**permalink**) 08/02/2013, 11:46

Tenés que llamar a tu función de validación con onsubmit="return funcion_validar();" en la etiqueta form, es decir que esto además
document.forms["fingreso"].submit();
te sobra

Lee estos posts y los ejemplos vinculados:

http://www.forosdelweb.com/f13/valid...3/#post4344960

http://www.forosdelweb.com/f13/que-h...5/#post4353569

SAludos

fandurc · #3 (**permalink**) 08/02/2013, 12:04

no lo he solucionado agregando el onsubmit en el form y eliminando:
document.forms["fingreso"].submit();
lo que quiero es que en cada ocasion que haya "return 0;" aborte la funcion y no pase nada pero igual, me manda el formulario con los cambios que me propones. Gracias por tu ayuda. Este es mi form por si a caso:

Código:

<form id="fingreso" name="fingreso" method="post" action="ingreso.php" onsubmit="return enviar()">
            <table>
                <tr>
                    <td>
                        Nombre de Usuario: 
                    </td>
                    <td>
                        <input type="text" id="username" name="username" size="20"> 
                    </td>
                </tr>
                <tr>
                    <td>
                        Contraseña:
                    </td>                               
                    <td>
                        <input type="password" id="password" name="password" size="20">
                    </td>
                </tr>
                <tr>
                    <td>                        
                    </td>
                    <td>
                        <input type="submit" value="Ingresar" onclick="enviar()"> 
                    </td>
                </tr>
            </table>
        </form>

caricatos · #4 (**permalink**) 08/02/2013, 12:16

Hola:

Cita:

Iniciado por fandurc

...lo que quiero es que en cada ocasion que haya "return 0;" aborte la funcion...

En javascript "return 0" no aborta nada

... tal vez ¿return false...?

Saludos

fandurc · #5 (**permalink**) 08/02/2013, 12:22

gracias caricatos, pero la misma historia, sucede exactamente lo mismo.
¿No habrá alguna forma alternativa de validar un login con un mensaje de error en caso de que falte algun dato y que no envie el form?

emprear · #6 (**permalink**) 08/02/2013, 12:33

Cita:

Iniciado por fandurc

gracias caricatos, pero la misma historia, sucede exactamente lo mismo.
¿No habrá alguna forma alternativa de validar un login con un mensaje de error en caso de que falte algun dato y que no envie el form?

Podés tomarte el trabajo de recrear y hacer funcionar cualquiera de los ejemplos, que para eso están, que dejé en los links citados, por ejemplo

Código HTML:

Ver original<!DOCTYPE html>
<html lang="es-ar">
<head>
<meta charset="utf-8" />
<title>Registrar usuario</title>
<script type="text/javascript">
//<![CDATA[
function validar(){
var elNombre = document.getElementById('nombre');
if(elNombre.value == ''){
document.getElementById('error_nombre').innerHTML = "Falta nombre";
return false;
}
}
//]]>
</script>
</head>
<body>
<form action="agregar_usuario.php" method="post" onsubmit="return validar();">
<p>
<label for="nombre">Nombre</label>
<input type="text" value="" id="nombre" name="nombre" autocomplete="off" /> 
<span id="error_nombre"><!-- mensaje de error --></span><br /><br />
<input type="submit" value="procesar" name="procesando" />
</p>
</form>
</body>
</html>

Si aún así no te funciona, la respuesta es muy sencilla, revisá en primer lugar la sintáxis con a consola de errores y si no los hay, tus condicionales están mal planteados

SAludos

fandurc · #7 (**permalink**) 08/02/2013, 13:00

gracias emprear. Tus ejemplo que salen en los links que me pasaste ya los habia leido. La cosa es que cuando pones

Código:

if(elNombre.value == ''){
document.getElementById('error_nombre').innerHTML = "Falta nombre";
return false;
}

necesito anidar 4 if, para un alert cuando falte el username y password, un alert cuando falte solo username, un alert cuando falte solo password, y otro que envie el form si no se cumplen los anteriores. Creo que ahi esta el error, no se como anidarlos. Por que siempre se ejecuta el ultimo que me envia el form. Saludos

fandurc · #8 (**permalink**) 08/02/2013, 13:02

me ejecuta envia siempre el form aunque falten datos y me ejecuta el php. Saludos

caricatos · #9 (**permalink**) 08/02/2013, 13:07

Hola:

Cita:

Iniciado por fandurc

...necesito anidar 4 if...

Para evitar ese anidamiento yo uso un array para los mensajes:

mensajes = [];

if (username == "") mensajes.push("falta el username");
if (nombre == "") mensajes.push("falsta el nombre");

if (mensajes.length > 0) alert (mensajes.join(" y ");
return (mensajes.length == 0);

Saludps

emprear · #10 (**permalink**) 08/02/2013, 13:20

Cita:

Iniciado por fandurc

me ejecuta envia siempre el form aunque falten datos y me ejecuta el php. Saludos

En casos asi vas a lo seguro

Código HTML:

Ver original<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" 
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title>titulo</title> 
<meta http-equiv="content-type" content="text/html; charset=utf-8" />
 
<script type="text/javascript">
//<![CDATA[
function enviar(){
    var username = document.getElementById("username").value;
    var password = document.getElementById("password").value;
    if ((username == "") || (password == "")) {        
        alert("Debe ingresar su Nombre de Usuario y su Contraseña para acceder");
        return false;        
    }
}
 
//]]>
</script>
</head>
<body>
<form action="ingreso.php" method="post" onsubmit="return enviar();">
usuario <input type="text" name="username" id="username" value="" /><br />
clave <input type="password" name="username" id="password" value="" /><br />
<input type="submit" name="procesar" />
</form>
</body>
</html>

Como tu mensaje afecta a los 2 errores, usas || (OR) y no && (AND)

Veo por ahi dando vueltas un trim(), supongo que en alguna parte dicha función esta definida, y el trim debés hacérlo a la s variables username y password antes del condicional

Si querés otra combinación, a experimentar un poco

SAludos

fandurc · #11 (**permalink**) 08/02/2013, 13:26

cambie el input del form de type="submit" a type="button" y se soluciono. Ya no se envía siempre el formulario. Gracias por su ayuda emprear y caricatos!!!

fandurc · #12 (**permalink**) 08/02/2013, 13:29

Incluso de esta forma se enviaba igual :

Código Javascript:

Ver originalif ((username == "") && (password == "")) {        
        alert("Debe ingresar su Nombre de Usuario y su Contraseña para acceder");
        document.getElementById("username").focus();
        return false;        
    }
    if ((username == "") && (password != "")) {
        alert("Debe ingresar su Nombre de Usuario para acceder");
        document.getElementById("username").focus();            
        return false;
    }
    if ((username != "") && (password == "")){
        alert("Debe ingresar su Contraseña para ingresar");
        document.getElementById("password").focus();
        return false;
    }
    if ((username != "") && (password != "")){
        document.fingreso.username.value = trim(username);
        document.forms["fingreso"].submit();
        return true;                
    }

Cambie el boton del form de submit a button y listo. Gracias emprear!

caricatos · #13 (**permalink**) 08/02/2013, 16:14

Hola:

He visto que el tema está marcado como "Solucionado", y yo afirmo que "NO" está solucionado

El error es muy evidente, "hay un error"...

Me explico:

Si no se cancela el envío con "return false", es simplemente porque no llega a esa línea de código... entonces hay una línea "incorrecta".

O el control no tiene el atributo id="username" (o id="password"), o está repe o similar... id es distinto de name.

No me gusta esa opción de marcar los temas como solucionado, ya que al menos en este caso no lo está.

Por cierto, sin un botón del tipo submit, tu formulario es "no accesible"

Si te vale considerar el tema solucionado, creo que estás engañándote.
Deberías ver la consola de errores de tu navegador y volver a intentarlo con un botón submit.

Saludos

emprear · #14 (**permalink**) 08/02/2013, 22:39

@fandurc

efectivamente, encontraste una solución, pero haciendo caso omiso de todas las indicaciones que te hemos hecho, a la casa se entra por la puerta, no por el tiro de la chimenea.
Y te aclaro por las dudas que con tu sistema, con que ingresen 2 espacios en blanco en los campos, se va a procesar el formulario igual. Se puede saber entonces por que tanto empeño en poner mensajes y advertencias?

Quisiera ver que tipo de validación hacés en el server.

ForroDelWeb · #15 (**permalink**) 07/11/2013, 07:50

.CS

this.btnGuardar.Attributes.Add("onclick","return MyFuncion();");

Javascript
function MyFunction()
{
if (....)
{
return false;
}

return true;
}