Индекс находился вне границ массива

@gracia45 · Регистрация: 05.11.2014

Студворк — интернет-сервис помощи студентам

После того как я сгенерировал массив, и должна выполнятся сортировка, выдает ошибку "Индекс находился вне границ массива " и указывает на эту строчку

C#
1
for (j = 0; j < m_max; j++) c[j] = 0;

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
 
namespace ConsoleApp4
{
    class Program
    {
        const int n_max = 100; 
        const int m_max = 255;
    
 
        static public int count;
        public int[] a = new int[count];
        public int[] b = new int[n_max];
        public int[] c = new int[m_max];
        public int[] a2 = new int[100];
 
 
        static void PoVozrostaniy(ref int[] a, int count)
        {
            {
                int i, j, t;
                Random rand = new Random();
                for (i = 0; i < count; i++)
                    a[i] = rand.Next(0, 400);
                for (i = 1; i < count; i++)
                {
                    t = a[i];
                    for (j = i; j > 0 && a[j - 1] > t; j--)
                    {
                        a[j] = a[j - 1];
                    }
                    a[j] = t;
                }
                Console.WriteLine();
                Console.WriteLine("Сгенерированный массив по возрастанию:");
 
                for (j = 0; j < count; j++)
                {
                    Console.Write(a[j] + " ");
                }
 
            }
        }
 
        static void PoUbivaniyu(ref int[] a, int count)
        {
            int i, j, t;
            Random rand = new Random();
            for (i = 0; i < count; i++)
 
                a[i] = rand.Next(0, 400);
 
            for (i = 1; i < count; i++)
            {
                t = a[i];
                for (j = i; j > 0 && a[j - 1] < t; j--)
                {
                    a[j] = a[j - 1];
                }
                a[j] = t;
            }
            Console.WriteLine();
            Console.WriteLine("Сгенерированный массив по спадонию:");
            for (j = 0; j < count; j++)
            {
                Console.Write(a[j] + " ");
            }
        }
 
        static void CreateMass(ref int[] a, int count)
        {
            Console.WriteLine("Сформированный массив:");
            Random rand = new Random();
            for (int i = 0; i < count; i++)
            {
                a[i] = rand.Next(0, 400);
 
                Console.Write(a[i] + " ");
            }
        }
 
        static void CountSort(ref int[] a, ref int[] b, ref int[] c, int count)
        { int i, j;
            for (j = 0; j < m_max; j++) c[j] = 0;
            for (i = 0; i < count; i++) c[a[i] + 1]++;
            for (j = 0; j < m_max; j++) c[j] += c[j - 1];
            for (i = 0; i < count; i++) b[c[a[i]]++] = a[i];
            for (i = 0; i < count; i++) a[i] = b[i];
 
            for (c[a[i]] = 0; c[a[i]] < count; c[a[i]]++)
                
            Console.WriteLine("сортировка подсчетом:" + c[a[i]]+1 +"]=" + b[c[a[i]]] );
 
        }
 
        static void Main(string[] args)
        {
            int s;
            int k;
 
            Console.WriteLine("Введите размер массива");
            int count;
            count = int.Parse(Console.ReadLine());
            Console.WriteLine("1. Рандомный массив");
            Console.WriteLine("2. Рандомный массив по убыванию");
            Console.WriteLine("3. Рандомный массив по возрастанию");
            k = int.Parse(Console.ReadLine());
 
 
            int[] a = new int[count];
            int[] b = new int[count];
            int[] c = new int[count];
 
            if (k == 1)
            {
                Program.CreateMass(ref a, count);
                Console.ReadLine();
            }
            else if (k == 2)
            {
                Program.PoUbivaniyu(ref a, count);
                Console.ReadLine();
            }
 
            if (k == 3)
            {
                Program.PoVozrostaniy(ref a, count);
                Console.ReadLine();
            }
 
            Program.CountSort(ref  a, ref  b, ref  c, count);
        }
    }
}

@and09reyy · 30.09.2017, 15:13

а когда возникает ошибка в каком случае,потому что я запустил все работает,какие данные вводите

@InTheHeartbeat · 30.09.2017, 15:59

Хочу дать вам несколько советов:
1. Следите за именованием переменных, ваш код жутко не читаемый. Такие имена как a,b,c,d понятны вам, но другим в них разобраться сложнее.
2. Старайтесь избегать транслита в именовании.
3. Итераторы циклов (i,j и т.п.) лучше объявлять в самом цикле, таким образом будет гораздо меньше путанницы.
Например:

C#
1
for (j = 0; j < count; j++){}

измените на

C#
1
for (int j = 0; j < count; j++)

а объявленную вне цикла переменную j - удалите.
4. У вас например в методе CountSort идет подряд много циклов, назовите их итераторы таким образом, чтобы вы через год, посмотрев на свой код, сразу поняли для чего эти циклы нужны и что они перебирают.

Теперь ближе к делу:
В методе CountSort, там где у вас вываливается эксепшн, у вас цикл идет до m_max, значение которого 255, а размер массива устанавливается вручную. Естественно, что при размере массива "с" в 100 элементов, цикл до 255 элементов пройти не сможет. Для таких вещей рекомендую вместо всяких переменных вроде m_max и count, используйте свойство ".Length", например вот так изменится проблемный цикл:

C#
1
for (j = 0; j < c.Length; j++) c[j] = 0;

Это свойство возвращает кол-во элементов массива.

Если вам нужно ограничивать количество при помощи m_max, то лучше делать это на этапе ввода с клавиатуры.

Теперь на указанной вами проблемной строке исключения нет, если изменить код вот так:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
static void CountSort(ref int[] a, ref int[] b, ref int[] c, int count)
        {
            int i, j;
            for (j = 0; j < c.Length; j++) c[j] = 0;
            for (i = 0; i < count; i++) c[a[i] + 1]++;
            for (j = 0; j < m_max; j++) c[j] += c[j - 1];
            for (i = 0; i < count; i++) b[c[a[i]]++] = a[i];
            for (i = 0; i < count; i++) a[i] = b[i];
 
            for (c[a[i]] = 0; c[a[i]] < count; c[a[i]]++)
                
            Console.WriteLine("сортировка подсчетом:" + c[a[i]]+1 +"]=" + b[c[a[i]]] );
 
        }

Но, есть на следующей строке. Откровенно говоря, я не совсем понял что там вообще происходит. Смотрите, вы в качестве индекса массива "c" используете значение взятое из массива "a[i]" и прибавляете 1.

C#
1
for (i = 0; i < count; i++) c[a[i] + 1]++;

Но массив "a", если я правильно понимаю, хранит случайные числа, следовательно там будет число, например 206, которое больше размера массива.

С вашим подходом, единственное что пришло мне на ум, это ограничить рандомайзер до значения count-1.
Это в методе PoVozrostaniy:

C#
1
2
3
Random rand = new Random();
                for (i = 0; i < count; i++)
                    a[i] = rand.Next(0, count-1);

Таким образом, исключение на втором цикле метода CountSort исправляется, но теперь на третьей строке исключение.

C#
1
for (j = 0; j < m_max; j++) c[j] += c[j - 1];

Дело в том, что цикл у вас идет от 0, т.е. при первой итерации цикла, переменная j будет равна нулю. В той же первой итерации вы пытаетесь сделать j-1, а 0-1=-1, индекс элемента массива не может быть отрицательным.
Что происходит в этом методе далее - я так и не разобрался.

Очень рекомендую вам собраться и переписать это дело с чистого листа, думаю я натолкнул вас на какие-то мысли.

Еще пара советов:
1. Вы внутри метода оборачиваете кусок кода фигурными скобками. Для чего? Лучше удалите
Например

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
static void PoVozrostaniy(ref int[] a, int count)
        {
            {
                int i, j, t;
                Random rand = new Random();
                for (i = 0; i < count; i++)
                    a[i] = rand.Next(0, count-1);
                for (i = 1; i < count; i++)
                {
                    t = a[i];
                    for (j = i; j > 0 && a[j - 1] > t; j--)
                    {
                        a[j] = a[j - 1];
                    }
                    a[j] = t;
                }
                Console.WriteLine();
                Console.WriteLine("Сгенерированный массив по возрастанию:");
 
                for (j = 0; j < count; j++)
                {
                    Console.Write(a[j] + " ");
                }
 
            }
        }

изменится на

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
static void PoVozrostaniy(ref int[] a, int count)
        {
            int i, j, t;
            Random rand = new Random();
            for (i = 0; i < count; i++)
                a[i] = rand.Next(0, count - 1);
            for (i = 1; i < count; i++)
            {
                t = a[i];
                for (j = i; j > 0 && a[j - 1] > t; j--)
                {
                    a[j] = a[j - 1];
                }
                a[j] = t;
            }
            Console.WriteLine();
            Console.WriteLine("Сгенерированный массив по возрастанию:");
 
            for (j = 0; j < count; j++)
            {
                Console.Write(a[j] + " ");
            }
        }

2. Если после цикла идет всего одна строка, то фигурные скобки можно не ставить, да, но ставить пустую строку - плохая идея. Сбивает с толку вот здесь, так же как и странный итератор последнего цикла:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
static void CountSort(ref int[] a, ref int[] b, ref int[] c, int count)
        {
            int i, j;
            for (j = 0; j < c.Length; j++) c[j] = 0;
            for (i = 0; i < count; i++) c[a[i] + 1]++;
            for (j = 1; j < count; j++) c[j] += c[j - 1];
            for (i = 0; i < count; i++) b[c[a[i]]++] = a[i];
            for (i = 0; i < count; i++) a[i] = b[i];
 
            for (c[a[i]] = 0; c[a[i]] < count; c[a[i]]++)                
                    
            Console.WriteLine("сортировка подсчетом:" + c[a[i]]+1 +"]=" + b[c[a[i]]] );
 
        }

3. Избегайте анти паттера "Волшебного числа". У вас много где вызывается

C#
1
rand.Next(0, 400)

, если вы вдруг захотите изменить предел генерации - вам придется искать все вызовы и исправлять. Заведите всего одну переменную

C#
1
2
int RandomMax = 400;
rand.Next(0, RandomMax)

и таким образом, при необходимости вам нужно будет изменить значение всего на одной строке, а не у каждой строки где происходит генерация. Это касается всех подобных случаев, у вас их много. Проанализируйте.

Вот вам рабочий код, т.е. сортировку подсчетом я переделал, она работает. Но очень рекомендую вам самому переписать проект с чистого листа придерживаясь рекомендаций, удивитесь как станет проще.
P.S. min и max в методе CountSort это самый большой и самый маленький элемент сортируемого массива.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
    class Program
    {
        const int n_max = 100; 
        const int m_max = 255;
    
 
        static public int count;
        public int[] a = new int[count];
        public int[] b = new int[n_max];
        public int[] c = new int[m_max];
        public int[] a2 = new int[100];
 
 
        static void PoVozrostaniy(ref int[] a, int count)
        {
            int i, j, t;
            Random rand = new Random();
            for (i = 0; i < count; i++)
                a[i] = rand.Next(0, 400);
            for (i = 1; i < count; i++)
            {
                t = a[i];
                for (j = i; j > 0 && a[j - 1] > t; j--)
                {
                    a[j] = a[j - 1];
                }
                a[j] = t;
            }
            Console.WriteLine();
            Console.WriteLine("Сгенерированный массив по возрастанию:");
 
            for (j = 0; j < count; j++)
            {
                Console.Write(a[j] + " ");
            }
        }
 
        static void PoUbivaniyu(ref int[] a, int count)
        {
            int i, j, t;
            Random rand = new Random();
            for (i = 0; i < count; i++)
 
                a[i] = rand.Next(0, 400);
 
            for (i = 1; i < count; i++)
            {
                t = a[i];
                for (j = i; j > 0 && a[j - 1] < t; j--)
                {
                    a[j] = a[j - 1];
                }
                a[j] = t;
            }
            Console.WriteLine();
            Console.WriteLine("Сгенерированный массив по спадонию:");
            for (j = 0; j < count; j++)
            {
                Console.Write(a[j] + " ");
            }
        }
 
        static void CreateMass(ref int[] a, int count)
        {
            Console.WriteLine("Сформированный массив:");
            Random rand = new Random();
            for (int i = 0; i < count; i++)
            {
                a[i] = rand.Next(0, 400);
 
                Console.Write(a[i] + " ");
            }
        }
 
        static void CountSort(ref int[] arr, int min, int max)
        {
            int[] tempArray = new int[max - min + 1];
            int buffIndex = 0;
 
            for (int i = 0; i < tempArray.Length; i++)
            {
                tempArray[i] = 0;
            }
            for (int i = 0; i < arr.Length; i++)
            {
                tempArray[arr[i] - min]++;
            }
 
            for (int i = min; i <= max; i++)
            {
                while (tempArray[i - min]-- > 0)
                {
                    arr[buffIndex] = i;
                    buffIndex++;
                }
            }
 
            Console.WriteLine("Сортировка подсчетом:");
            for (int i = 0; i < arr.Length; i++)
            {
                Console.Write(arr[i] + "   ");
            }
        }
 
        static void Main(string[] args)
        {
            int s;
            int k;
 
            Console.WriteLine("Введите размер массива");
            int count;
            count = int.Parse(Console.ReadLine());
            Console.WriteLine("1. Рандомный массив");
            Console.WriteLine("2. Рандомный массив по убыванию");
            Console.WriteLine("3. Рандомный массив по возрастанию");
            k = int.Parse(Console.ReadLine());
 
 
            int[] a = new int[count];
            int[] b = new int[count];
            int[] c = new int[count];
 
            if (k == 1)
            {
                Program.CreateMass(ref a, count);
                Console.ReadLine();
            }
            else if (k == 2)
            {
                Program.PoUbivaniyu(ref a, count);
                Console.ReadLine();
            }
 
            if (k == 3)
            {
                Program.PoVozrostaniy(ref a, count);
                Console.ReadLine();
            }
 
            Program.CountSort(ref a, a.Min(),a.Max());
 
            Console.ReadKey();
        }
    }

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .	SDL3 для Web (WebAssembly): Работа со звуком через SDL3_mixer 8Observer8 08.02.2026 Содержание блога Пошагово создадим проект для загрузки звукового файла и воспроизведения звука с помощью библиотеки SDL3_mixer. Звук будет воспроизводиться по клику мышки по холсту на Desktop и по. . .	SDL3 для Web (WebAssembly): Основы отладки веб-приложений на SDL3 по USB и Wi-Fi, запущенных в браузере мобильных устройств 8Observer8 07.02.2026 Содержание блога Браузер Chrome имеет средства для отладки мобильных веб-приложений по USB. В этой пошаговой инструкции ограничимся работой с консолью. Вывод в консоль - это часть процесса. . .
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .

@and09reyy 17 / 17 / 23 Регистрация: 22.10.2016 Сообщений: 183
	30.09.2017, 15:13
	а когда возникает ошибка в каком случае,потому что я запустил все работает,какие данные вводите 0