Jawa / Говнокод #4792 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
for (int j = 0; j < fieldsToRemove.size(); j++) {
	if (fieldsToDelete.getField(j).getKind().equals("GroupField")) {
		resFieldContr.remove(j--);
	}
}

собственно цикл.
нашел в рабочем проекте

Запостил: tas tas, (Updated )

Комментарии (21) RSS

  • ava gegMOPO4 # Ссылка на оригинал 0

    Может и работает, но в одной конструкции обращаться к одному списку тремя разными способами -- это нужно уметь.
    Ответить

  • ava absolut # Ссылка на оригинал 0

    модификация переменной цикла в разных местах не есть гуд.
    Ответить

    • ava gegMOPO4 # Ссылка на оригинал 0 ⇈

      Это нормальная практика, когда в цикле по условию удаляют элементы.
      for (int i = 0; i < somelist.size(); i++)
    if(todelete(somelist.get(i)))
        somelist.remove(i--);

      Можно
      for (int i = 0; i < somelist.size();)
    if(todelete(somelist.get(i)))
        somelist.remove(i);
    else
        i++;

      но это длиннее и непривычно.
      Ну и для длинных списков и большого количества удалений имеет смысл избавиться от O(N^2):
      int j = 0;
for (int i = 0; i < somelist.size(); i++)
    if(!todelete(somelist.get(i)))
        somelist.set(j++, somelist.get(i));
// дальше усекаем до размера j
      Ответить

      • ava bugmenot # Ссылка на оригинал 0 ⇈

        --i++ и что в этом нормального
        с хвоста перебрать элементы религия не позволяет?
        Ответить

      • ava absolut # Ссылка на оригинал 0 ⇈

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

  • ava alt # Ссылка на оригинал 0

    А итератор никто не придумал?
    for (Iterator<BlaBlaBla> iterator = fieldsToRemove.iterator(); iterator.hasNext();) {
	if (iterator.next().getKind().equals("GroupField")) {
		iterator.remove();
	}
}
    Ответить

      • ava ArtPetroff # Ссылка на оригинал 0 ⇈

        Учим матчасть и идём запускать код руками.
        Итератор на то и имеет метод remove, чтобы можно было текущий элемент удалить.
        Ответить

          • ava ArtPetroff # Ссылка на оригинал 0 ⇈

            Назовите реализацию List'а, которая поддерживает метод remove, но не поддерживает remove у Iterator'a по этому List'у.
            Ответить

              • ava ArtPetroff # Ссылка на оригинал 0 ⇈

                LinkedList использует в качестве итератора внутренний класс ListItr, в котором метод remove вполне себе определён и ConcurrentModificationException кинет только если этот список поменял в процессе итерации кто-либо другой.

                И вообще, предлагаю запустить код.

                public void testArrayListIterator() {
        final List<String> list = new ArrayList<String>();
        list.add("1");
        list.add("2");
        list.add("1");
        for (Iterator<String> iterator = list.iterator(); iterator.hasNext();) {
            if ("1".equals(iterator.next())) {
                iterator.remove();
            }
        }
        assertEquals(1, list.size());
    }

    public void testLinkedListIterator() {
        final List<String> list = new LinkedList<String>();
        list.add("1");
        list.add("2");
        list.add("1");
        for (Iterator<String> iterator = list.iterator(); iterator.hasNext();) {
            if ("1".equals(iterator.next())) {
                iterator.remove();
            }
        }
        assertEquals(1, list.size());
    }
                Ответить

                • ava Анонимус # Ссылка на оригинал 0 ⇈

                  да, Вы оказались правы.
                  Удалять из итератора можно. Нельзя удалять напрямую из коллекции, по которой создан итератор.

                  я пальцем в лужу, бывает)
                  Ответить

Добавить комментарий

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?

    ↑
    8
    ↓