Hay varios errores en tu código, tanto en la praxis como en la lógica.
En la praxis
- Debes indicar el parámetro para
$id
en getPost()
- No estás ejecutando bien la consulta preparada. Los valores deben ser pasados como array en el
execute()
.
Corregimos esos dos puntos:
class Usuario extends Dbh {
public function getPost($id){
$sql = "SELECT * FROM usuario WHERE id = ?";
$stmt = $this->connect()->prepare($sql);
$stmt->execute(array($id));
$resultado = $stmt->fetch();
return $resultado;
}
}
En el contexto, tampoco es correcto el código. Debes hacer una asignación a la llamada a getPost()
para tener una referencia de lo que este método retorna, por ejemplo:
$id = 1;
$user = new Usuario();
$mUser=$user->getPost($id);
echo 'Hola '.$mUser['nombre'];
Con ambas cosas se resuelven los errores en la praxis.
Los errores lógicos son más delicados, pero me permito señalar algunos.
En la lógica
Lo primero que llama la atención es que tu clase Usuario
extienda de Dbh
, que supongo que es la clase de conexión. Esto es un grave error lógico. Una clase Usuario
podría extender de una clase Persona
por ejemplo, porque compartiría propiedades comunes como podría ser el nombre, apellido, etc. Hacer extender una clase de otra cosa utilitaria como es la conexión es un error de diseño.
Las clases que son de entidades (como Usuario
) deberías pensarlas como un molde para construir objetos de ese tipo, sin importar de dónde vengan los datos. Si mañana quieres usar esa clase para informarla con datos que vengan de un archivo de texto, un XML, un JSON, de Firebase, etc ... vas a estar arrastrando con la conexión en la clase sin necesidad de usarla.
Es un tema muy amplio que daría para mucho, sólo quiero llamar la atención sobre el error de lógica de querer hacer extender una clase entidad de una conexión a la base de datos que es más parte del contexto que de otra cosa.
Dicho eso, en getPost()
faltan controles. ¿Qué pasa si no hay conexión? ¿qué pasa si no hay filas? ¿qué pasa si hay error en la consulta?, etc. Nada de eso es controlado. Debes concebir tu programa como algo robusto, que prevea todos los posibles problemas, los controle y de una respuesta adecuada. Si no haces eso, tu código podría romperse en un determinado escenario, dejando al usuario con un crash o en el limbo.
Y, si vamos más lejos, habría que preguntarse, ¿qué hace un método llamado getPost()
en una clase llamada Usuario
? ¿No debería ese método ser parte de una clase Post
? Una de las grandes ventajas de la POO es que nos permite programar pensando las cosas como son en realidad. Un poco de metafísica no viene mal cuando se diseñan las clases y las relaciones entre ellas mediante la herencia.
getPost
hace un return de$resultado
. Por lo que nunca se asigna nada al objetoUsuario
.