[SOLUCIONADO] Permitir clave con letra y numeros

jmfmagnum · #1 (**permalink**) 03/11/2014, 10:38

Buenas tardes, una vez mas necesito del saber de los miembros de esta pagina,
veran tengo un programa que valida usuario con datos enviados de un simple formulario, el validador es:

Código PHP:

Ver original<?php
ob_start();
?>
<head>
<title>validar</title>
<script>
    function noexiste(){
        alert("Usuario no existe...");
        window.location = "login.php";
    }
    function bienvenido(){
        alert("Bienvenido...");
        window.location = "user/menu.php";
    }
</script>
</head>
 
<body>
<?php
 
    session_start();
    include("coneV.php");
    
    $user = $_POST["txtUser"];
    $pass = $_POST["txtPass"];
 
    $res = validarUser($user,$pass);
    
    if($row = mysql_fetch_array($res)) 
    
    
    {
        $_SESSION["user"] = $row["usuario"];
        echo "<script>bienvenido()</script>";
        desconectar();
    }
    else{
        echo "<script>noexiste()</script>";
        desconectar();
    }
    
    ob_end_flush();
?>
</body>
</html>

Y el archivo "coneV.php" es:

Código PHP:

Ver original<?php
ob_start();
?>
<?php
//quitar error de php 5.5
error_reporting(E_ALL ^ E_DEPRECATED);
 
function conectar(){
    $cnn = mysql_connect("localhost", "root", "");
    mysql_select_db("automotora", $cnn);
    return $cnn;
}
 
function validarUser($user, $pass){
    $res = mysql_query("SELECT * FROM usuarios WHERE usuario='".$user."' and clave=".$pass."",conectar());
    return $res;
}
function mostrarAuto(){
    $res = mysql_query("SELECT * FROM vehiculo  WHERE estado='en_venta'",conectar());
    return $res;
}
 
function desconectar(){
    mysql_close(conectar());
}
ob_end_flush();
?>

normalmente funciona bien...con contraseñas que requieran solo numeros , pero si uso una contraseña con letras recibo esto:

por si no se ve bien la imagen, esta dice "Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in C:\wamp\www\automotoraPHP\validar.php on line 32"

La linea que menciona el error es esta para que no busquen tanto en la primera que puse arriba :

Código PHP:

Ver originalif($row = mysql_fetch_array($res)) 
    {
        $_SESSION["user"] = $row["usuario"];
        echo "<script>bienvenido()</script>";
        desconectar();
    }
    else{
        echo "<script>noexiste()</script>";
        desconectar();
    }

lineas 32 en adelante.

Espero que no sea mucho pedir, ay que necesito saber como poner clave con letra y numero o fallare mis examenesde nuevo

gnzsoloyo · #2 (**permalink**) 03/11/2014, 10:45

Si la clave tiene letras y números, es un string, y debe ser evaluado en SQL como string... Pero lo estás poniendo como si fuese un número.
En MySQL las cadena s de texto van siempre entre apostrofos ('):
Mal:

Código PHP:

Ver originalfunction validarUser($user, $pass){
    $res = mysql_query("SELECT * FROM usuarios WHERE usuario='".$user."' and clave=".$pass."",conectar());
    return $res;
}

Bien:

Código php:

Ver originalfunction validarUser($user, $pass){
    $res = mysql_query("SELECT * FROM usuarios WHERE usuario='".$user."' and clave='".$pass."'",conectar());
    return $res;
}

Acuérdate que porque PHP sepa que el contenido es un string no implica que MySQ se entere de eso, porque no comparten nada. No son el mismo sistema y no se comunican ese tipo de cosas.

jmfmagnum · #3 (**permalink**) 03/11/2014, 10:58

Cita:

Iniciado por gnzsoloyo

Si la clave tiene letras y números, es un string, y debe ser evaluado en SQL como string... Pero lo estás poniendo como si fuese un número.
En MySQL las cadena s de texto van siempre entre apostrofos ('):
Mal:

Código PHP:

Ver originalfunction validarUser($user, $pass){
    $res = mysql_query("SELECT * FROM usuarios WHERE usuario='".$user."' and clave=".$pass."",conectar());
    return $res;
}

Bien:

Código php:

Ver originalfunction validarUser($user, $pass){
    $res = mysql_query("SELECT * FROM usuarios WHERE usuario='".$user."' and clave='".$pass."'",conectar());
    return $res;
}

Acuérdate que porque PHP sepa que el contenido es un string no implica que MySQ se entere de eso, porque no comparten nada. No son el mismo sistema y no se comunican ese tipo de cosas.

Gracias por tu ayuda, soy muy torpe en PHP aun