duda entre modelo y controlador

[destor77]

Hola,
tengo que armar un modelo y un controlador para crear galerias de fotos y subir fotos. Así que estoy armando un modelo para las galerias y otro para las fotos, y desde el controlador "galerias" uso ambos modelos.
La duda que tengo es cuando tengo que hacer las validaciones respecto al nombre de las galerias y las imagenes, para eso uso una clase llamada utilidades que contiene un método que reemplaza el nombre que se pasa por uno que sea valido para ser un directorio o un nombre de archivo (espacios, tildes, etc). Bien la consulta es esta validacion la tengo que hacer en el controlador o se puede hacer en el modelo? Yo la estaba haciendo en el modelo, pero para usar validar los datos tengo que instanciar la clase utilidades y la de imagenes eso me parece medio raro..... y no me convence.

Desde ya muchas gracias

[GatorV]

Es mejor hacerlo en el controlador para que pases los datos ya filtrados a ambos modelos. Aunque no hay una limitación o restricción de hacerlo en el modelo y que estos interactuen.

Saludos.

[destor77]

muchas gracias gatorV, como siempre "salvando las papas" a la gente jejej.

[maturano]

Iniciado por destor77

Bien la consulta es esta validacion la tengo que hacer en el controlador o se puede hacer en el modelo? Yo la estaba haciendo en el modelo, pero para usar validar los datos tengo que instanciar la clase utilidades y la de imagenes eso me parece medio raro..... y no me convence.

En tu modelo, los datos provienen del sistema de archivos. La validación que se haga sobre ese sistema de archivos es lógica de tu modelo; para tener un código reusable deberías hacer esa validación ahí y no en el controlador. Que no te convenza quizá será cuestión de tu arquitectura, pero tú querrás agarrar un modelo y tener la confianza que la información que ingreses y/o recuperes funcionará y-ya.

Una simple acrónimo: DRY

Dependiendo de tu aplicación: ¿Qué pasará cuando quieras utilizar el mismo modelo en otro controlador?, ¿Y en otro módulo?, ¿Mismo controlador y diferentes acciones?, ¿vas a estar haciendo la misma validación una y otra y otra vez?. Entonces, ¿es lógica del controlador o del modelo?; Pregunto

[destor77]

Iniciado por maturano

En tu modelo, los datos provienen del sistema de archivos. La validación que se haga sobre ese sistema de archivos es lógica de tu modelo; para tener un código reusable deberías hacer esa validación ahí y no en el controlador. Que no te convenza quizá será cuestión de tu arquitectura, pero tú querrás agarrar un modelo y tener la confianza que la información que ingreses y/o recuperes funcionará y-ya.

En realidad las validaciones de los nombres, como las clases que uso para crear las imágenes, son clases independientes que las instancio en mi controlador base precisamente para eso para ser reutilizadas en cualquier lugar del sistema.. Por eso cuando llegue al momento de validar los datos, me encontré con esa duda, si usar los métodos de esas clases en el modelo o en el controlador.

salu2

[maturano]

No entiendo, ¿tu preocupación es porque tienes una instancia de la clase?.

Lo que digo es, si es lógica de tu modelo (cosa que no sé, no haz colocado ejemplos como para saber) debería ir en tu modelo. No tiene caso hacer lo mismo una y otra y otra vez en cada controlador/acción/módulo donde utilices ese modelo.

Uno de las ventajas de la orientación a objetos es su mantenibilidad. ¿Qué pasa si hay un error en esa validación?; utilizando OO deberías --en teoría -- solo modificar un método, atender solo una parte bien identificada de tu código y la solución debería "expandirse" a cualquier cosa que utilice esa misma validación. Pregunto: ¿con la validación en tu controlador va a suceder así o tendrías que buscar todos los lugares donde hiciste lo mismo?.

Que utilizas alguna clase genérica para hacer esa validación, ¿y?. Tú deberías traer tu modelo y confiar en que funciona, no pensar en cómo hace su trabajo, bien puede utilizar una "clase genérica", código plano y duro, algún servicio web, llamadas al sistema, lo que sea. Ese es el concepto de encapsulación: Tú no deberías pensar en cómo hace lo que hace, solo asegurarte que lo haga bien. Si el asunto es que ya hay una instancia de una clase, para eso está Singleton, o podría ser que en realidad tiene que ser un método estático y no tendría por qué instanciar el objeto.

Y no, no digo que la validación DEBE ir en tu modelo . Puede ir en uno u otro lugar, solo planteo diferentes escenarios ...

[NUCKLEAR]

Quizas es el error más frecuente(que muchas veces pasamos por alto).
Tampoco soy un fundamentalista del estandar, pero un codigo limpio y bien ordenado no solo te ayudara a ti, si no a cualquieera que lea tu codigo.

Es simple:

Controlador "hace de pasarela entre el modelo y la vista"
Modelo Agrupal la logica del programa, recibe la orden la procesa y envia la respuesta al controlador
Vista recibe y muestra lo que el controlador le envia.

Siguiendo este patron:
Por lógica el proceso de validación lo debe hacer tu modelo y devolver una respuesta.

Pero nada impide hacerlo en cualquier lugar, es cuestion de tu comodidad, pero algo bien ordenado es mas facil de encontrar.

Saludos.

[destor77]

maturano y nucklear,
muchas gracias por sus respuestas, ya había leído que las validaciones pueden ir en el controlador o el modelo. Sólo que he visto en algunos framework que el modelo solo tiene en algunos la tabla a la que hacen referencia y nada más (porque usan algun activeRecord) y en otros solo los metodos pero sin validación. Yo como dije lo estaba intentando validar como creo las galerías y subo las fotos desde el modelo, pero no me pareció "limpio" que en el modelo este usando clases para subir imágenes, ni haciendo validaciones para ver si la carpeta se creo correctamente. Por eso decidí hacer todo eso desde el controlador y dejar los modelos para los abm en la base de datos, nose si esta bien o no, pero sin llegar a ser un "fundamentalista" me pareció "mas limpio" jeje.
Gracias
Salu2

[GatorV]

Es por eso la razón por la cual Zend separo lo que es el DbTable de lo que es el modelo y en el Quickstart te animan a usar un DataMapper, la idea es que tu modelo sea muy rico en validaciones y este modelo se comunique internamente con el DbTable y cree un row con el ActiveRecord para salvar los datos, pero así dejas el modelo con todas sus validaciones y lo puedes instanciar desde cualquier controlador y esperar el mismo resultado solo llamando a $modelo->guardar() por ejemplo.

Saludos.

[destor77]

Entiendo Gatorv, es mas en la empresa donde trabajaba hasta hace unos meses, tenían algo parecido implementado a lo de $modelo->guardar(), pero ellos en el modelo solo hacía un:

Código PHP:

class mimodelo extends model{
protect $table="el nombre de la tabla del modelo";
}
 

y despues desde el controlador se llamaba asi:

Código PHP:

class micontrolador extends controller{
private $mimodelo
 
function __construct(){
$this->mimodelo = new mimodelo();
}
 
public function algo(){
  $busqueda=$this->mimodelo->find(1);
  $salva= $this->midemodelo->save();
}
 

Todas la validaciones de tipo de campo en la db (si era string, date, etc) la hacia en el acitverecord, pero las validaciones como la que yo estoy intentando hacer las hacia desde el controlador.

[webness]

>En casos como esto, yo hago un controlador generico con metodos que se que puedo usar una y otra ves en mas controladores, por la tanto estos nuevos controladores heredan de la clase controladora hecha al inicio.

[GatorV]

Iniciado por destor77

Entiendo Gatorv, es mas en la empresa donde trabajaba hasta hace unos meses, tenían algo parecido implementado a lo de $modelo->guardar(), pero ellos en el modelo solo hacía un:

Código PHP:

class mimodelo extends model{
protect $table="el nombre de la tabla del modelo";
}
 

y despues desde el controlador se llamaba asi:

Código PHP:

class micontrolador extends controller{
private $mimodelo
 
function __construct(){
$this->mimodelo = new mimodelo();
}
 
public function algo(){
  $busqueda=$this->mimodelo->find(1);
  $salva= $this->midemodelo->save();
}
 

Todas la validaciones de tipo de campo en la db (si era string, date, etc) la hacia en el acitverecord, pero las validaciones como la que yo estoy intentando hacer las hacia desde el controlador.

No de hecho si ves el Quickstart de ZF, puedes ver que lo que es el DataTable (como el primer ejemplo) esta integrado al modelo pero no llega al controlador, lo que llega al controlador es el modelo como tal.

Saludos.