Olá pessoal, tudo bem?
Vi o código da aula 4 - Otimizando consultas e me deparei com esse código
public void validar(SolicitacaoAdocaoDto dto){
boolean petTemAdocaoEmAndamento = adocaoRepository
.existsByPetIdAndStatus(dto.idPet(), StatusAdocao.AGUARDANDO_AVALIACAO);
if(petTemAdocaoEmAndamento){
throw new Validacaoexception(...);
}
Quando vejo um método que retorna boolean para uma variável e depois um if com essa variável, sempre me dá uma vontade de tirar a variável e colocar o método direto no if, ex:
public void validar(SolicitacaoAdocaoDto dto){
if (adocaoRepository
.existsByPetIdAndStatus(dto.idPet(),
StatusAdocao.AGUARDANDO_AVALIACAO)){
throw new Validacaoexception(...);
}
Isso é considerado algo bom ou ruim, na minha cabeça estou tirando uma variável desnecessário e parece que consigo ler melhor assim.
Alguém conseguiria me responder se essa ação é uma boa prática ou apenas um vício que deveria ser evitado?
Agradeço desde já!!
