[info]ru_java

ru.java

все о языке программирования java

Previous Entry в избранное рассказать другу Next Entry
Простенький вопросик
коровешка
[info]aefimov пишет в [info]ru_java
9 июля, 2008
Что будет в напечатано после запуска? Не заглядывайте под кат, попытайтесь ответить сначала.

public class TestIterator {
    public static void main(String[] arg) {
        List<String> list = new ArrayList<String>(Arrays.asList("a", "b"));
        Iterator<String> it = list.iterator();
        while (it.hasNext()) {
            String string = it.next();
            list.remove(string);
        }
        System.out.println("list = " + list);
    }
}

Сюрприз, разрывающий мозг. Никакого ConcurrentModificationException не будет. А напечатано будет list = [b]. Это java version "1.6.0_06".
Будьте бдительны, на коллекциях из двух элементов это работает так как не должно работать.
Метки:
(комментировать)
hasNext не проверяет модификацию
(Ответить) (Уровень выше) (Ветвь дискуссии)
Ну скажем так. Ведет он себя крайне странно. Посмотри что происходит с коллекций из одного и трех элементов.
(Ответить) (Уровень выше) (Ветвь дискуссии)
Если бы он не проверял модификацию, напечатано бы было list = [a, b]
(Ответить) (Уровень выше) (Ветвь дискуссии) (Expand)
— [info]earwin (Expand)
Fail-fast operations throw ConcurrentModificationException on a best-effort basis. Therefore, it would be wrong to write a program that depended on this exception for its correctness: ConcurrentModificationException should be used only to detect bugs.
+ дока на AbstractList.iterator()

никто в общем и не обещал этого эксепшна
(Ответить) (Ветвь дискуссии)
Тем не менее никто не также ожидал, что это зависит от количества элементов в коллекции.
(Ответить) (Уровень выше) (Ветвь дискуссии)
Скорее - от позиции того элемента на котором ты сделал модификацию.
Но если люди пишут "как повезёт", то они имеют полное легальное право вставить туда вызов рандома и чек на фазу луны :)
(Ответить) (Уровень выше) (Ветвь дискуссии) (Expand)
— [info]cgem (Expand)
— [info]earwin (Expand)
Любопытно, конечно, что hasNext() не проверяет modcount, но ничего разрывающего мозг, IMHO. Это не ошибка (итератор не обязан кидать CME), да и никто писать так не будет.

Надеюсь, вы не будете этот вопрос на интервью задавать :)
(Ответить) (Ветвь дискуссии)
Ну вообщем мы задавали (я задавал), конечно. Теперь буду задавать с оговорками. Все же надо спрашивать это, имхо.

А мозг взрывается от того, что это появляется только на коллекциях из двух элементов.
(Ответить) (Уровень выше) (Ветвь дискуссии) (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
— [info]earwin (Expand)
List<String> list = new ArrayList<String>(Arrays.asList("a", "b"));
Зачем new ArrayList<String>()? В исходниках jdk1.6.0_10:
This method also provides a convenient way to create a fixed-size list initialized to contain several elements: 
List<String> stooges = Arrays.asList("Larry", "Moe", "Curly");
(Ответить) (Ветвь дискуссии)
Arrays.asList вернет не модифицируемый лист.
(Ответить) (Уровень выше) (Ветвь дискуссии) (Expand)
jdk1.5.0_12 аналогичное поведение.
(Ответить) (Ветвь дискуссии)
а я вот не знал про Arrays.asList("a", "b"). теперь знаю, спасибо!
(Ответить) (Ветвь дискуссии)
Нельзя так делать. Удалять нужно через итератор.
(Ответить) (Ветвь дискуссии)
если писать на java 5 то объекта итератора доступно не будет :)

for (String aList : list) {
list.remove(aList);
}
(Ответить) (Уровень выше) (Ветвь дискуссии)
у думаю, что это нормально
вопрос, однако, возникает, почему мы до сих пор обсуждаем 1.6.0_06, когда на дворе уже _07
(Ответить) (Ветвь дискуссии)

как бэ вот

(Анонимно)

2008-07-09 14:52 (UTC)

http://bugs.sun.com/view_bug.do?bug_id=4902078
(Ответить) (Ветвь дискуссии)

Re: как бэ вот

[info]aefimov

2008-07-09 14:55 (UTC)

Closed, Will Not Fix, bug
Так я и думал. Бага это, всетаки.
(Ответить) (Уровень выше) (Ветвь дискуссии) (Expand)
— [info]earwin (Expand)
Should в том то и дело. Но тут говорят что это не баг.
(Ответить) (Уровень выше) (Ветвь дискуссии)
> Что будет в напечатано после запуска?
> Не заглядывайте под кат, попытайтесь ответить сначала.
Должно быть напечатано:
Compilation error:
while (it.hasNext()) {
String string = it.next();
list.remove(string); <--- КГ/АМ!!!
}
(-:

Шутка, но согласитесь - писать такое не надо.
И +1 к ораторам, считающим что это не баг.
(Ответить) (Ветвь дискуссии)
Отвечаю вам и ораторам. CME это защита от такого кода. Я например расчитывал, что если человек не знает что удалять надо через итератор, то в рантайме ему об этом сообщат этим Exception-ом. Теперь представьте, что это не всегда, и что если он всетаки так написал по неопытности, то найти такую ошибку будет крайне сложно.
(Ответить) (Уровень выше) (Ветвь дискуссии)
(Deleted comment)
(комментировать)
архив
Июль 2009
Пн
Вт
Ср
Чт
Пт
Сб
Вс
  12345
6789101112
13141516171819
20212223242526
2728293031  
Ссылки
Java Language Specification
Java Developers Almanac
Open Source Software in Java&trade;
JUG.RU
Метки
, , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , ,

Как вступить в сообщество?


Вступление в сообщество премодерируемое. Если вам отказали в вступлении, значит у вас пустой журнал, созданный совсем недавно или нам показалось, что ваш журнал ведут боты. Чтобы упростить вступление в сообщество достаточно показать заинтересованность в Java в своих интересах в профиле, иметь в друзьях сообщества, связанные с программированием. Иметь пост в своем журнале, например, про особенности Spring или Hibernate. В случаях, когда мы не можем понять, что этот журнал явный бот, мы разрешаем вступление, но если мы затем видим, что он ведет себя плохо, мы его баним и выкидываем из сообщества.
Вы также можете использовать этот пост, чтобы написать комментарий о том, что вас незаслуженно не пускают в сообщество, мы постараемся исправиться.

Как найти ответ на вопрос?


  1.  

  2. Заглянуть в Java Developers Almanac
  3. Заглянуть в Java Language Specification
  4. Посмотреть в Избранном


Правила публикации вакансий



  • Вакансии, публикуемые в сообществе, должны быть только на те позиции, где используется Java-технологии. Публикация вакансии с явным отклонением от темы Java-технологий приведёт к нажатию на кнопку «Удалить и пометить как спам».
  • Текст вакансии или рекламы должен быть оформлен тегом lj-cut. Обычно мы просим автора вакансии убрать вакансию под кат, если это не так. В случае отсутствия реакции в течение часа вакансия будет удалена кнопкой «Удалить и пометить как спам».
  • Если вы хотите опубликовать сразу несколько вакансий, оформляйте все вакансии в одной публикации, а не создавайте под каждую вакансию свою отдельную публикацию. Все "лишние" и повторные публикации будут удалены кнопкой «Удалить и пометить как спам».
  • Если вы публикуете вакансию, пожалуйста, указывайте город, в котором она открыта. Публикация вакансии без указания города в 100% случаев гарантирует вам кучу комментариев в почте с вопросом «какой город?».
  • Вакансии с указанием ограничений по возрасту, полу, семейному положению, национальности, вероисповеданию будут удаляться без предупреждения.
  • Вакансии с отключенными комментариями удаляются кнопкой «Удалить и пометить как спам».

Другие правила


  • Публикации должны явно указывать на Java или на технологии, основанные на Java. Про JavaScript пишем в [info]ru_javascript, про все остальное — куда угодно, только не сюда. Публикации, явно не про Java будут удаляться.
  • Большие участки кода должны быть оформлены тегом lj-cut.
  • Умышленное разведение флейма с указанием ссылки на свой сайт/блог не приветствуется. Проще говоря, любой пиар своего блога в виде спама типа такого будут отправлены в реактор кнопкой «Удалить и пометить как спам». Если надо дать ссылку на свой сайт, напишите описание в один абзац и дайте ссылку, иначе это будет расцениваться как спам.
  • Публикации с отключенными комментариями не приветствуется. Такие публикации будут удаляться.
Разработано LiveJournal.com