Работа со списком в цикле

@VoltDeMar · Регистрация: 05.06.2012

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

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

@Хозяин · 15.11.2015, 19:39

VoltDeMar, Используйте цикл Foreach, а в нем метод Remove/RemoveAt. После каждого удаления из коллекции, свойство Count уменьшается на единицу, тем самым, вы выходите за диапазон значений, если делаете удаление по индексу RemoveAt.

@WinDev · 15.11.2015, 19:42

Когда вы используете цикл for, все данные в его шапке (нач условия, условия окончания и увеличение цикла) вычисляются один раз, перед первым оператором тела этого цикла.

Добавлено через 2 минуты
Хозяин, цикл foreach нельзя использовать для модификации коллекции

C#
1
2
3
4
5
6
7
8
9
10
            List<int> a = new List<int>();
            a.Add(1);
            a.Add(1);
            a.Add(1);
            a.Add(1);
            a.Add(1);
            foreach (var item in a)
            {
                a.Remove(item);
            }

Выдаст исключение.

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

@Хозяин · 15.11.2015, 19:45

WinDev, Почему вы так думаете? Все успешно удаляется в EF, вот пример:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
using (var context = new EducationalServicesEntities())
{
    var delRecord = context.Location.Where(l => l.LocationID.Equals((int)id)).Select(l => l);
    foreach (var d in delRecord)
        context.Location.Remove(d);
    try
    {
        context.SaveChanges(); RefreshDB();
    }
    catch (Exception ex)
    {
        MessageBox.Show(ex.Message, "Ошибка удаления записи.", MessageBoxButtons.OK, MessageBoxIcon.Error);
    }
}

@WinDev · 15.11.2015, 19:48

Хозяин, вы путаеете. Вы в своем цикле указали одну коллекцию delRecord, а удаляете в другой context.Location.Remove(d);
Посмотрите внимательно нам мой пример.

Добавлено через 40 секунд
Я имею в виду нельзя модифицировать коллекцию по которой идет проход форычем. Код с простым списком тому доказательство, можете проверить у себя в студии.

@Хозяин · 15.11.2015, 19:56

WinDev, Такой вариант работает:

C#
1
2
3
4
5
6
7
8
9
10
11
12
List<int> a = new List<int>();
a.Add(1);
a.Add(2);
a.Add(3);
a.Add(4);
a.Add(5);
for (int i = 0; i < a.Count; i++)
    if (a[i] == 3 || a[i] == 1)
        a.Remove(a[i]);
foreach (var item in a)
    Console.WriteLine(item);
Console.ReadKey();

@WinDev · 15.11.2015, 20:00

Хозяин, мы спорим о разных вещах. Вы сейчас удалили элементы списка в цикле for а не foreach. Не спорю, что элементы из списка нельзя удалять...

Someone007 · 15.11.2015, 20:06

Сообщение от Хозяин

WinDev, Такой вариант работает:

Не работает это

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
            List<int> a = new List<int>() { 1, 2, 3, 4, 5 };
 
            for (int i = 0; i < a.Count; i++)
            {
                if (a[i] == 1 || a[i] == 2 || a[i] == 4 || a[i] == 5)
                {
                    a.Remove(a[i]);
                }
            }
 
            foreach (var item in a)
                Console.WriteLine(item);
 
            Console.ReadKey();

В итоге у вас останется 2, 3, 5, что явно не соответствует ожиданиям.

@Хозяин · 15.11.2015, 20:15

WinDev, Я не спорю, я просто не так понял ваше высказывание выше, поэтому да, соглашусь, что если делать удаление в цикле foreach из коллекции с которой работаешь, то выдаст исключение компилятор

Someone007, Сделаем тогда по другому, так конечно делать не верно, но все же

C#
1
2
3
4
5
6
7
8
9
10
List<int> a = new List<int>() { 1, 2, 3, 4, 5 };
for (int i = 0; i < a.Count; i++)
    if (a[i] == 1 || a[i] == 2 || a[i] == 4 || a[i] == 5)
    {
        a.Remove(a[i]);
        i--;
    }
foreach (var item in a)
    Console.WriteLine(item);
Console.ReadKey();

@WinDev · 15.11.2015, 20:15

Мне кажется наши споры нужно заканчивать, автор темы не найдет ответа своего )

Новые блоги и статьи Все статьи Все блоги /
Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	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 и по. . .

@VoltDeMar 18 / 18 / 4 Регистрация: 05.06.2012 Сообщений: 1,020

	Работа со списком в цикле 15.11.2015, 19:34. Показов 19449. Ответов 9 Метки нет (Все метки) Здравствуйте. в цикле for перебираю элементы списка и некоторые удаляю, столкнулся с ошибкой выхода за размер коллекции. Подскажите цикл for в компилированном виде перебирает до начального размера коллекции или каждый проход сверяет размер ? Подозреваю что сверяет. Каким в таком случае способом рациональнее всего удалять элементы из коллекции? (На ум приходит массив индексов и удаление после выборки). 0

@Хозяин Заблокирован
	15.11.2015, 19:39
	VoltDeMar, Используйте цикл Foreach, а в нем метод Remove/RemoveAt. После каждого удаления из коллекции, свойство Count уменьшается на единицу, тем самым, вы выходите за диапазон значений, если делаете удаление по индексу RemoveAt. 0

@WinDev 78 / 77 / 52 Регистрация: 11.06.2015 Сообщений: 189
	15.11.2015, 19:48
	Хозяин, вы путаеете. Вы в своем цикле указали одну коллекцию delRecord, а удаляете в другой context.Location.Remove(d); Посмотрите внимательно нам мой пример. Добавлено через 40 секунд Я имею в виду нельзя модифицировать коллекцию по которой идет проход форычем. Код с простым списком тому доказательство, можете проверить у себя в студии. 0

@WinDev 78 / 77 / 52 Регистрация: 11.06.2015 Сообщений: 189
	15.11.2015, 20:00
	Хозяин, мы спорим о разных вещах. Вы сейчас удалили элементы списка в цикле for а не foreach. Не спорю, что элементы из списка нельзя удалять... 0

@WinDev 78 / 77 / 52 Регистрация: 11.06.2015 Сообщений: 189
	15.11.2015, 20:15
	Мне кажется наши споры нужно заканчивать, автор темы не найдет ответа своего ) 0