En los días pasados vi varias preguntas donde se cometían errores garrafalesgraves en el diseño de clases.
En al menos dos casos, se incluía como miembro de la clase un objeto que era la conexión a la base de datos. Mi argumento para desmontar ese error fue sencillo: ¿qué Ente
de una clase cualquiera nace con una conexión a la base de datos? - Ninguno . Entonces, ¿por qué dar a Ente
una propiedad que no les intrínseca? Cada vez que se creen instancias de Ente
, se estará creando también una conexión a la base de datos. Esto no es necesario, primero porque no es lógico y segundo porque en algunos contextos se necesitará solamente el modelo, sin tener que llenarlo necesariamente de datos obtenidos de la base de datos.
Otra cosa peor que se hacía era: dentro del constructor de la clase Ente
conectar conectar a la base de datos, hacer una consulta y llenar con cada columna obtenida las propiedades de la clase. O sea, meter en el modelo
En algunos casos eran clases relacionadas y toda una implementación de laesa lógica del programa que se cargabaen el modelo y fastidiaba todo. No digo nada si se tratase de clases dependientes unas de otrasconstructor complicaba las cosas.
Dicho esto: ¿existe una forma más fácil de mapear los resultados de una consulta a una clase, para evitar tener que crear una clase como esta:
class Actor {
private $actor_id;
private $actor_nombre;
private $actor_apellido;
private $actor_sexo;
private $last_update;
public function __construct($id)
{
$pdo = new PDO($dsn, $user_name, $pass_word);
$strSQL="SELECT * FROM actor_20171002 WHERE actor_id = ?";
$stmt = $pdo->prepare($strSQL);
$arrParams=array($id);
$stmt ->execute($arrParams);
while ( $row= $stmt->fetch(PDO::PDO::FETCH_OBJ) )
{
$this->actor_id=$row->actor_id;
$this->actor_nombre=$row->actor_nombre;
$this->actor_apellido=$row->actor_apellido;
$this->actor_sexo=$row->actor_sexo;
$this->last_update=$row->last_update;
}
}
public function getName()
{
return $this->actor_nombre;
}
public function getLastName()
{
return $this->actor_apellido;
}
public function getLastUpdate()
{
return new DateTime($this->last_update);
}
}