Discussoes de Programacao e Tecnologia

Localizar posicao de um objeto em uma lista

8 respostas Resolvido
java
A
Adilson_Magalhaes

Salve galera, estou estudando listas e como usar class dentro de outras classes.
Tennho 3 classes no programa q estou criando.

  1. Products.
  2. Clients.
  3. Restaurante.

Dentro de cliente tem uma lista de produtos.
Dentro de restaurante tem uma lista de clientes.

Estou tentando salvar uma ordem dentro do cliente em específico, mas meu indexOf só retorna -1.

(OBS: Ta bem loouco isso, mas é de proposito para entender bem como as coisas funcionam.)

Restaurant r1 = new Restaurant();
r1.creatAnClient(new Client("Adilson", 555));

Aqui eu faço uma pesquisa pelo ID pra saber se cliente existe.

Integer hasId = clientList.indexOf(clientList.stream().filter(x -> x.getID() == id).findFirst().orElse(null));
            if (hasId != null ){
                clientList.get(0).shopping(order); 
                System.out.println("HAS ID: " + hasId)  // Aqui é teste é sempre retorna -1;
            }

clientList.get(0).shopping(order);
Colocando 0 ele adiciona certinho e imprimi os dados do cliente com os produtos.
Queria era passar hasId dentro do get() para ser adicionado dentro do cliente responsável pelo id.

Alura Desenvolvimento Back-End Java Sua Carreira em desenvolvimento back-end Java: dos fundamentos à arquitetura de sistemas...

8 Respostas

L
Lucas_Camara

Esse ID do cliente é de qual tipo?

A
Adilson_Magalhaes

Do tipo Integer.

Conseguir agora pq a IDE falou pra mudar o ==
ficou assim: x -> Objects.equals(x.getID(), id

Mas nao entendi muito bem o porque disso.

W
Solucao aceita
wldomiciano 2 likes

Como vc já está usando o filter(), não faz sentido estar usando o indexOf também.

Eu entendo que o indexOf vc usa quando vc já tem o objeto em mãos e só quer saber o indice dele dentro da lista. Já o filter() é o que vc precisa, pois vc não tem o objeto em mãos e precisa dele. O indice é irrelevante. Então ficaria algo assim:

import java.util.ArrayList;
import java.util.List;

public class Main {
  public static void main(String... args) {
    class Order {
    }

    class Cliente {
      private String nome;
      private int id;

      Cliente(String nome, int id) {
        this.nome = nome;
        this.id = id;
      }

      int getId() {
        return this.id;
      }

      void shopping(Order order) {
      }
    }

    Order order = new Order();

    List<Cliente> clientList = new ArrayList<>();
    clientList.add(new Cliente("aaa", 111));
    clientList.add(new Cliente("bbb", 222));
    clientList.add(new Cliente("ccc", 333));

    clientList // @formatter:off
      .stream()
      .filter(c -> c.getId() == 222)
      .findFirst()
      .ifPresent(c -> c.shopping(order));
  }
}

EDIT
Como é do tipo Integer, poderia ficar assim também:

.filter(c -> c.getId().equals(222))
A
Adilson_Magalhaes

uiii nunca vi esse .ifPresent(c -> c.shopping(order));
Vou testar aqui.

Eu conseguir da seguinte forma:

Main:

r1.creatAnClient(new Client("Adilson", 555));
    r1.creatAnClient(new Client("Adilson", 244));

    r1.makerAOrder(555, new Products("Pizza 30 cm", "Coca-Cola", true));
    r1.makerAOrder(244, new Products("Pizza 30 cm", "Coca-Cola", true));

Class:

Client hasId = clientList.stream().filter(x -> Objects.equals(x.getID(), id)).findFirst().orElse(null);
        Integer i = clientList.indexOf(hasId);
        if (hasId != null ){
            clientList.get(i).shopping(order);
        }

Vou testar da forma que tu falou.

W
wldomiciano 1 like

Que bom que conseguiu. Mas vc viu como vc tá fazendo 1 ação desnecessária?

Vc já tem o Client, não precisa mais saber o indice. Poderia ficar assim:

Client hasId = clientList.stream().filter(x -> Objects.equals(x.getID(), id)).findFirst().orElse(null);
if (hasId != null ){
    hasId.shopping(order);
}
A
Adilson_Magalhaes

Dessa forma: Objects.equals(x.getID(), id) eu conseguir pq a IDE deu erro. Mas nao entendi muito bem desse codigo.

Conseguir da forma que voce falou, achei mais pratico, mas ele so funciona se for com equal, se for com == nao funciona.

Vlllw demais, ja aprendi alguma coisa.

W
wldomiciano 1 like

É. Como vc tá usando Integer ao invés de int, o correto é usar equals() mesmo. Eu tinha sugerido isso antes de saber que era Integer.

Sobre o Objects.equals(), do meu ponto de vista, esta é a vantagem de usar:

Digamos que vc queira comparar 2 Integer, X e Y, e vc sabe com toda certeza do mundo que nenhum deles é null.

Integer X = 1;
Integer Y = 2;

Os 2 modos abaixo são equivalentes:

// Modo 1
System.out.println(Objects.equals(X, Y));

// Modo 2
System.out.println(X.equals(Y));

Mas digamos que haja a possibilidade de X ser null. Para evitar um NullPointerException vc teria que adicionar um if:

if (X != null) {
    X.equals(Y);
}

Mas usando Objects.equals() vc não precisa deste if porque ele já faz esta checagem por vc internamente, veja:

Não sei se tem mais vantagens, foi só essa que eu consegui observar até o momento.

A
Adilson_Magalhaes 1 like

Entendi. Entao nesse caso eu poderia usar o Objects.equals() para caso nao fosse passado o id ou fosse colocado null de proposito. Assim ele trataria esse null diferente do x.iguals(y).

Criado 29 de outubro de 2021
Ultima resposta 29 de out. de 2021
Respostas 8
Participantes 3

Topicos relacionados

JPA - Assumir valor default do banco 1 respostas Como verificar se um EditText esta vazio 1 respostas [RESOLVIDO]Manipulação de horas e minutos Java + Postgres Resolvido 3 respostas Como fazer "Select" vindo do banco de dados usando thymeleaf? Resolvido 4 respostas EXCEPTION_STACK_OVERFLOW - Problematic frame: nio.dll+0x7407 2 respostas
Alura POO: o que é programação orientada a objetos? Aprenda os conceitos básicos da programação orientada a objetos, como classes, objetos, herança, encapsulamento e polimorfismo, com exemplos.
Casa do Codigo Desbravando SOLID: Praticas avancadas para codigos de... Por Alexandre Aquiles — Casa do Codigo