Как сказали ранее, проблема находится в строке: for (int i = 0; i <= n; i++)
.
Для решения этой проблемы я бы подошел немного с другой стороны. Во время перебора коллекции можно использовать не сторонние переменные, которые могут быть ошибочными, а длину массива. Вы в первом for
использовали это.
К примеру, если n = 10
, а в массиве у Вас будет девять элементов, то Вы снова получите исключение. Вдруг переменная n
где-то поменялась?
Можно использовать Length
(для списков Count
). Вот так:
for (int index = 0; index < arr.Length; index++) {...} // в случае с массивом
for (int index = 0; index < lst.Count; index++) {...} // в случае со списком
Также очень удобен метод foreach, in для перебора коолекции, если Вам не нужны индексы, в таком случае Вам не нужно беспокоиться о знании количества элементов массива:
foreach (var item in array) {...}
Однако его не следует использовать для добавления или удаления элементов исходной коллекции во избежание непредвиденных побочных эффектов. Если нужно добавить или удалить элементы исходной коллекции, следует использовать цикл for.
С foreach
немного проще, так как он работает для массивов или коллекций объектов, реализующих интерфейс IEnumerable
или IEnumerable<T>
из System.Collections.Generic
.
Но и это еще не всё, что может облегчить работу программисту, также имеется метод Enumerable.Sum, который вычисляет сумму последовательности. И Вам не придется перебирать все элементы коллекции. К примеру:
double sum = a.Sum(); // сумма элементов Вашего массива
Также можно отметить метод Enumerable.Aggregate, который применяет к последовательности агрегатную функцию. К примеру, получение суммы будет таким:
double sum = a.Aggregate((result, item) => result + item);