excepcion en pdo

[Montes28]

hola amigos tengo la siguiente situacion

necesito mostrar un mensaje de error con la siguiente funcion me conecto a la base de datos y cuando ingreso mal el usuario ya la calve observo el error ose que estoy capturando bien la excepcion hasta ahi todo bien .

Código PHP:

public function conectar(){
 
    $usuario = $_POST['usuario'];
    $contrasenia = $_POST['clave'];
 
                
            try {
                 $db = new PDO('pgsql:host=localhost;dbname=volcanoes',$usuario,$contrasenia);
                }catch(PDOException $e) {
                //echo $e->getMessage();
                echo "Usuario y Clave Invalidas";
                }
                
                return $db;
        }
 

ahora requiero caputar la excepcion en

Código PHP:

$consulta = $db->prepare("select usu_usuario, usu_clave from usuarios where usu_usuario='$usuario' and usu_clave='$clave' and usu_estado = 1");
            $consulta->execute();
 

aprosito no envie ningun valor y me aparece el siguiente error

Fatal error: Call to a member function prepare() on a non-object in C:\websites\volcanicos\models\login_Model.php on line 16

y la linea 16 es $consulta = $db->prepare("select ussuario, usu_clave from usuar ....

Código PHP:

class Login
    {
        
            
        /*----  Validar Usuarios -----------------------------------------*/
        public function validate_users($db)
        {
        
            $usuario = $_POST['usuario'];
            $clave = $_POST['clave'];
            
                
            $consulta = $db->prepare("select usu_usuario, usu_clave from usuarios where usu_usuario='$usuario' and usu_clave='$clave' and usu_estado = 1");
            $consulta->execute();
            
            $count = $consulta->rowCount();
 
            
            
            while($fila = $consulta->fetch())
            {
            
            $_SESSION['nombre'] = $_POST['usuario'];
            $_SESSION['clave'] = $_POST['clave'];
 
            header("Refresh: 0; URL=/volcanicos/");
            die();
            
            }
            
    
            
             if (!$this->consulta)
                {
                    $_SESSION['invalidate_user'] = true;
                }
                
        }
}
 

por si los datos enviados por post estan vacios utilizo

if (empty($usuario) || empty($clave)) {
echo "No ingreso el Usuario y Contraseña";
}else{.....

como y donde debo de capturar la excepcion , me interesa $consulta = $db->prepare(......

[GatorV]

Pues en tu bloque catch() ya estas capturando la excepción...

[Montes28]

GatorV el error es cuando no se puede realizar la consulta por que los parametros no cumplen

Usuario y Clave Invalidas
Fatal error: Call to a member function prepare() on a non-object in C:\websites\volcanicos\models\login_Model.php on line 21

esta es la linea
$consulta = $db->prepare("select ussuario, usu_clave from usuarios where ussuario='$usuario' and usu_clave='$clave' and usu_estado = 1");

[GatorV]

Pues sí, ya que cachas la excepción y no haces nada más que imprimir un mensaje y aún así regresas el objeto $db que es nulo.

El problema es tú código, tienes que agregar un chequeo antes de usar el objeto $db para que verifiques si es diferente de nulo.

[Montes28]

Código PHP:

public function conectar(){
 
                $usuario = $_POST['usuario'];
                $contrasenia = $_POST['clave'];
 
                
            try {
                 $db = new PDO('pgsql:host=localhost;dbname=volcanoes',$usuario,$contrasenia);
                }catch(PDOException $e) {
                //echo $e->getMessage();
                echo "Usuario y Clave Invalidas";
                }
                
                return $db;
        }
 

Código PHP:

if (empty($db)) {
               echo "Consulta fallida";
        }
 

pero no se como colocar el el try

[abimex]

Código PHP:

try {
  $db = new PDO('pgsql:host=localhost;dbname=volcanoes',$usuario,$contrasenia);
}catch(PDOException $e) {
  throw new Exception("Usuario y Clave Invalidas");
}
 

Ahi lanzas la excepcion con tu mensaje, o simplemente puedes hacer un die

[Montes28]

abimex gracias por responder pero obtengo este error

Fatal error: Uncaught exception 'Exception' with message 'Usuario y Clave Invalidas' in C:\websites\volcanicos\models\conexion.php:22 Stack trace: #0 C:\websites\volcanicos\controllers\login_Controlle r.php(18): Conexion->conectar() #1 C:\websites\volcanicos\index.php(64): login_users() #2 {main} thrown in C:\websites\volcanicos\models\conexion.php on line 22

la linea 22 es esta

Código PHP:

throw new Exception("Usuario y Clave Invalidas");
 

[GatorV]

¿pero cual es el caso de cachar la excepción para mandar otra? Eso lo que hace es romper con todo el trace, si lo quieres hacer así al menos pasa la excepción para no perder la original:

Código PHP:

try {
* $db = new PDO('pgsql:host=localhost;dbname=volcanoes',$usuario,$contrasenia);
}catch(PDOException $e) {
* throw new Exception("Usuario y Clave Invalidas", 0, $e);
}
 

Saludos.

[Montes28]

GatorV hice lo que indicas y obtengo este error

SQLSTATE[08006] [7] FATAL: la autentificaci�n password fall� para el usuario �gggggg�
Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[08006] [7] FATAL: la autentificaci�n password fall� para el usuario �gggggg�' in C:\websites\volcanicos\models\conexion.php:20 Stack trace: #0 C:\websites\volcanicos\models\conexion.php(20): PDO->__construct('pgsql:host=loca...', 'gggggg', 'hoe798cs') #1 C:\websites\volcanicos\controllers\login_Controlle r.php(18): Conexion->conectar() #2 C:\websites\volcanicos\index.php(64): login_users() #3 {main} Next exception 'Exception' with message 'Usuario y Clave Invalidas' in C:\websites\volcanicos\models\conexion.php:25 Stack trace: #0 C:\websites\volcanicos\controllers\login_Controlle r.php(18): Conexion->conectar() #1 C:\websites\volcanicos\index.php(64): login_users() #2 {main} thrown in C:\websites\volcanicos\models\conexion.php on line 25

[GatorV]

Sí, así no pierdes la excepción original al "cacharla" y agregar una excepción nueva.

[Montes28]

GatorV pero te entiendo , hago lo que me indicas y obtengo un errro en del mensaje" lugar de Usuario y Clave Invalidas"

Código PHP:

public function conectar(){
 
                $usuario = $_POST['usuario'];
                $contrasenia = $_POST['clave'];
 
                
            try {
                 $db = new PDO('pgsql:host=localhost;dbname=volcanoes',$usuario,$contrasenia);
                 
                }catch(PDOException $e) {
                echo $e->getMessage();
                //echo "Usuario y Clave Invalidas";
                throw new Exception("Usuario y Clave Invalidas", 0, $e);
                }
                
    
                return $db;
        }
 

Fatal error: Uncaught exception 'PDOException' with message 'SQLSTATE[08006] [7] FATAL: la autentificaci�n password fall� para el usuario �sasasasas�' in C:\websites\volcanicos\models\conexion.php:20 Stack trace: #0 C:\websites\volcanicos\models\conexion.php(20): PDO->__construct('pgsql:host=loca...', 'sasasasas', 'hoe798cs') #1 C:\websites\volcanicos\controllers\login_Controlle r.php(18): Conexion->conectar() #2 C:\websites\volcanicos\index.php(64): login_users() #3 {main} Next exception 'Exception' with message 'Usuario y Clave Invalidas' in C:\websites\volcanicos\models\conexion.php:25 Stack trace: #0 C:\websites\volcanicos\controllers\login_Controlle r.php(18): Conexion->conectar() #1 C:\websites\volcanicos\index.php(64): login_users() #2 {main} thrown in C:\websites\volcanicos\models\conexion.php on line 25

linea 25

Código PHP:

throw new Exception("Usuario y Clave Invalidas", 0, $e);
 

[GatorV]

Claro, yo solo indicaba que lo que dice @abimex de cachar y volver a lanzar con otra excepción no es correcto, ya que pierdes la excepción original y la traza.

Lo correcto es (si vas a cachar/re-lanzar) es incluir la excepción original, así se hace más fácil el debug.

Respecto a tu error, pues es un error de diseño de tu aplicación, ya te indique que debes de comparar el resultado de tu función conectar antes de intentar enviar el objeto a otra clase, puedes usar un if() para comparar el resultado de tu función y en base a eso decidir si entras al segundo bloque o muestras un mensaje de error.

Saludos.

[Montes28]

GatorV gracias por ayudarme , y disculpa por mi poco conocimiento , pero intentando creo que seria algo asi

Código PHP:

public function conectar(){
 
                $usuario = $_POST['usuario'];
                $contrasenia = $_POST['clave'];
 
                
            try {
                 $db = new PDO('pgsql:host=localhost;dbname=volcanoes',$usuario,$contrasenia);
                 
                }catch(PDOException $e) {
                //echo $e->getMessage();
                echo "Usuario y Clave Invalidas";
                //throw new Exception("Usuario y Clave Invalidas", 0, $e);
                }
                
                if (empty($db)){
                    echo "vacio";
                    }else{
    
                return $db;
        }
        }
 

me corrigues por favor

gracias

saludos

[GatorV]

No, la validación tienes que hacerla antes de intentar usar el objeto en tus otras funciones. ¿Haz hecho el diagrama de flujo de tu sistema? Tienes que hacerlo antes de programar tu sistema, a grandes rasgos tiene que ser así:

- Inicio de tu Programa
- Recoges variables de entrada
- Validas usuario / contraseña
- ¿Es correcto?
+ Muestras Perfil
- Else
+ Muestras mensaje de error
- Fin

[GatorV]

En Pseudocódigo:

Código:

leer(usuario)
leer(pass)
if (loginUsuario(usuario, pass) {
       mostrarPerfil(usuario)
} else {
       mostrarError("usuario contraseña incorrectos")
}

[Montes28]

creo que tendria que realizar esta validacion en esta funcion pero me cuenta un poco realizarla

Código PHP:

function login_users()
    
    {
    
          require 'models/conexion.php';        
          require 'models/login_Model.php';
    
       if($_POST)
        {
             
 
              $lc=new Conexion();
              $db = $lc -> conectar();
               
                
                            
              $ls = new Login();
              $ls -> validate_users($db);
 
        }
       
        require 'views/login.php';
    }
 

[GatorV]

Así es, es en ese sitio donde debes de realizar la validación antes de intentar usar el objeto PDO.

Saludos.

[Montes28]

lo realize de la siguiente manera , funciona pero el problema es que muestra el error pero regreesa al formulario login ,me gustaria que solo quedara un mensaje de errror

Código PHP:

function login_users()
    
    {
    
          require 'models/conexion.php';        
          require 'models/login_Model.php';
    
       if($_POST)
        {
             
 
 
              $lc=new Conexion();
              $db = $lc -> conectar();
               
              if (empty($db)){
              
                            header("Refresh: 0; URL=/volcanicos/");
            die(); 
                }else{              
              $ls = new Login();
              $ls -> validate_users($db);
                }
        }
       
        require 'views/login.php';
    }
 

[GatorV]

Pues quita el header...