Estoy intentando adaptar una web que ya tenía más o menos hecha de forma estructurada a POO + MVC y me ha surgido la siguiente duda:
Pongamos el caso de un modelo y un controlador de usuarios, de manera que el modelo tiene un método guardar, que hace un INSERT
en la base de datos, y el controlador recibe los datos del usuario vía POST
desde un formulario, instancia un objeto del modelo y se los pasa; mi duda es: ¿por qué no pasarle los datos simplemente como parámetros del método guardar -que más o menos es lo que ya venía haciendo en mi web estructurada- en vez de recurrir a definir atributos, getters y setters? Es decir:
class UsuariosModelo
{
function guardar($nombre, $contrasena)
{
$sql = "INSERT INTO usuarios VALUES (NULL, $nombre, $contrasena)";
etc.
}
}
class UsuariosControlador
{
function guardar()
{
foreach($_POST as $clave => $valor) $$clave = $valor;
$usuario = new UsuariosModelo;
$resultados = $usuario->guardar($nombre, $contrasena);
etc.
}
}
De la forma ortodoxa habría que definir dentro del modelo atributos tanto para el nombre como para la contraseña y sus respectivos métodos get (a los que habría que llamar dentro del método guardar()
modelo) y set (a los que habría que llamar dentro del método guardar()
del controlador):
class UsuariosModelo
{
$nombre;
$contrasena;
function getNombre() {return $this->nombre};
function setNombre($nombre) {$this->nombre = $nombre};
function getContrasena() {return $this->contrasena};
function setContrasena($contrasena) {$this->contrasena = $contrasena};
function guardar()
{
$sql = "INSERT INTO usuarios VALUES (NULL, '{$this->getNombre()}', '{$this->getContrasena()}')";
etc.
}
}
class UsuariosControlador
{
function guardar()
{
foreach($_POST as $clave => $valor) $$clave = $valor;
$usuario = new UsuariosModelo;
$usuario->setNombre($nombre);
$usuario->setContrasena($contrasena);
$resultados = $usuario->guardar();
etc.
}
}
Es posible que el código tenga errores, porque lo acabo de improvisar, pero espero que se entienda la duda que planteo: ¿qué problemas -en cuanto a funcionalidad, seguridad, etc.- puede suponer hacer las cosas de manera "abreviada" más allá de que no se siga a rajatabla el paradigma POO?
Un saludo y gracias de antemano por vuestras respuestas.