1
public Objeto busca(String nome){
    for(Objeto id : MinhaLista){
        if(id.getNome().equals(nome)){
            return null;
        } else {                
            return id;
        }                   
    }
}

este é o codigo, ele está retornando o id ou null repetidamente. como faço pra parar o loop quando achar o resultado e só retornar apenas um resultado ao invés de vários?

2
  • Voce pode usar break quando encontrar o resultado. Commented 27/11/2014 às 1:40
  • 2
    Este código não faz sentido e se está repetindo o problema está na chamada do método busca. Note que isto será executado apenas uma vez sempre. Na primeira interação, ele vai cair em um dos dois blocos do if e em ambos existe um return encerrando o método.
    – Maniero
    Commented 27/11/2014 às 1:44

1 Resposta 1

3

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 nulls e evitar NullPointerExceptions:

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.

4
  • 2
    Vale adicionar um tratamento ao NullPointer no código para evitar exceções indesejadas Commented 27/11/2014 às 1:48
  • @FelipeFonseca Ok, editado. Muito obrigado. :) Commented 27/11/2014 às 1:52
  • sou grato pela ajuda. positivei. Commented 27/11/2014 às 1:56
  • @JoseMaximilian Valeu, obrigado! Também positivei você. :) Commented 27/11/2014 às 1:57

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .