ContainerIterator

[Gustav]

Цитата:

Сообщение от Hyper

В коде логическая ошибка. Перед iterator.next() необходим else.

На самом деле результат одинаков и так, и так (по крайней мере в 3.0 SP4). Думаю, что класс ListIterator всё сам корректно обрабатывает и не делает казалось бы лишний next для уже удаленного элемента.

Что касается визуального восприятия кода, то добавление else, на мой взгляд, затуманит наглядность, так как глаз привык к шаблону вида:

while (iterator.more())
{
    // bla-bla-bla

    iterator.next();
}

А пряча продвижение указателя за else, можно потерять уверенность, что будут обойдены все элементы коллекции.
Но в любом случае все претензии по данному фрагменту - авторам "Inside"

[Hyper]

Цитата:

Сообщение от Gustav

Думаю, что класс ListIterator всё сам корректно обрабатывает и не делает казалось бы лишний next для уже удаленного элемента.

В том-то и дело, что делает; это достаточно легко проверяется. AX 4.0.

В приведенном примере результат, разумеется, одинаков, но с итератором, стоящим на 300, в цикл мы никогда не попадаем.

Цитата:

Но в любом случае все претензии по данному фрагменту - авторам "Inside"

Ну да. Это я так, на заметку.

[plumbum]

Цитата:

Сообщение от belugin

Для пуристов сожно сделать так:

class ContainerCollection
{  
    container contents;
    Enumerator getEnumerator();
}

Можно перекрыть класс ContainerClass и добавить метод getEnumerator()

public class MyContainerClass extends ContainerClass
{
    ContainerCollection enumerator;
}

public ContainerCollection getEnumerator()
{
    if (!enumerator)
        enumerator = new ContainerCollection(this.value());    
    return enumerator;
}