1

Почему выводит, что индекс вне границ.
Необходимо вывести сумму всех элементов массива.

static void Main(string[] args)
{
    Console.WriteLine("введите кол-во значений в массиве:");
    int n = int.Parse(Console.ReadLine());
    double[] a = new double[n];
    Random rnd = new Random();
    for (int i = 0; i < a.Length; i++)
    {
        a[i] = rnd.Next(1, 100);
        Console.Write("{0: 0}", a[i]);
    }
    Console.WriteLine();
    double max = a.Max();
    Console.WriteLine("Максимальное:{0}", max);
    double min = a.Min();
    Console.WriteLine("Минимальное:{0}", min);
    double sum = 0;
    for (int i = 0; i <= n; i++)
    {
      sum += a[i];
    }
    Console.WriteLine("Сумма м/у мах и мin:{0} ", sum);
    Console.ReadLine();
}

В результате выполнения кода получаю ошибку на строке: sum += a[i];

3
  • 3
    Должно быть i < n;
    – koks_rs
    20 ноя 2016 в 15:32
  • 2
    Если вам дан исчерпывающий ответ, отметьте его как верный (галка напротив выбранного ответа). 21 ноя 2016 в 6:00
  • Трассируйте для n=1 или 2 - и всё будет понятно:)
    – nick_n_a
    21 ноя 2016 в 14:05

3 ответа 3

7
for (int i = 0; i <= n; i++)
{
    sum += a[i];
}

замените на

for (int i = 0; i < n; i++)
{
    sum += a[i];
}
2
  • Огромное Вам спасибо! а я все другое исправлял..
    – roy
    20 ноя 2016 в 15:34
  • @roy34 отблагодарите человека не только на словах, но и на деле - поставьте галку напротив этого ответа и вопрос закроете, и человеку баллов добавите.
    – Bulson
    20 ноя 2016 в 16:38
7

Причина ошибки состоит в том, что вы пытаетесь использовать индексы там, где это не требуется. В результате когда вы делаете это самостоятельно, вы можете допустить неточность или опечатку.

Так как вы создали массив, содержащий n элементов, то допустимый диапазон индексов [0, n-1]

       for (int i = 0; i <= n; i++)
                       ^^^^^^^
       {
          sum += a[i];
       }

поэтому лучше было бы написать

foreach ( var x in a ) sum += x;

не ссылаясь на индексы.:)

2

Как сказали ранее, проблема находится в строке: 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);
3
  • foreach неудобен, если нужны индексы
    – Grundy
    21 ноя 2016 в 13:14
  • @Grundy, бесспорно. Но это как альтернатива, когда они не нужны и нужно перебрать коллекцию. Так-то он очень удобен. 21 ноя 2016 в 13:15
  • до кучи, можно еще про Aggregate пару слов сказать
    – Grundy
    21 ноя 2016 в 13:26

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.