1

Estou tendo uma dificuldade em encontrar o erro em uma lógica que utilizo o for para percorrer um arraylist.

  List<Caneta> canetas = new ArrayList<>(); 
  canetas.add(c1);
  canetas.add(c2);

  System.out.print(Arrays.toString(canetas.toArray()));

  for (Caneta caneta : canetas) {
    int x = 0;
    System.out.printf(canetas.get(x).getModelo());
    x++; 
  }

O sout me retorna a mesma resposta, como se o x não tivesse se autoincrementado, alguém poderia me explicar o problema? Grato.

3
  • Mostre o código da criação do c1 e c2
    – Sorack
    26/09/2017 às 19:35
  • 3
    Você deve declarar o x fora do loop for, do jeito que tá ele é zerado a cada iteração
    – Math
    26/09/2017 às 19:35
  • Para percorrer a lista com for each não é necessário contador, a cada laço o objeto já é incrementado. Na tradução for each é 'para cada'. 27/09/2017 às 0:06

2 Respostas 2

5

O x está sendo inicializado dentro do foreach. Desta forma, a cada iteração (loop) ele é inicializado com o valor 0.

A solução seria declarar o x fora do loop.

int x = 0;
for (Caneta caneta : canetas) {
    System.out.printf(canetas.get(x).getModelo());
    x++; 
}

Apesar do código acima estar certo, ele não faz muito sentido. O foreach é usado para "passar" por todos os elementos de uma coleção e, a cada loop, alimentar a variável (caneta, no caso) com um item da lista.

Então, se você precisa usar o get(x), seria melhor usar um for.

for(int x = 0; i < canetas.size(), x++) {
    System.out.printf(canetas.get(x).getModelo());
}

Ou, se quiser continuar usando o "foreach", deveria fazer assim

for(Caneta caneta : canetas){
     System.out.printf(caneta.getModelo());
}

Talvez você se interesse por ler esta pergunta: Quais são as sintaxes do laço for em java?

0
1

Tenta desta forma:

  List<Caneta> canetas = new ArrayList<>(); 
  canetas.add(c1);
  canetas.add(c2);

  System.out.print(Arrays.toString(canetas.toArray()));

  for (Caneta caneta : canetas) {
    System.out.printf(caneta.getModelo());
  }
2
  • 1
    Apesar de você ter utilizado de fato o foreach, você manteve aquele x do mesmo jeito. Seria bom dar uma explicação mínima sobre o funcionamento do laço foreach para o OP 26/09/2017 às 19:40
  • Eu posso ter mantido, mas o meu vai percorrer a lista.
    – Dev
    26/09/2017 às 19:41

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 .