O Laravel usa o padrão de active record, o que nos permite que a partir apenas da model consigamos persistir e consultar dados, sem a necessidade de uma camada extra para o banco especificamente.
Não vejo problema algum em chamar algumas queries pontuais como Model::find()
e Model::all()
, por exemplo.
O que talvez ela queria dizer é aqui :
$sql = Agenda::where('imovel_id', '=', $imovel)->where('data', '=', $dia)->first();
Usando este trecho de código como exemplo, talvez a professora tenha querido dizer que você deveria fazer isso na model, criando um método para retornar agendamentos por data num imóvel específico, por exemplo:
// na model
public function AgendamentosPorImovel($imovel_id, $data)
{
return $this->where('imovel_id', $imovel_id)
->where('data', $data)
->first();
}
Assim no controller você só teria que chamar esse método, eliminando assim a responsabilidade do seu controller de fazer queries.