ContainerIterator

[Hyper]

Цитата:

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

The only situation in which you cannot avoid using the iterator is when you must remove elements from a List collection. The following code shows how this is accomplished.

List list = new List(Types::Integer);
ListIterator iterator;
;
list.addEnd(100);
list.addEnd(200);
list.addEnd(300);

iterator = new ListIterator(list);
while (iterator.more())
{
    if (iterator.value() == 200)
        iterator.delete();
    iterator.next();
}
print list.toString(); //{100, 300}
pause;

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

[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"

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