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

@misterio2224 · Регистрация: 16.01.2021

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

Всем добрый день! Выдает ошибку, индекс находится все границ массива. Я новенький, не обзывайте)
Вывести отсортированый массив по уменьшению значения, ранее создал массив и все ок, первый элемент это минимальный по значению элемент, последний - максимальный. Нужно вывести во вторую строку элемента dataGridView

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
int maxIndex = 0, minIndex = 0;
            double max = -99, min = 99;
            for (int i = 0; i < myArray.Length; i++)
            {
                if (myArray[i] > max)
                {
                    maxIndex = i;
                    max = myArray[i];
 
                }
                if (myArray[i] < min)
                {
                    minIndex = i;
                    min = myArray[i];
                }
            }
            double temp, tempmin, tempmax;
            tempmin = myArray[0];
            myArray[0] = myArray[minIndex];
            myArray[minIndex] = tempmin;
 
            tempmax = myArray[myArray.Length - 1];
            myArray[myArray.Length - 1] = myArray[myArray.Length];
            myArray[maxIndex] = tempmax;
            for (int i = 1; i < myArray.Length - 2; i++)
            {
                for (int j = i + 1; j < myArray.Length - 1; j++)
                {
                    if (myArray[i] < myArray[j])
                    {
                        temp = myArray[i];
                        myArray[i] = myArray[j];
                        myArray[j] = temp;
                    }
                }
 
            }
 
            for (int i = 0; i < myArray.Length; i++)
            {
                dataGridViewArray[i, 1].Value = myArray[i];
                dataGridViewArray.Columns[i].HeaderText = i.ToString();
            }

Алексей1153 · 03.04.2021, 17:50

например, тут

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

myArray[myArray.Length - 1] = myArray[myArray.Length];

@misterio2224 · 03.04.2021, 18:00 **[ТС]**

Да, спасибо большое, но это не последняя ошибка(, теперь пишет: индекс за пределами диапазона, индекс должен быть положительным

Добавлено через 5 минут
все, решил, не обьявил глобальную переменную

Алексей1153 · 03.04.2021, 18:00

misterio2224, может, массив нулевой длины? Полный бы код в студию

@misterio2224 · 03.04.2021, 20:19 **[ТС]**

Нет,

C#
1
2
3
int choise = Convert.ToInt32(numericUpDownCount.Text);
            dataGridViewArray.RowCount = 2;
            dataGridViewArray.ColumnCount = choise;

я не перенес эту переменную choise(количество элементов) в событие кнопки, и после этого все заработало

но если хотите, даю код

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
namespace Arrays1DWinForms
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
            Console.OutputEncoding = Encoding.Unicode;
            Console.InputEncoding = Encoding.Unicode;
            System.Globalization.CultureInfo customCulture = (System.Globalization.CultureInfo)
            System.Threading.Thread.CurrentThread.CurrentCulture.Clone();
            customCulture.NumberFormat.NumberDecimalSeparator = ".";
            System.Threading.Thread.CurrentThread.CurrentCulture = customCulture;        
        }
        double [] myArray;
 
 
        private void generatebutton_Click(object sender, EventArgs e)
        {
            int choise = Convert.ToInt32(numericUpDownCount.Text);
            myArray = new double[choise];
            dataGridViewArray.RowCount = 1;
            dataGridViewArray.ColumnCount = choise;
            Random rnd = new Random();
            for (int i = 0; i < myArray.Length; i++)
            {
                myArray[i] = rnd.Next(-123, 163)/10.0;
                dataGridViewArray[i, 0].Value = myArray[i];
                dataGridViewArray.Columns[i].HeaderText = i.ToString();            
            }
 
        }
 
        private void buttonsumm_Click(object sender, EventArgs e)
        {
            int indexSumm = 0;
            for (int i = 0; i < myArray.Length; i++)
            {
                if (myArray[i] < 0 && i % 2 == 0)
                    indexSumm += i;
            }
            textBoxsumm.Text = indexSumm.ToString();
        }
 
        private void buttonsort_Click(object sender, EventArgs e)
        {
            int choise = Convert.ToInt32(numericUpDownCount.Text);
            dataGridViewArray.RowCount = 2;
            dataGridViewArray.ColumnCount = choise;
            int maxIndex = 0, minIndex = 0;
            double max = -99, min = 99;
            for (int i = 0; i < myArray.Length; i++)
            {
                if (myArray[i] > max)
                {
                    maxIndex = i;
                    max = myArray[i];
 
                }
                if (myArray[i] < min)
                {
                    minIndex = i;
                    min = myArray[i];
                }
            }
            double temp, tempmin, tempmax;
            tempmin = myArray[0];
            myArray[0] = myArray[minIndex];
            myArray[minIndex] = tempmin;
 
            tempmax = myArray[myArray.Length - 1];
            myArray[myArray.Length - 1] = myArray[maxIndex];
            myArray[maxIndex] = tempmax;
            for (int i = 1; i < myArray.Length - 2; i++)
            {
                for (int j = i + 1; j < myArray.Length - 1; j++)
                {
                    if (myArray[i] < myArray[j])
                    {
                        temp = myArray[i];
                        myArray[i] = myArray[j];
                        myArray[j] = temp;
                    }
                }
 
            }
 
            for (int i = 0; i < myArray.Length; i++)
            {
                dataGridViewArray[i, 1].Value = myArray[i];
                dataGridViewArray.Columns[i].HeaderText = i.ToString();
            }
 
        }
    }
}

Подскажите ,может , как легче сделать или что-то лишнее наделал
Задание: 1) генерация массива
2) найти сумму индексов отрицательных элементов, чьи индексы делятся на 2
3) Отсортировать масив по убыванию: первый элемент - минимальное по значению, последний - максимальный

Алексей1153 · 03.04.2021, 20:47

2) - if (myArray[i] < 0 && i % 2 == 0)
чисто из соображений шустродействия (хотя, на маленьких объёмах разницу не увидеть):
сначала проверить индекс, только потом содержимое

C#
1
if (i%2==0 && myArray[i] < 0 )

3) - double max = -99, min = 99; странный выбор инициализации min и max. Возьми просто первый элемент массива для обоих (если, конечно, размер массива не нулевой)

Добавлено через 2 минуты

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

double temp, tempmin, tempmax;

почему объявления переменных так далеко от их реального использования?

сделай

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

double tempmin = myArray[0];

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

double tempmax = myArray[myArray.Length - 1];

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

double temp = myArray[i];

Добавлено через 1 минуту
тут нужна проверка длины Length >=1

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

tempmax = myArray[myArray.Length - 1];

@Lexeq · 03.04.2021, 21:13

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

чьи индексы делятся на 2

Можно не проверять через остаток от деления, очевидно, что на 2 делится каждое второе число:

C#
1
2
3
4
5
6
            int indexSumm = 0;
            for (int i = 0; i < myArray.Length; i+=2)
            {
                if (myArray[i] < 0)
                    indexSumm += i;
            }

Алексей1153 · 03.04.2021, 21:24

Lexeq, тоже верно ) Правда, если препод поменяет условие проверки, придётся шапку цикла снова менять

@misterio2224 · 03.04.2021, 22:42 **[ТС]**

Спасибо за подсказки, обязательно учту!

Новые блоги и статьи Все статьи Все блоги /
Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .	Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .
SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .

@misterio2224 0 / 0 / 0 Регистрация: 16.01.2021 Сообщений: 10
	03.04.2021, 18:00 [ТС]
	Да, спасибо большое, но это не последняя ошибка(, теперь пишет: индекс за пределами диапазона, индекс должен быть положительным Добавлено через 5 минут все, решил, не обьявил глобальную переменную 0

Алексей1153 фрилансер 6461 / 5669 / 1130 Регистрация: 11.10.2019 Сообщений: 15,087
	03.04.2021, 18:00
	misterio2224, может, массив нулевой длины? Полный бы код в студию 0

Алексей1153 фрилансер 6461 / 5669 / 1130 Регистрация: 11.10.2019 Сообщений: 15,087
	03.04.2021, 21:24
	Lexeq, тоже верно ) Правда, если препод поменяет условие проверки, придётся шапку цикла снова менять 0

@misterio2224 0 / 0 / 0 Регистрация: 16.01.2021 Сообщений: 10
	03.04.2021, 22:42 [ТС]
	Спасибо за подсказки, обязательно учту! 0

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

Решение