Problema validando formulario en PHP con JavaScript

[Uri Macías]

Tengo un problema.
Al intentar validar un formulario con JavaScript en un archivo PHP se sigue de paso como si no existiera la opción onSubmint="return valida(this);" en el formulario o como si no se ejecutara el código JS.
¿podría ser error de configuración?

el código PHP para salida JabaScript es:

Código PHP:

  print '<script language="javascript" type="text/javascript"> 
            alert("está por enviar datos") 
            function vacio(campo) { 
                for ( i = 0; i < campo.length; i++ ) { 
                        if ( campo.charAt(i) != " " ) { 
                                return true 
                        } 
                } 
                return false 
            } 
             
            function valida(Formulario){ 
                var retorna = false 
                if(vacio(Formulario.usuario.value) == false){ 
                    retorna = false 
                } 
                else{ 
                    if(vacio(Formulario.contrasena.value) == false){ 
                        retorna = false 
                    } 
                    else 
                        return retorna = true 
                    } 
                } 
                if (retorna == false){ 
                    alert ("No debe haber campos vacios") 
                } 
                return retorna 
            } 
        </script>'  
   

el código PHP para la salida del formulario es:

Código PHP:

  print '<form name="autentica" method="post" action="inicioPrueba.php" onSubmit="return valida(this);"> 
        <label>Usuario</label> 
        <input name="usuario" type="text" id="usuario" size="16" maxlength="16"> 
        <label>Contraseña</label> 
        <input name="contrasena" type="password" id="contrasena" size="16" maxlength="16"> 
        <input type="submit" name="Submit" value="Registrar"> 
        </form>'  
   

De antemano, gracias.

[djreficul]

Una sugerencia, sal de PHP todo lo q sea necesario, te será muy útil para diseñar.

Otra sugerencia, si ves que un código javascript no te funciona una buena opción es usar la claúsula try { } catch(e) { } para poder ver cual es el error y, en tu caso, mandar un return false y q no se envíe el formulario. La consola de Firefox te puede servir de ayuda para determinar los problemas.

Y agrego q el problema es más de javascript q de PHP.

Saludos.

EDITO para explikarme mejor:

Con lo de sal de PHP me refiero a q en lugar de usar print 'lo q quiera imprimir con o sin etiquetas html'; es mejor:

Código PHP:

 //Tu caso: 
print '<script language="javascript" type="text/javascript">  
            alert("está por enviar datos")  
            function vacio(campo) {  
                for ( i = 0; i < campo.length; i++ ) {  
                        if ( campo.charAt(i) != " " ) {  
                                return true  
                        }  
                }  
                return false  
            }  
              
            function valida(Formulario){  
                var retorna = false  
                if(vacio(Formulario.usuario.value) == false){  
                    retorna = false  
                }  
                else{  
                    if(vacio(Formulario.contrasena.value) == false){  
                        retorna = false  
                    }  
                    else  
                        return retorna = true  
                    }  
                }  
                if (retorna == false){  
                    alert ("No debe haber campos vacios")  
                }  
                return retorna  
            }  
        </script>'; 
 
//A lo q me refiero: 
?> 
<script language="javascript" type="text/javascript">  
            alert("está por enviar datos")  
            function vacio(campo) {  
                for ( i = 0; i < campo.length; i++ ) {  
                        if ( campo.charAt(i) != " " ) {  
                                return true  
                        }  
                }  
                return false  
            }  
              
            function valida(Formulario){  
                var retorna = false  
                if(vacio(Formulario.usuario.value) == false){  
                    retorna = false  
                }  
                else{  
                    if(vacio(Formulario.contrasena.value) == false){  
                        retorna = false  
                    }  
                    else  
                        return retorna = true  
                    }  
                }  
                if (retorna == false){  
                    alert ("No debe haber campos vacios")  
                }  
                return retorna  
            }  
        </script> 
<?php   

[Uri Macías]

mmm, no entiendo cual es mi problema...

el código:

Código PHP:

  <script language="javascript" type="text/javascript"> 
alert("esta por enviar datos") 
</script>  
  

entre <head> </head>

en un archivo HTML si muestra la salida, pero el mismo código en un archivo PHP no muestra nada.

[djreficul]

Si no hay errores en el codigo PHP debe mostrarse también pero por comodidad para el diseño e incluso para leer los códigos de otros lenguajes creo q es más cómodo entrar y salir de PHP sólo lo necesario.

¿Tienes activada la visualización de errores? ¿Te da algún error el script PHP? ¿Tienes algo q defina si se muestra una parte del código y otra no?

Con los print solos no puedo determinar la causa. Aparentemente parece estar bien a excepción de q no están los ; para finalizar la instrucción.

Saludos.

[Uri Macías]

Ya he separado el código de JS y PHP, espero que no sea un posterior problema la presentación de todo el código.

Respecto a lo demás, si display_errors = On, y todo funciona correctamente hasta donde se supone debe de hacer la validación.

si no es molestia muestro el código entero.

Código PHP:

  <?php 
    ini_set('session.cookie_lifetime','120'); 
    ini_set('session.gc_maxlifetime','120'); 
    session_start(); 
?> 
<html> 
<head> 
    <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /> 
    <title>Prueba de autenticaci&oacute;n</title> 
    <script language="javascript" type="text/javascript"> 
        try{alert("está por enviar datos")}catch(e){print e} 
        function vacio(campo) { 
            for ( i = 0; i < campo.length; i++ ) { 
                    if ( campo.charAt(i) != " " ) { 
                            return true 
                    } 
            } 
            return false 
        } 
         
        function valida(Formulario){ 
            var retorna = false 
            if(vacio(Formulario.usuario.value) == false){ 
                retorna = false 
            } 
            else{ 
                if(vacio(Formulario.contrasena.value) == false){ 
                    retorna = false 
                } 
                else 
                    return retorna = true 
                } 
            } 
            if (retorna == false){ 
                alert ("No debe haber campos vacios") 
            } 
            return retorna 
        } 
    </script> 
</head> 
 
<body> 
    <?php 
    if(!isset($_POST['usuario'])){ 
        formulario(); 
    } 
    else{ 
        autentica($_POST['usuario'], $_POST['contrasena']); 
    } 
    ?> 
     
</body> 
</html> 
<?php  
    function formulario(){ 
        print '<form name="autentica" method="post" action="inicioPrueba.php" onSubmit="return valida(this);"> 
        <label>Usuario</label> 
        <input name="usuario" type="text" id="usuario" size="16" maxlength="16"> 
        <label>Contraseña</label> 
        <input name="contrasena" type="password" id="contrasena" size="16" maxlength="16"> 
        <input type="submit" name="Submit" value="Registrar"> 
        </form>';         
        } 
         
        function autentica($nick, $pass){ 
            if(!isset($_SESSION['intentos'])){ 
                $_SESSION['intentos']=1; 
            } 
            else{ 
                $_SESSION['intentos'] += 1; 
            } 
            if($_SESSION['intentos'] < 4){ 
                include "../Connections/conexion.php"; 
     
                $query = sprintf("SELECT * FROM tb_usuarios WHERE nick='%s' and pass='%s'", 
                    mysql_real_escape_string($nick), 
                    mysql_real_escape_string($pass)); 
                $resultado = mysql_query($query); 
                 
                if(!$resultado){ 
                    $message  = 'Consulta inválida: ' . mysql_error() . "\n"; 
                    $message .= $query; 
                    die($message); 
                } 
                else{ 
                    if($row = mysql_fetch_row($resultado)){ 
                        print $row[7] . "\n"; 
                        print $row[8]; 
                    } 
                    else{ 
                        print "El usuario o contraseña son inválidos, por favor intente de nuevo"; 
                        formulario(); 
                    } 
                }//Fin If valida resultado 
                 
            }//Fin comprobación de intentos 
            else{ 
                print "A intentado más de tres veces. Por favor intente de nuevo en dos minutos"; 
            }     
        } 
?>   

[djreficul]

Iniciado por Uri Macías

Ya he separado el código de JS y PHP, espero que no sea un posterior problema la presentación de todo el código.

Respecto a lo demás, si display_errors = On, y todo funciona correctamente hasta donde se supone debe de hacer la validación.

si no es molestia muestro el código entero.

Código PHP:

  <?php 
    ini_set('session.cookie_lifetime','120'); 
    ini_set('session.gc_maxlifetime','120'); 
    session_start(); 
?> 
<html> 
<head> 
    <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /> 
    <title>Prueba de autenticaci&oacute;n</title> 
    <script language="javascript" type="text/javascript"> 
        try{alert("está por enviar datos")}catch(e){print e} 
        function vacio(campo) { 
            for ( i = 0; i < campo.length; i++ ) { 
                    if ( campo.charAt(i) != " " ) { 
                            return true 
                    } 
            } 
            return false 
        } 
         
        function valida(Formulario){ 
            var retorna = false 
            if(vacio(Formulario.usuario.value) == false){ 
                retorna = false 
            } 
            else{ 
                if(vacio(Formulario.contrasena.value) == false){ 
                    retorna = false 
                } 
                else 
                    return retorna = true 
                } 
            } 
            if (retorna == false){ 
                alert ("No debe haber campos vacios") 
            } 
            return retorna 
        } 
    </script> 
</head> 
 
<body> 
    <?php 
    if(!isset($_POST['usuario'])){ 
        formulario(); 
    } 
    else{ 
        autentica($_POST['usuario'], $_POST['contrasena']); 
    } 
    ?> 
     
</body> 
</html> 
<?php  
    function formulario(){ 
        print '<form name="autentica" method="post" action="inicioPrueba.php" onSubmit="return valida(this);"> 
        <label>Usuario</label> 
        <input name="usuario" type="text" id="usuario" size="16" maxlength="16"> 
        <label>Contraseña</label> 
        <input name="contrasena" type="password" id="contrasena" size="16" maxlength="16"> 
        <input type="submit" name="Submit" value="Registrar"> 
        </form>';         
        } 
         
        function autentica($nick, $pass){ 
            if(!isset($_SESSION['intentos'])){ 
                $_SESSION['intentos']=1; 
            } 
            else{ 
                $_SESSION['intentos'] += 1; 
            } 
            if($_SESSION['intentos'] < 4){ 
                include "../Connections/conexion.php"; 
     
                $query = sprintf("SELECT * FROM tb_usuarios WHERE nick='%s' and pass='%s'", 
                    mysql_real_escape_string($nick), 
                    mysql_real_escape_string($pass)); 
                $resultado = mysql_query($query); 
                 
                if(!$resultado){ 
                    $message  = 'Consulta inválida: ' . mysql_error() . "\n"; 
                    $message .= $query; 
                    die($message); 
                } 
                else{ 
                    if($row = mysql_fetch_row($resultado)){ 
                        print $row[7] . "\n"; 
                        print $row[8]; 
                    } 
                    else{ 
                        print "El usuario o contraseña son inválidos, por favor intente de nuevo"; 
                        formulario(); 
                    } 
                }//Fin If valida resultado 
                 
            }//Fin comprobación de intentos 
            else{ 
                print "A intentado más de tres veces. Por favor intente de nuevo en dos minutos"; 
            }     
        } 
?>   

Prueba con la claúsula try { } catch (e) { }. Te escribo el ejemplo:

Código HTML:

function valida(Formulario){
try {
			var retorna = false
			if(vacio(Formulario.usuario.value) == false){
				retorna = false
			}
			else{
				if(vacio(Formulario.contrasena.value) == false){
					retorna = false
				}
				else
					return retorna = true
				}
			}
			if (retorna == false){
				alert ("No debe haber campos vacios")
			}
			return retorna
} catch (e) {
    alert ("Ocurrio una excepcion no controlada: " + e.message)
    return false
}
		}

De esta manera no enviará el formulario y te mostrará el error en caso de q se ejecute la función. Una vez tengas la función completamente testeada y depurada puedes eliminar la claúsula o el alert, como veas.

Saludos

[Uri Macías]

Probado y nada, ni mensaje alert("está por enviar datos") ni un error, ¿podrá ser algún error de configuración? es como si todo el código JS no estuviera allí.

[djreficul]

Bien, he mirado en profundidad y te falta una llave después de un else en la función valida. Concretamente:

function valida(Formulario){
var retorna = false
if(vacio(Formulario.usuario.value) == false){
retorna = false
}
else{
if(vacio(Formulario.contrasena.value) == false){
retorna = false
}
else {
return retorna = true
}
}
if (retorna == false){
alert ("No debe haber campos vacios")
}
return retorna
}

Por otro lado la claúsula try catch por sí sola no sirve para nada, hay q anidarla en una función como yo te expuse en mi ejemplo anterior.

Saludos.

[Uri Macías]

Bien, disculpa la molestia, ahora todo está funcionando correctamente.

Gracias.

[djreficul]

De nada hombre y no te preocupes, estamos todos aquí para aprender. Y molestias ninguna, molestias sólo cuando la gente dice tonterias y gilipolleces varias.

Aquí buen rollo siempre.

Saludos.