CodeIgniter ¿Es recomendable un método que haga muchas cosas en un modelo?

[nanotime]

La verdad es que tengo esta duda de "buenas costumbres".

Estaba limpiando algo de código de un modelo y hasta este punto no había presentado problema alguno, pero cuando llegué a los métodos que hacen consultas a las bases de datos me di cuenta de que son redundantes, osea, tengo un método para pedir todos los items de una bd, luego uno para pedirlos por su categoría y otros por su id o su slug.

El código se ve mas o menos así en esa sección:

Código PHP:

public function get_curso($id){
        $rs = $this->db->get_where('cursos', array('id' => $id));
            
        if($rs->num_rows() > 0){
            return $rs->row_array();
        }
        else{
            return false;
        }
    }
 
        public function get_categoria($cat)
    {
        $rs = $this->db->get_where('cursos', array('categoria' => $cat));
 
        if ($rs->num_rows() > 0) {
            return $rs;
        }
        else{
            return false;
        }
    }
 

Básicamente es eso cuando digo que son muchas funciones para mas o menos lo mismo, de hecho hasta se repite código ... la cosa es que he estado tratando de tener una sola función "get_curso" que devuelva determinada consulta luego de recibido un parámetro y que, si ninguno de estos se le pasa, simplemente devuelva la consulta general de la base de datos (para el dashboard).

¿Código? Este:

Código PHP:

public function get($id = false, $slug = false)
    {
        if (!$id || !$slug) {
            $query = $this->db->get('cursos');
            return $query->result_array();
        }
        elseif ($slug) {
            $query = $this->db->get_where('cursos', array('slug' => $slug));
            return $query->row_array();
        }
        else {
            $query = $this->db->get_where('cursos', array('id' => $id));
            return $query->row_array();
        }
    }
 

Es un borrador, y de hecho me da problemas, no de sintaxis sino en las vistas y el controlador, porque los errores que me lanza son justamente de "no existe la variabe tal", simplemente no consigue nada pero al parecer funciona porque me pinta la vista, solo que me da errores en el contenido.

¿Es buena idea o costumbre tener un método que haga tantas cosas? ¿O es mejor separar cada consulta con un método?

Gracias por adelantado

[Lautaro_eb]

Buenas, el tener funciones separadas casi idénticas puede agilizar el trabajo aunque tu método también puede ser conveniente depende de cada programador creo yo

[nanotime]

Sí, de hecho acabo de terminar lo que sería el método que responde a distintos parámetros (en teoría) y bueno, la verdad es que funciona a medias.

Código PHP:

public function get($id = false, $slug = false, $categ = false)
    {
        if (!$id && !$slug && $categ) {
            $query = $this->db->get('cursos');
            return $query->result_array();
        }
        elseif ($slug) {
            $query = $this->db->get_where('cursos', array('slug' => $slug));
            
            if ($query->num_rows() == 1) {
                return $query->row_array();
            }
            else {
                return false;
            }
        }
        elseif ($categ) {
            $query = $this->db->get_where('cursos', array('categoria' => $categ));
 
            if ($query->num_rows() == 1) {
                return $query->row_array();
            }
            else {
                return false;
            }
        }
        elseif ($id) {
            $query = $this->db->get_where('cursos', array('id' => $id));
            return $query->row_array();
         }
    }
 

Técnicamente lo único que funciona es que mientras que sean false los parámetros, devuelve la consulta de todos los items de la base de datos y que cuando se le pase el id devuelva el id de la consulta, pero las categorías y el slug ... hmmm no dan señales de vida ...

Respecto a que depende de cada programador, sí, supongo que si pero a veces me pregunto si no termina siendo mas confuso tener get_curso_id, get_curso_categ, get_curso_slug, etc,etc ... quizá para este proyecto pequeño funcione por ser pocas consultas pero es que el código para cada una es prácticamente idéntico y estaría rompiendo el paradigma DRY, que no es que sea pecado pero no sé si vaya a crecer el código, sin mencionar que estos mismos métodos se usan para un servicio rest.

[cachusan]

Una cuestión a tener en cuenta es cuando queres encadenar métodos, si los tenes todos juntos no lo podrás hacer. De todas maneras, todo depende del tipo de projecto.

[dashtrash]

Esos métodos están manejando instancias de modelos, o simplemente filas de la base de datos?
Supongo que son métodos que tienes en un modelo..Cuál es ese modelo? Es es modelo "cursos"? Si lo es, y una instancia de la clase "cursos" representa 1 cierto curso:
- Para obtener los cursos de una categoría, necesito una instancia de curso?
- Cuando se llama a get_categoria (que creo que se debería llamar getCoursesByCategory), se obtienen modelos,o simples filas de la bd?

En general, los sistemas MVC dejan de lado los datasources, y los "incrustan" dentro de los modelos.Pero los modelos son 1 cierta instancia.Que algo que modela 1 instancia de "Curso", también sepa obtenerlos, según distintos criterios, es hacer que una clase haga 2 papeles.De instancia de un objeto de negocio, y de Factoría de esos objetos.

[bet7o]

Buenas,

Yo me quedo con la primera versión, métodos que hagan cosas especificas y sean re utilizables, también como dicen cambiaría el nombre de los modelos para que fuesen mas descriptivos.

PSR-1 codificacion estandar basica

Imagina si vas a utilizar el método "get" 1000 veces en todo tu código, cuantos if elseif deberá recorrer para encontrar el correcto? sin duda algo que pegara en el performance.


Saludos.