não é incorreto, mas existem maneiras que o próprio eloquent entrega:
- o seu primeiro exemplo
getPoduto
deveria ter um contexto melhor de nome talvez (estou imaginando que está usando o model "Produto") ... digamos peloPreco e então usar um scope em seu model:
public function peloPrecoScope($query, $preco = 100) {
return $query->where('preco', 100);
}
Desta forma você usaria assim:
$produto = Produto->peloPreco(200)->first(); // ou get - aqui estou pegando o primeiro registro apenas do retorno
E sim, o check provavelmente seria como você especificou.
Veja nada impede que use como descreveu, frequentemente tenho situações onde preciso que o retorno serja "duro" (hard coded) com um método get, mas frequentemente implementar a opção de filtros no modelo usando scopes é a saída que entrega mais reuso de código e melhor compreensão de leitura do mesmo.
Para entender melhor os scopes veja na documentação: Eloquent - Query scopes