Vejamos o seu código:
public Objeto busca(String nome){
for(Objeto id : MinhaLista){
if(id.getNome().equals(nome)){
return null;
} else {
return id;
}
}
}
Quando ele começa a iterar o primeiro elemento, se o nome coincidir com o esperado, retorna null
, e quando não coincidir retorna este objeto.
Isto não está certo. Ele sempre retorna na primeira iteração! Além disso, ele só retorna o primeiro elemento se ele não for o que você procura. Se for o que você procura ele não retorna nada! Isso não tem sentido.
Acho que o que você queria é isso:
public Objeto busca(String nome){
for(Objeto id : MinhaLista){
if(id.getNome().equals(nome)){
// Achou o que queria, retorna.
return id;
}
}
return null; // Tentou todos e não achou nenhum, desiste.
}
Como sugerido pelo Felipe Fonseca em um comentário, vale a pena tratar os null
s e evitar NullPointerException
s:
public Objeto busca(String nome){
Objects.requireNonNull(nome, "O nome do objeto a buscar deve ser informado.");
for(Objeto id : MinhaLista){
if(nome.equals(id.getNome())){
// Achou o que queria, retorna.
return id;
}
}
return null; // Tentou todos e não achou nenhum, desiste.
}
Estou assumindo que MinhaLista
não contém elementos null
nela porque isso não costuma ter sentido. Se isso ocorre, você pode adicionar um if (id == null) continue;
antes deste outro if
que já está no código.
break
quando encontrar o resultado.busca
. Note que isto será executado apenas uma vez sempre. Na primeira interação, ele vai cair em um dos dois blocos doif
e em ambos existe umreturn
encerrando o método.