MVC options en controlador o vista

[h2swider]

Estoy usando una arquitectura propia basada en el modelo MVC, y me preguntaba que seria lo ideal

Guardar en el controlador una variable que contenga los options de un select,

Código PHP:

$vehiculoOption = '<option value="">Seleccione VehÃ*culo</option>';
/* @var $tipoVehiculo TipoVehiculo */
foreach ($tiposVehiculo as $tipoVehiculo) {
    $vehiculoOption .= '<option value="'.$tipoVehiculo->getIdTipoVehiculo().'">';
    $vehiculoOption .= $tipoVehiculo->getDTipoVehiculo().'</option>';
}
 

O recorrerlo en la vista.

Código PHP:

<option value="">Seleccione Veh&iacute;culo</option>
     <?php foreach ($tiposVehiculo as $tipoVehiculo) { ?>
         <option value="<?php echo $tipoVehiculo->getIdTipoVehiculo() ?>">
             <?php echo $tipoVehiculo->getDTipoVehiculo(); ?>
         </option>
<?php } ?>
 

[CesarHC]

Pues segun lo que yo conozco y uso en la vista solo van los datos tratados por el controlador es decir la salida y nada mas.

Y el controlador es el encargado de tratar el comportamiento de las acciones entre vista y modelo(Pues yo creo que por eso se llama controlador).

Yo recomendaria segun lo anterior que lo apliques en el controlador.

[h2swider]

Iniciado por CesarHC

Pues segun lo que yo conozco y uso en la vista solo van los datos tratados por el controlador es decir la salida y nada mas.

Y el controlador es el encargado de tratar el comportamiento de las acciones entre vista y modelo(Pues yo creo que por eso se llama controlador).

Yo recomendaria segun lo anterior que lo apliques en el controlador.

Las vistas pueden tener bucles o if para mostrar contenido, sin logica de negocio, de ahi surge mi duda, y no se que tan bien esta el hecho de concatenar html en una variable, nunca me pareció buena practica. Pero queria escuchar sus opiniones

[masterpuppet]

De las dos opciones que muestras me quedaría con la segunda, de todas formas en tu lugar utilizaría alguna suerte de view helper(o mejor aún, twig con un macro )

Saludos.

[abimex]

en el controlador no deberias tener nada de html, la 2da opcion es la correcta :)

[h2swider]

Iniciado por abimex

en el controlador no deberias tener nada de html, la 2da opcion es la correcta :)

Yo también voy mas por esta opción, algún argumento de las ventajas que me otorga?

[masterpuppet]

La ventaja seria que sigues las buenas practicas, ahora si quieres una ventaja "real" deberías tener un view helper/partial/macro o como lo quieras nombrar que te permita reutilizar.

[dashtrash]

Usando el MVC "típico", efectivamente, tendrás que usar un "snippet", "helper", "partial","twig",etc,etc.Personalmente, yo en esas cosas leo "hack","pseudohack","otro hack",etc,etc.

Una forma ligeramente mejor de hacerlo, es una mezcla de las dos opciones que propones.En el controlador, inicializar una fuente de datos, y pasar a la vista esa fuente de datos.La fuente de datos NO es un modelo.Una fuente de datos puede ser una query, o array, un fichero ini, o cualquier otra cosa.(Hay Datasources simples o compuestos, los simples pueden ser de acceso aleatorio o no,etc...Hay muchas formas de hacer esto).

Una forma que yo considero "elegante" de hacerlo, es pasarle a una factoría, el modelo, y el nombre de la columna del modelo que quieres editar.Con esa información, acceder a la metadata de la columna.Comprobar que es una relación con la tabla "TipoVehiculo", y que la forma por defecto de editar ese tipo de columna, es con un "select".Así que genera el select, a menos que la metadata de la columna especifique otra cosa.
Es decir, que la definición del campo "TipoVehiculo" en el modelo , es:

Código HTML:

$this->definition=array(
     ....
     "FIELDS"=>array(...
               "TipoVehiculo"=>array("Type"=>"Relationship",
                                                  "Table"=>"TiposVehiculo",
                                                  "RemoteField"=>"id_tipovehiculo",
                                                  "Multiplicity"=>"1:N",
                                                   "Required"=>true,
                                                   "Default"=>"NULL",
                                                   "Output"=>array(
                                                          "HTML"=>array(
                                                                     "EditAs"=>"Select"                                                                      
                                                                     )
                                                    )
                                       )
     .....
 

Si en este array cambias "Select" por "Radio", todos los formularios que editen este modelo
deberían cambiar su tipo de input.
Esta solución es DRY.Supón que, en vez de una relación, el tipo de dato es una cadena de caracteres.En la metadata del campo, puedes especificar la longitud máxima del campo.Leyendo esta metadata, puedes generar un input de tipo TEXT con ese número de caracteres.Cualquier framework que se autoproclame "DRY", pero que no sea capaz de llevar las restricciones de los tipos de datos de las columnas del modelo, desde la base de datos, hasta el javascript del navegador, no es DRY.
Pero bueno, ya me he ido mucho por las ramas