¿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