3

Me percate que la validación no funcionaba y que no importa los datos que ingrese desde el mantenedor este código siempre devuelve 1. Lo que busco es que devuelva 0 si es que la consulta sql no devolvió ningún resultado.

if (!$cnn) {
    die("Conexion Fallida: " . mysqli_connect_error());
  }else {
    $nombre_usuario = $_POST['txt_nombre_admin']; //Nombre del usuario
    $contraseña_usuario = $_POST['txt_password_admin'];//Contraseña usuario
    $privilegio_usuario = $_POST['sel_privilegio']; //Nivel del usuario

    $rs = mysqli_prepare($cnn,"SELECT privilegio FROM usuarios WHERE (nombre = ? and password = ? and privilegio = ?)");
    $ok = mysqli_stmt_bind_param($rs, "ssi", $nombre_usuario, $contraseña_usuario, $privilegio_usuario);
    $ok = mysqli_stmt_execute($rs); //Siempre da 1

    if ($ok == false) {
      echo $ok; 
    }else {
      echo $ok;
    }
3
  • 1
    Sebastián, cuando uses funciones PHP lee lo que dice el apartado Valores devueltos de esa función en el Manual. En el caso de mysqli_stmt_execute dice esto textualmente: Devuelve TRUE en caso de éxito o FALSE en caso de error.. La ejecución puede ser correcta TRUE pero no devolver ningún resultado. Para verificar si hay registros lo correcto sería que hagas una consulta con COUNT(*) y verifiques si la misma devolvió filas.
    – A. Cedano
    el 27 nov. 2018 a las 15:58
  • 1
    Si miras la documentación php.net/manual/es/mysqli-stmt.execute.php, verás que la función devuelve true o false dependiendo de si tuvo éxito o no. Luego que retorne 1 no indica si encontró registros o no, solo indica que no hay error.
    – track3r
    el 27 nov. 2018 a las 15:59
  • Entendí perfecto gracias por la aclaración! el 27 nov. 2018 a las 16:07

1 respuesta 1

1

La comprobación de la existencia de datos no puede ser dependiente de el éxito o el fracaso de una función cualquiera. Lo correcto sería revisar los mismos datos. Puedes hacerlo por ejemplo mediante una consulta COUNT(*). Si te interesa información de alguna columna de la tabla, entonces puedes hacer un SELECT normal de esas columnas y determinar al mismo tiempo la existencia de datos mediante num_rows. En cualquiera de los dos casos la comprobación se estaría haciendo sobre los datos en sí.

Propongo este código:

$ok=FALSE;
if (!$cnn) {
    die("Conexion Fallida: " . mysqli_connect_error());
} else {
    $nombre_usuario = ( empty ($_POST['txt_nombre_admin']) )   ? NULL : $_POST['txt_nombre_admin'];
    $clave_usuario = ( empty ($_POST['txt_password_admin']) )  ? NULL : $_POST['txt_password_admin'];
    $privilegio_usuario = ( empty ($_POST['sel_privilegio']) ) ? NULL : $_POST['sel_privilegio'];

    if ($nombre_usuario && $clave_usuario && $privilegio_usuario){
        $sql="SELECT COUNT(*) total FROM usuarios WHERE nombre = ? AND password = ? AND privilegio = ?";
        if ($rs = mysqli_prepare($cnn,$sql)){
            $stmt = mysqli_stmt_bind_param($rs, "ssi", $nombre_usuario, $clave_usuario, $privilegio_usuario);
            mysqli_stmt_execute($rs);
            mysqli_stmt_bind_result($rs, $numFilas);
            $ok= ($numFilas > 0) ? TRUE : FALSE;
        }else{
            echo "Error preparando la consulta";
        }
    }else{
        echo "No se pasaron los datos completos en el POST";
    }
}
echo $ok;

En un código controlado, que informa siempre de lo que ha ocurrido. Además, verifica los datos que hay en POST mediante operadores ternarios. Tiene la variable $ok en FALSE por defecto y sólo cambiará cuando haya filas en la consulta.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.