Ошибка по обращению к индексу массива?

Question

[P40b0s]

Добрый вечер, давненько я не задавал вопросы, но тут не совсем понимаю почему ошибка...
Index was out of range. Must be non-negative and less than the size of the collection. (Parameter 'index')

var runs = p.Elements<DocumentFormat.OpenXml.Wordprocessing.Run>()
                            .Reverse()
                            .Select(s=> new{run = s, props = wordPropertiesService.ExtractRunProperties(s, style) })
                            .ToList();
for (int i = runs.Count() - 1; i >= 0; i--)
                {
                    var currentRun = runs[i];
                    var txt = currentRun.run.Elements<Text>().FirstOrDefault();
                    var equals = runs.TakeWhile(t => 
                                     t.props.Equals(currentRun.props) 
                                     && (t.run.Elements<Drawing>().FirstOrDefault() == null 
                                      && t.run.Elements<Picture>().FirstOrDefault() == null)).ToList();
                        if(equals.Count > 2)
                        for(int e = equals.Count -2; e>= 0; e--)
                        {
                            if (equals[e].run.Elements<Text>().FirstOrDefault() != null)
                                txt.Text += equals[e].run.Elements<Text>().FirstOrDefault()?.Text ?? "";
                            runs.Remove(equals[e]);
                        }
                    //Собственно здесь после того как удалили из массива runs идентичные свойства
                   //в массиве runs остается 1 элемент (изначально было 4)
                   //index здесь равен 0
                        var index = runs.IndexOf(currentRun);
                       //удаляем из массива по индексу 0 1 элемент 
                      // ии получаем вышеприведенную ошибку....
                        runs.RemoveAt(index);
                }

Comments

[Василий Банников]

Видимо падает не на этой строке, либо вы что-то упустили.
Что будет, если по шагам выполнять?

[P40b0s]

Василий Банников, Да падает именно на этой... я в дебагере ловлю, причем на последнем элементе... 3 удаляет а на последнем ошибка....
И так та же ошибка:

if (equals.Count() == runs.Count())
    runs.Clear();

Ну и кстати runs.Clear() вообще не должно такое исключение кидать... но исключение ловится именно в этом методе...

[Василий Банников]

P40b0s, а вы точно не в релизе пытаетесь дебажить? Может вам строчки не те показывает?
Кажется пришло время отладки writeline ом

[P40b0s]

Василий Банников, Ну ошибка вроде нашлась, но она какая-то странная...
runs.RemoveAt(index);
Тут все удаляется корректно и runs.Count = 0, но итерация продолжается...
а при следующей итерации
for (int i = runs.Count() - 1; i >= 0; i--)
i почему-то равен 3... Ерунда какая-то
Получаетсяint i = runs.Count() он вызывает единожды а не при каждой итерации? Я почему то всю жизнь думал что он это количество доложен при каждой итерации обновлять.
Спасибо Василий! Подтолкнула ваша мысль что падает не на этой строке :)

Answer 1

[P40b0s]

В общем ошибка в том что элементы удаляются а значение счетчика i не меняется, все работает хорошо до тех пор, пока вам надо удалить только 1 элемент за итерацию, а если больше, тут и вылезает эта проблема.
Поэтому после каждого удаления элемента из массива надо значение i корректировать вручную.
Изменяем цикл на for (int i = runs.Count - 1; i >= 0;)
И после каждого успешного удаления подкручиваем счетчик.

if(runs.Remove(equals[e]))
   i--;

Comments

[Роман]

Итерировать коллекцию и удалять в ней элементы нехорошо. Лучше элементы копировать в новую и ее использовать.