Слияние двух списков. Ошибка "Индекс должен находиться в границах этого списка"

@Cyberlol · Регистрация: 14.02.2016

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

Есть метод слияния двух списков List<int> один.
Например,подаю на вход первый список 27 42 -26 -12 -10 и второй список -45 -12 24 36 46.

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

Кликните здесь для просмотра всего текста

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
 private List<int> MergeArrays(List<int> firstArray, List<int> secondArray)
        {
            List<int> result = new List<int>();
            result.AddRange(firstArray);
            for (int i = 0; i < secondArray.Count; i++)
            {
                for (int j = 0; j < result.Count-1 ; j++)
                {
                    if (secondArray[i] < result[j])
                        result.Insert(secondArray[i], j);
                    if (secondArray[i] == result[j])
                        result.Insert(secondArray[i], j);
                    if (secondArray[i] > result[j] && secondArray[i] < result[j + 1])
                    {
                        result.Insert(secondArray[i], j );
                    }
                    if (secondArray[i] > result[j] && secondArray[i] > result[j + 1])
                    {
                        int h = i;
                        while((h==result.Count-1)&&secondArray[i]>result[h])
                        { h++; }
                        result.Insert(secondArray[i], h );
                    }
                }
            }
            return result;
        }

@Usaga · 28.05.2017, 05:01

Cyberlol, пройдитесь отладчиком по своему коду и найдите ошибку.

@Cyberlol · 28.05.2017, 05:03 **[ТС]**

пробовал,все равно пишет. Хотя нет ни отрицательных индексов,не индексов >= количеству элементов
Ошибка именно когда вызываю метод .Insert

@Usaga · 28.05.2017, 05:14

Cyberlol, что ты попробовал?))) Прошёлся по улице в позе отладчика?)))

Я имел в виду использовать отладчик для поиска проблемы. Запускаешь приложение и исследуешь его строка за строкой, смотришь как и где меняются переменные, чтобы понять в каком месте одна из переменных, используемая как индекс массива, принимает неверное значение. Так яснее?)

@Cyberlol · 28.05.2017, 05:28 **[ТС]**

все равно ошибка,я делал пошаговой отладкой с заходом (F11), даже пробовал задать непосредственно значения вместо переменной i,не помогло,к сожалению(

@LeniumSoft · 28.05.2017, 05:42

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

все равно ошибка,

C#
1
for (int i = 0; i < secondArray.Count; i++)

Да тут без дебага видно отсутствие "- 1" после Count

@Usaga · 28.05.2017, 05:46

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

все равно ошибка,я делал пошаговой отладкой с заходом (F11), даже пробовал задать непосредственно значения вместо переменной i,не помогло,к сожалению(

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

Ответ тебе уже, видимо, дали, но имей в виду, что с таким подходом далеко ты не уйдёшь в программировании.

@Cyberlol · 28.05.2017, 05:57 **[ТС]**

Я пробую подать firstArray -33 -32 -5 3 42 и secondArray -44 -5 11 14 29

C#
1
2
if (secondArray[i] < result[j])
                        result.Insert(secondArray[i], j);

уже здесь выпадает ошибка,хотя i=0,j=0
result - это и есть firstArray(элементы просто скопировал в result)

@Usaga · 28.05.2017, 06:32

Cyberlol, нельзя модифицировать коллекцию, которую в данный момент перебираешь. Если такое нужно, то введи вторую коллекцию, в которую складывай нужное, а уже после цикла сливай коллекции в одну.

@Cyberlol · 28.05.2017, 07:37 **[ТС]**

Благодарю за ответы.Оказывается я местами перепутал значения в .Insert. Нужно было result.Insert( j,secondArray[i]) , а не result.Insert(secondArray[i], j)

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	[В процессе разработки] SDL3 для Web (WebAssembly): Сборка библиотек SDL3 и Box2D из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»

@Usaga 14293 / 9378 / 1352 Регистрация: 21.01.2016 Сообщений: 35,344
	28.05.2017, 05:01
	Cyberlol, пройдитесь отладчиком по своему коду и найдите ошибку. 0

@Cyberlol 4 / 4 / 1 Регистрация: 14.02.2016 Сообщений: 126
	28.05.2017, 05:03 [ТС]
	пробовал,все равно пишет. Хотя нет ни отрицательных индексов,не индексов >= количеству элементов Ошибка именно когда вызываю метод .Insert 0

@Usaga 14293 / 9378 / 1352 Регистрация: 21.01.2016 Сообщений: 35,344
	28.05.2017, 05:14
	Cyberlol, что ты попробовал?))) Прошёлся по улице в позе отладчика?))) Я имел в виду использовать отладчик для поиска проблемы. Запускаешь приложение и исследуешь его строка за строкой, смотришь как и где меняются переменные, чтобы понять в каком месте одна из переменных, используемая как индекс массива, принимает неверное значение. Так яснее?) 0

@Cyberlol 4 / 4 / 1 Регистрация: 14.02.2016 Сообщений: 126
	28.05.2017, 05:28 [ТС]
	все равно ошибка,я делал пошаговой отладкой с заходом (F11), даже пробовал задать непосредственно значения вместо переменной i,не помогло,к сожалению( 0

@Usaga 14293 / 9378 / 1352 Регистрация: 21.01.2016 Сообщений: 35,344
	28.05.2017, 06:32
	Cyberlol, нельзя модифицировать коллекцию, которую в данный момент перебираешь. Если такое нужно, то введи вторую коллекцию, в которую складывай нужное, а уже после цикла сливай коллекции в одну. 1

@Cyberlol 4 / 4 / 1 Регистрация: 14.02.2016 Сообщений: 126
	28.05.2017, 07:37 [ТС]
	Благодарю за ответы.Оказывается я местами перепутал значения в .Insert. Нужно было result.Insert( j,secondArray[i]) , а не result.Insert(secondArray[i], j) 0