Petter, se a lógica da consulta vai repetir, não tem por que encher "a parada" de if
. Recomendo que você simplifique seu código encapsulando a lógica da consulta dentro de um loop.
Por exemplo, você quer que os campos primeiro_nome
, email
e sobrenome
sejam pesquisados com LIKE
e nivel_id
, idade
e curso
sejam pesquisados com o operador =
.
Você não precisaria fazer vários ifs
, mas poderia fazer um foreach e aplicar a lógica lá dentro.
$callback = function ($query) use($request) {
$likes = ['primeiro_nome', 'email', 'sobrenome'];
foreach ($request->only($campos_like) as $name => $value)
{
$value && $query->where($name, 'LIKE', "%{$value}%");
}
$equals = ['nivel_id', 'idade', 'curso'];
foreach ($request->only($campos_equal) as $name => $value) {
$value && $query->where([$name => $value]);
}
};
Usuario::orderBy('primeiro_nome')->where($calback)->paginate(15)->appends($request->all());
Observe que não é necessário encher o código de if
. Se você puder simplificar, sempre simplifique.
Talvez você esteja se perguntando: "Mas aonde estão os ifs"?
Se observar bem, eu substituí o if
pela expressão $value && $query->where(...)
. Costumo fazer isso quando quero simplificar um if
. Mas não vou entrar em muitos detalhes, pois já respondi isso antes :)
Refatorar vale a pena se a alma não é pequena!

if($outro)
? Nessa estrutura que mostrou porque não faz comelse if..
, não a necessidade de fazer as restantes verificações depois de uma retornarTrue
...$consulta
vai ser sempre a ultima condição em que entra