1

Estou fazendo um método de adicionar Usuários, e apenas uma coisa não está funcionando, a função que:

"Não será permitido adicionar amigos com o mesmo número de celular."

Ai precisa checar os usuários já adicionados, para ver se eles não tem um número de celular igual ao que for adicionado.
Ele está adicionando, mas também está adicionando os usuários com o número de celular igual. E não poderia adicionar se o número fosse igual

inserir a descrição da imagem aqui

 public void AdicionaAmigo(Usuario usr)
    {
       boolean existeEsteNum = false;
       if(ListadeAmigos == null || ListadeAmigos.length == 0) { 

         for(int i=0; i<ListadeAmigos.length; i++){
            if(ListadeAmigos[i].getCel() == usr.getCel()){
                existeEsteNum = true;
                System.out.println ("Já existe um usuário cadastrado com esse número de celular");
                break;
                }
         }
       } 

       if(!existeEsteNum){
           if(qtdeAmigos < ListadeAmigos.length)
           {
            ListadeAmigos[qtdeAmigos++] = usr;
            System.out.println ("Usuario adicionado com sucesso!");
           } else {
            System.out.println ("A lista está cheia");
           }
       }
    }
5
  • 2
    Por favor, adicione o código em forma de texto, para facilitar que seja testado.
    – user28595
    Commented 7/06/2017 às 13:45
  • E qual é o erro que retorna? Commented 7/06/2017 às 13:47
  • 1
    E informe o que não está funcionando. O que acontece? Em que circunstâncias acontece?
    – Maniero
    Commented 7/06/2017 às 13:47
  • Ele está adicionando, mas também está adicionando os usuários com o número de celular igual. E não poderia adicionar se o número fosse igual.
    – Daniel
    Commented 7/06/2017 às 13:49
  • A parte de verificar o celular não está funcionando, ele adiciona de qualquer maneira os usuários;
    – Daniel
    Commented 7/06/2017 às 13:50

2 Respostas 2

4

O que está errado neste método, é que você está percorrendo o "for" se a lista está vazia, logo não vai cair na condição nunca.

O que você deve fazer é revisar o primeiro IF o correto seria:

if ( ListadeAmigos != null && ListadeAmigos.length > 0 ) 
7
  • Acabei de tentar fazer assim, deu a mensagem de erro "java.lang.NullPointerException: null" e marcando em amarelo o if(ListadeAmigos[i].getCel() == usr.getCel()){
    – Daniel
    Commented 7/06/2017 às 13:54
  • Então veja se usr está nulo ou se tem alguma posição na lista que está nula. A resposta está correta.
    – Jéf Bueno
    Commented 7/06/2017 às 13:55
  • @Daniel, quando não se tem certeza se você possui valor preenchido, você nunca deve acessar direto, pois causará o nullPointerException, verifique antes de acessar o getCel() se o usr ou ListadeAmigos[i] não está nulo Commented 7/06/2017 às 13:58
  • Entendi a lógica. Mas como posso fazer?
    – Daniel
    Commented 7/06/2017 às 14:00
  • if ( ( usr != null && ListadeAmigos[i] != null) && ListadeAmigos[i].getCel() == usr.getCel() ) Commented 7/06/2017 às 14:02
0

Você pode trocar esse "==" no if ListadeAmigos[i].getCel() == usr.getCel()por equals(), creio que ficaria melhor, o "==" testa a referência, e não o valor. Essa seria a primeira coisa que vi logo de cara, vou dar mais umas olhadas e ver se encontro algo. Esse aqui é um exemplo e explicação dessas comparações

1
  • Obrigado pela dica!
    – Daniel
    Commented 7/06/2017 às 14:15

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 .