como optimizar el siguiente codigo

[Montes28]

Les explico en que consiste la aplicación que estoy desarrollando:

Tengo 20 crud y los cree con Laravel-4-Generators, por lo tanto tengo 20 controladores, 20 modelos, y 20 directorios con sus respectivas vistas
Autenticacion de usuarios y permisos y estoy utilizando sentry.

Un requerimiento que me hicieron es que dependiendo del usuario tendrá acceso a diferentes datos.

entonces es acá donde veo que se me repite mucho el código, en cada método de cada controlador yo valido que usuario esta autenticado y dependiendo de su perfil y a la entidad a la que pertenece hago diferentes consultas a la base de datos.

cada crud tiene su entrada (osea pagina donde listo todos los registros de dicha entidad o tabla)y lo hago mediante este método

Código PHP:

//controlador
 
public function index()
    {
//obtengo el valor de la variable de sesion para obtener el perfil del usuario
$perfil =Session::get('perfil');
//obtener el usuario logeado
$user = Sentry::getUser();
//obtengo el un objeto al que pertenece el usuario logeado
$observatorio = Observatory::find($user->observatorio_id);
//en esta variable concateno el perfil y una cadena para saber a que entidad pertenece el usuario
$perfilObservatorio = $perfil.$observatorio->alias;
//este codigo lo hago en todos los metodos de los 20 controladores
 
//continua el switch 
 
            switch ($perfilObservatorio) {
            case 'Superadministradorovsm': //Superadministrador
            $stations = Station::with('volcan')->orderBy('id', 'Asc')->get();
            $submenuObservatorios = 'todos';
            break;
            case 'Coordinador Proyectoovsm'://Coordinador Proyecto OVSM
            case 'Coordinador Proyectoovsp'://Coordinador Proyecto Ovsp
            case 'Coordinador Proyectoovspop'://Coordinador Proyecto Ovspop
            $stations = Station::with('volcan')->orderBy('id', 'Asc')->get();
            $submenuObservatorios = 'todos';
            break;
            case 'Administrador Observatorioovsm': //Administrador Observatorio ovsm
            case 'Administrador Observatorioovsp': //Administrador Observatorio ovsp
            case 'Administrador Observatorioovspop': //Administrador Observatorio ovspop
            case 'Superadministradorovsp':
            case 'Superadministradorovspop':
            case 'Coordinador Observatorioovsm':
            case 'Coordinador Observatorioovsp':
            case 'Coordinador Observatorioovspop':
            case 'Tematicoovsm':
            case 'Tematicoovsp':
            case 'Tematicoovspop':                  
            $stations = Station::with('volcan')->where('observatorio_id', '=', $user->observatorio_id )->orderBy('id', 'Asc')->get();
            $submenuObservatorios ='uno';
            break;
            default:
            break;
            }
 
//y tengo que realizar estas consultas que no dependen del perfil o la entidad a la que pertenece el usuario
//estas consultas tambien las tengo que repetir en cada uno de los metodos de los 20 controladores
 
                        $transmisiones = Connectivity::all();
            $volcanes =Volcano::all();  
            $numeroEstaciones = Station::all()->count();
            $menu = 'menu.menu'.Session::get('perfil');
            $nombreModulo = 'Estaciones';
            $observatorios = Observatory::orderBy('id', 'Asc')->get();
            $areas = Area::where('monitoreo','=','TRUE')->get();
            $tiposEstaciones = Station_type::all(); 
            return View::make('stations.index', compact('tiposEstaciones','volcanes','transmisiones','areas','observatorio','observatorios','submenuObservatorios','perfilObservatorio','stations', 'nombreModulo','perfilObservatorio','menu'));
 
}
 

yo lo que pienso hacer es una clase que valide el usuario y le entregue al controlador dicho valor y retorno la vista. estoy bien o estoy mal?

que me recomiendan?

[hhs]

Vaya que tiene un problema. Una parte de tu problema es el uso de los CRUD en aplicaciones grandes crean muchas cosas que no necesitas. Esa parte la vas a tener que refactorizar para reducir el codigo que se debe de estar repitiendo en las vistas.

El uso de sentry lo vas a tener que integrar mas a Laravel. es decir usa sentry en combinación con Auth y Protección de rutas que supongo ya lo estas haciendo. La parte difícil es sacar todo lo que estas haciendo igual en los 20 controladores y para eso mi mejor recomendación es que crees la clase que comentas pero aprovecha el sistemas de eventos de laravel para sacar el código repetido. Con eso debes de reducir el tamaño de los controladores. Cuando tengas tiempo ve si puedes reducir el numero de controladores.

[Montes28]

hhs gracias por responde, podrias darme un ejemplo pequeño del sistema de eventos de laravel es que sinceramente no lo comprendo y lo peor no veo la utilidad.

los crud me han servido para separar las operaciones sobre cada tabla de la base de datos,ante los ojos del usuario el software tiene 20 modulos. un problema que veo es que a una vista le paso muchas objetos y colecciones

Código PHP:

return View::make('stations.index', compact('tiposEstaciones','volcanes','transmisiones','areas','observatorio','observatorios','submenuObservatorios','perfilObservatorio','stations', 'nombreModulo','perfilObservatorio','menu'));
 

con lo que me indicas de reducir el numero de controladores me pregunto no aumentaria el numero de cosas que envia a la vista?

[hhs]

Para ejemplos de eventos: http://www.craighooghiem.com/applica...aravel-events/.

Volviendo al tema, lo que necesitas hacer es mover el codigo que repites fuera del controlador las lineas 6-44 solo determinan detalles del usuario que ya debieras de saber cuando llega al controlador. Para lograr eso segun la documentación de Laravel puedes hacer uso de los filtros de esa forma puedes determinar el perfil, estaciones y el menú antes de que cargues el controlador correspondiente para detalles mira: http://laravel.com/docs/routing#route-filters para un ejemplo con sentry: http://laravelsnippets.com/snippets/...-route-filters.

En tu código no entiendo por que esta separado el perfil en una sesion el usuario por un lado y la entidad a la que pertenece por otro. No debieran de estar relacionados ? si ya tengo el usuario ya debiera de conocer su perfil y entidad. y no tener que determinarlos como lo haces en tu código.

Lo de los controladores no necesariamente. tu tienes 20 controladores y supongo que es uno por cada entidad de tu modelo debido al uso de los generadores de CRUD. Pero generalmente este tipo de herramientas te generan mucho código repetido y no reflejan el comportamiento real de los objetos de tu dominio.

[guardarmicorreo]

¿Pero no se podría haber hecho de otra manera más fácil? Pregunto, soy muy ignorante en cuanto a programación, pero creo que sí se podría haber hecho de otra manera mucho más fácil.

[hhs]

Iniciado por guardarmicorreo

¿Pero no se podría haber hecho de otra manera más fácil? Pregunto, soy muy ignorante en cuanto a programación, pero creo que sí se podría haber hecho de otra manera mucho más fácil.

Siempre existe otra forma de manejar la complejidad, pero eso no tiene nada que ver con la programación, si no con el campo del análisis y diseño de los sistemas. Puedes ser un buen programador y eso no significa que abordes un proyecto complejo y lo hagas bien si no tienes el conocimiento necesario para analizar y diseñar software.
Lamentablemente es una practica común ir primero por las lineas de código y eso es lo mas sencillo de hacer.