3

Есть класс Account обладающий списком ArrayList<Card> cardList. В цикле мне требуется искать подходящий объект класса Card, и при соблюдении требуемых параметров обновлять строку и обновлять сам объект класса Card из списка cardList.

for (int i = 0; i < cardList.size(); i++) {
 if (cardList.get(i).multi_parent != null && cardList.get(i).multi_parent.equals(multi_parentEntry)) {
     outputStr += cardList.get(i).balance;
     cardList.get(i).multi = true;
 }

}

Среда разработки Idea предупреждает, что 'for' loop replaceable with 'foreach'.

Поясните, пожалуйста, в чем тут проблема? Если я верно понимаю предупреждение, среда разработки предлагает использовать более компактный способ объявления цикла - foreach, но тут возникает вопрос - разве при объявлении цикла подобным образом

for (Card card: cardList) {
  if (card.multi_parent != null && card.multi_parent.equals(multi_parentEntry)) {
    outputStr += card.balance;
    card.multi = true;
  }
}

разве будет обновляться параметры объектов из списка (в моем случае каждый объект класса Card из списка cardList)?

4
  • 3
    а почему бы и нет? И она не предупреждает, а советует. Commented 5 февр. 2016 в 13:46
  • 1
    кроме того: в foreach запрещено изменение самой коллекции а не конкретного элемента.
    – Grundy
    Commented 5 февр. 2016 в 13:54
  • @Grundy а что Вы подразумеваете под изменением "самой коллекции"? Я правильно вас понимаю - имеется ввиду удаление/добавление элементов, например?
    – abbath0767
    Commented 5 февр. 2016 в 18:49
  • @mamba0767, да, именно это
    – Grundy
    Commented 5 февр. 2016 в 18:54

1 ответ 1

7

Возьмём следующий код:

ArrayList<Card> cardList;
for (Card card : cardList) {
    // . . .
    card.multi = true;
}

Фактически этот код эквивалентен следующему:

ArrayList<Card> cardList;
for (Iterator<Card> iter = cardList.iterator; iter.hasNext(); ) {
    Card card = iter.next();
    // . . . 
    card.multi = true;
}

На самом деле цикл foreach будет развёрнут в цикл с итераторами и foreach просто более короткая запись этого цикла. В коде с итераторами видно, что никаких копий элементов списка не создаётся, а в переменной card хранится ссылка на объект из списка. Поэтому изменение переменной card меняет элемент списка.

Тем самым, предложение IntelliJ IDEA абсолютно верно, предложенный ею код эквивалентен исходному.

1
  • @kiff Благодарю за развернутый ответ и пример с итератором, я, не знаю с чего, всегда считал будто с объекта, который мы используем, можно брать параметры, но изменяя его, мы не изменяем аналогичный объект, который взяли из списка, то есть мы работали с копией объекта из списка
    – abbath0767
    Commented 5 февр. 2016 в 18:46

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.