Не получается удалить элемент массива из for

@Jettard · Регистрация: 27.05.2010

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

Вот такая вот конструкция. Работает наполовину, половину массива выводит. А на второй половине эксепшн: Необработанное исключение: System.ArgumentOutOfRangeException: Индекс за пределами диапазона. Индекс должен быть положительным числом, а его размер не должен пр
превышать размер коллекции.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public static void CompareFilesRec(List<string> array)
...
List<string> outp = new List<string>();
...
for (int i = 0; i <= array.Count; i++)
            {
                List<string> fromAll = Comparer.CompareLength(array[i], array);
                if (fromAll.Count != 0)
                    foreach (var f2 in fromAll)
                    {
                        if (Checksum.CompareHash(array[i], f2))
                            outp.Add(f2);
                        array.RemoveAt(i);
                    }
                Comparer.ShowResults(outp);
                outp.Clear();
            }

@EvilFromHell · 09.03.2013, 04:59

Я не совсем понял код, ибо у меня в классе Comparer таких методов нет, да и приведен он не полностью, но очевидно, что код обработает только половину массива, ведь в условии for количество элементов в массиве, с которого вы их в том же цикле удаляете.. А эксепшен вылетает из-за = в том же условии цикла for. Очевидно, делать это надо не через for, а через while к примеру, либо при удалении элемента писать i--.

@Jettard · 09.03.2013, 15:50 **[ТС]**

EvilFromHell, Comparer это у меня свой класс, со своими методами. Если добавляю і-- то тот же эксепшн.

@Монфрид · 09.03.2013, 15:57

вы удаляете элемент массива, по которому идёте циклом, так не пойдёт.
Замените это

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

на это

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

@Jettard · 09.03.2013, 16:06 **[ТС]**

А как вы предлагаете через while?

Добавлено через 5 минут
Монфрид, всеравно эксепшн.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
for (int i = 0; i <= array.ToList().Count; i++)
            {
                List<string> fromAll = Comparer.CompareLength(array[i], array);
                if (fromAll.Count != 0)
                    foreach (var f2 in fromAll)
                    {
                        if (Checksum.CompareHash(array[i], f2))
                            outp.Add(f2);
                        array.RemoveAt(i);
                        i--;
                        
                    }
                Comparer.ShowResults(outp);
                outp.Clear();
            }

@Wolfdp · 09.03.2013, 16:08

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

for (int i = 0; i <= array.Count; i++)

Отсчет начинается с нуля, а значит....

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

@Jettard · 09.03.2013, 16:37 **[ТС]**

Wolfdp, нет, не помогло.

@EvilFromHell · 09.03.2013, 17:08

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.IO;
using System.Diagnostics;
using System.Globalization;
using System.Threading;
using System.Collections;
using System.ComponentModel;
using System.Data;
 
namespace alena
{
    class Program
    {
        static void Main(string[] args)
        {
        
             List<string> z = new List<string>();
                z.Add("aaa");
                z.Add("bbb");
                z.Add("ccc");
                z.Add("ddd");
                z.Add("eee");
                z.Add("eee");
                z.Add("ddd");
            CompareFilesRec(z);
        }
 
        public static void CompareFilesRec(List<string> array)
        {
            int i=0;
            while(i<array.Count){
                array[i].Split('8');
                if (array[i].Contains("e")) { array.RemoveAt(i); }
                else { i++; }
            }
        }
    }
}

Вот логика, как перевести в свой код думаю очевидно, я же писал об этом вчера. Если удалять все элементы с массива-то просто while(array.Count>0)

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

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
for (int i = 0; i <= array.ToList().Count; i++)
            {
                List<string> fromAll = Comparer.CompareLength(array[i], array);
                if (fromAll.Count != 0)
                    foreach (var f2 in fromAll)
                    {
                        if (Checksum.CompareHash(array[i], f2))
                            outp.Add(f2);
                        array.RemoveAt(i);
                        i--;
                        
                    }
                Comparer.ShowResults(outp);
                outp.Clear();
            }

Тут у вас i уходит в глубокий минус, так как вы его отнимаете много раз еще тогда, когда оно = 0. Отнимать надо 1 раз-но это в любом случае нерационально, надо просто не прибавлять все время. Да и вообще-смысл удалять массив таким аццки извращенным способом-поясните

Добавлено через 6 минут
Да-и под foreach удаляется сразу много элементов массива, а проверки, не стало ли там элементов меньше i, нигде нет-тут уж логика окончательно уплывает и я перестаю понимать смысл сего действа окончательно...

@Jettard · 09.03.2013, 17:11 **[ТС]**

foreach вообще ниочем тут. Вариант с while работает отлично, спасибо.

@wolfinil · 09.03.2013, 18:45

Вроде всегда если отсчет с 0 нуля в цикле по листу или массиву,то условие должно быть List.Count - 1 или Array.Length - 1.

Новые блоги и статьи Все статьи Все блоги /
Переходник USB-CAN-GPIO Eddy_Em 20.03.2026 Достаточно давно на работе возникла необходимость в переходнике CAN-USB с гальваноразвязкой, оный и был разработан. Однако, все меня терзала совесть, что аж 48-ногий МК используется так тупо: просто. . .	Оттенки серого Argus19 18.03.2026 Оттенки серого Нашёл в интернете 3 прекрасных модуля: Модуль класса открытия диалога открытия/ сохранения файла на Win32 API; Модуль класса быстрого перекодирования цветного изображения в оттенки. . .	SDL3 для Desktop (MinGW): Рисуем цветные прямоугольники с помощью рисовальщика SDL3 на Си и C++ 8Observer8 17.03.2026 Содержание блога Финальные проекты на Си и на C++: finish-rectangles-sdl3-c. zip finish-rectangles-sdl3-cpp. zip	Символические и жёсткие ссылки в Linux. algri14 15.03.2026 Существует два типа ссылок — символические и жёсткие. Ссылка в Linux — это запись в каталоге, которая может указывать либо на inode «файла-ИСТОЧНИКА», тогда это будет «жёсткая ссылка» (hard link),. . .
[Owen Logic] Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ФедосеевПавел 14.03.2026 Поддержание уровня воды в резервуаре количеством включённых насосов: моделирование и выбор регулятора ВВЕДЕНИЕ Выполняя задание на управление насосной группой заполнения резервуара,. . .	делаю науч статью по влиянию грибов на сукцессию anaschu 13.03.2026 прикрепляю статью	SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .

@EvilFromHell 979 / 874 / 350 Регистрация: 26.04.2012 Сообщений: 2,647
	09.03.2013, 04:59
	Я не совсем понял код, ибо у меня в классе Comparer таких методов нет, да и приведен он не полностью, но очевидно, что код обработает только половину массива, ведь в условии for количество элементов в массиве, с которого вы их в том же цикле удаляете.. А эксепшен вылетает из-за = в том же условии цикла for. Очевидно, делать это надо не через for, а через while к примеру, либо при удалении элемента писать i--. 0

@Jettard 41 / 39 / 7 Регистрация: 27.05.2010 Сообщений: 252
	09.03.2013, 15:50 [ТС]
	EvilFromHell, Comparer это у меня свой класс, со своими методами. Если добавляю і-- то тот же эксепшн. 0

@Jettard 41 / 39 / 7 Регистрация: 27.05.2010 Сообщений: 252
	09.03.2013, 16:37 [ТС]
	Wolfdp, нет, не помогло. 0

@Jettard 41 / 39 / 7 Регистрация: 27.05.2010 Сообщений: 252
	09.03.2013, 17:11 [ТС]
	foreach вообще ниочем тут. Вариант с while работает отлично, спасибо. 0

@wolfinil 1 / 3 / 2 Регистрация: 30.12.2010 Сообщений: 45
	09.03.2013, 18:45
	Вроде всегда если отсчет с 0 нуля в цикле по листу или массиву,то условие должно быть List.Count - 1 или Array.Length - 1. 0