No entiendo bien el propósito de
$conectar->base->prepare
Si tu intención es correr la consulta en la base de datos base
, al menos en PDO, el objeto de conexión no soporta esa sintaxis para cambiarse de base de datos, si es que estableciste la conexión sin especificar una. La manera correcta sería:
$conectar->exec('USE mi_base');
$conectar->prepare(...);
Eso puede ser el origen del problema. Ahora bien, no parece del todo lógico que tu constructor sea el encargado de manejar las variables $_POST. Yo en cambio haría que el constructor instanciara la conexión y añadiría un método para verificar la contraseña. Algo como:
class CheckUser {
private $_conn;
public function __construct() {
$this->_conn = new Conexion();
}
public function checkPassword($usuario, $password) {
$stmt = $this->_conn->prepare("SELECT contra FROM usuarios WHERE correo = :usuario;");
$stmt->bindValue(':usuario', $usuario, \PDO::PARAM_STR);
$stmt->execute();
$registros = $stmt->fetchAll(\PDO::FETCH_ASSOC);
if (count($registros) === 0) {
return self::noexiste();
}
$registro = $registros[0];
if (password_verify($password, $registro['contra'])) {
return self::correcta();
}
return self::incorrecta();
}
public static function correcta() {
echo 'Contraseña correcta';
}
public static function incorrecta() {
echo 'Contraseña incorrecta';
}
public static function noexiste() {
echo 'Usuario no existe';
}
}
Como verás, ese enfoque asume que el campo correo
es una llave única, y también contempla tres casos:
- usuario no existe en la BBDD
- usuario existe y su contraseña es correcta
- usuario existe y su contraseña es incorrecta
Y en la parte procedural de tu código, en donde manejas las variables del request, harías:
$userChecker = new CheckUser();
$userChecker->checkPassword($_POST['usuario'], $_POST['contra']);
Si estás usando otro driver (mysqli en particular) la sintaxis cambia, pero la estructura que te sugiero es la misma.
De este modo la clase está completamente desacoplada de las variables del request y podrás usarla para soportar otros métodos de autentificación: por $_GET (mala idea!), por variables de entorno (esa sí es buena opción), o por el payload en una petición en donde usuario y contraseña vengan en el body del objeto.
password_hash()
, ¿cierto?$this->registro['CONTRA']
, por lo tanto, puedes depurar lo que hay en esa variable mediantevar_dump($this->registro['CONTRA']);
; 2. Un problema de juego de caracteres. Para evitarlo, establece el charsetutf-8
a la conexión actual. Esto conviene hacerlo siempre, mejor en la claseConexion
que usas, aunque aquí, para fines de prueba, puedes hacerlo así:$conectar->exec("set names utf8");
Conexion()
? Qué base de datos usas y qué driver? Al ojo pareciera que estás usando PDO, pero no entiendo el uso de$conectar->base
. Si$conectar
es la conexión misma, debieras usar$conectar->prepare