Ошибка при использование цикла foreach

@_Vertigo_ · Регистрация: 07.09.2013

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

Всем доброго времени суток. Есть небольшая проблема с циклом foreach. Смысл ошибки я понимаю, но исправить ее не могу:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
class StudentGroup{
public void DeleteElement()
        {
            if (mn.Count > 1) 
            {
                System.Console.WriteLine("Список идентификаторов группы:");
                foreach (Student ob in mn)
                    System.Console.WriteLine(ob.getId());
                System.Console.WriteLine("Введите идентификатор удаляемого студента:");
                int Id = Convert.ToInt32(Console.ReadLine());
                foreach (Student ob in mn)
                {
                    if (Id == ob.getId())
                        mn.Remove(ob); // нельзя же удалять объект используемый в цикле foreach или нет?
                }
            }
            else
                System.Console.WriteLine("Группа пуста!");
        }
 
        private System.Collections.Generic.List<Student> mn;
}

@Cupko · 05.05.2014, 14:26

C#
1
mn.RemoveAll(x => x.getId() == Id);

@kolorotur · 05.05.2014, 14:27

Доброго времени года!

Не удаляйте в цикле foreach или замените foreach на какой-нибудь другой цикл.

@_Vertigo_ · 05.05.2014, 14:37 **[ТС]**

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

Не удаляйте в цикле foreach или замените foreach на какой-нибудь другой цикл.

Все гениальное просто. Но без foreach я не знаю, как обратиться ко всем элементам списка.
Cupko, я немного не могу понять Вашей реализации. Можно пару комментариев?

@kolorotur · 05.05.2014, 14:39

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

Но без foreach я не знаю, как обратиться ко всем элементам списка.

Кроме foreach обход коллекции можно организовать такими конструкциями:
for, do, while, да тот же пресловутый goto.

Какой тип у переменной mn?

А, заметил.
Тогда так, например:

C#
1
2
3
for (int i = mn.Count - 1; i >= 0; i--)
   if (mn[i].getId() == Id)
      mn.RemoveAt(i);

@Cupko · 05.05.2014, 14:39

_Vertigo_, Почитайте на MSDN, я даже не знаю, что вам прокомментировать.

@_Vertigo_ · 05.05.2014, 14:42 **[ТС]**

kolorotur, тип Student. Лучше весь код покажу:

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
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
 
namespace INF_lab_9
{
    class Program
    {
        static void Main(string[] args)
        {
            Student[] somemas = new Student[3];
            somemas[0] = new Student("Андрей", "Неважно", "Сергеевич", 22, 12223, 5.6);
            somemas[1] = new Student("Сергей", "Лучший", "Сергеевич", 19, 11112, 5.2);
            somemas[2] = new Student("Антон", "Неважно", "Васильевич", 20, 12313, 3.6);
            List<Student> mn = new List <Student>();
            for (int i = 0; i < 3; i++)
                mn.Add(somemas[i]);
            StudentGroup ob = new StudentGroup(mn);
            int k = 0;
            System.Console.WriteLine("(1) - Вывести весь список студентов");
            System.Console.WriteLine("(2) - Поиск студента по идентификатору");
            System.Console.WriteLine("(3) - Поиск студента по фамилии");
            System.Console.WriteLine("(4) - Добавить студента");
            System.Console.WriteLine("(5) - Удалить студента");
            System.Console.WriteLine("(6) - Закончить работу");
            do 
            {
                System.Console.WriteLine("");
                k = Convert.ToInt32(Console.ReadLine());
                switch (k)
                {
                    case (1): ob.Show(); break;
                    case (2): ob.SearchById(); break;
                    case (3): ob.SearchBySecondName(); break;
                    case (4): ob.AddElement(); break;
                    case (5): ob.DeleteElement(); break;
                    default: System.Console.WriteLine(""); break;
                }
            } while (k != 6);
            System.Console.WriteLine("Работа завершена");
        }
    }
    class Student
    {
        public Student(string name, string secondname, string lastname, int age, int id, double average)
        {
            this.name = name;
            this.secondname = secondname;
            this.lastname = lastname;
            this.age = age;
            this.id = id;
            this.average = average;
        }
        ~Student()
        {
        }
        public string getName()
        {
            return name;
        }
        public string getLastName()
        {
            return lastname;
        }
        public string getSecondName()
        {
            return secondname;
        }
        public int getAge()
        {
            return age;
        }
        public int getId()
        {
            return id;
        }
        public double getAverage()
        {
            return average;
        }
        public void setName(string n)
        {
            name = n;
        }
        public void setLastName(string l)
        {
            lastname = l;
        }
        public void setSecondName(string s)
        {
            secondname = s;
        }
        public void setAge(int a)
        {
            age = a;
        }
        public void setId(int i)
        {
            id = i;
        }
        public void setAverage(int av)
        {
            average = av;
        }
        public void Show()
        {
            System.Console.WriteLine("Фамилия: " + secondname);
            System.Console.WriteLine("Имя: " + name);
            System.Console.WriteLine("Отчество: " + lastname);
            System.Console.WriteLine("Возраст: " + age);
            System.Console.WriteLine("Идентификатор: " + id);
            System.Console.WriteLine("Средняя успеваемость: " + average);
            System.Console.WriteLine();
        }
        private string name = "";
        private string secondname = "";
        private string lastname = "";
        private int age = 0;
        private int id = 0;
        private double average = 0.0;
    }
    
    class StudentGroup
    {
        public StudentGroup(System.Collections.Generic.List<Student> mn)
        {
            this.mn = mn;
        }
        public void Show()
        {
            if (mn.Count > 0)
            {
                foreach (Student ob in mn)
                    ob.Show();
            }
            else
                System.Console.WriteLine("Список пуст!");
 
        }
        public void DeleteElement()
        {
            if (mn.Count > 1) 
            {
                System.Console.WriteLine("Список идентификаторов группы:");
                foreach (Student ob in mn)
                    System.Console.WriteLine(ob.getId());
                System.Console.WriteLine("Введите идентификатор удаляемого студента:");
                int Id = Convert.ToInt32(Console.ReadLine());
                foreach (Student ob in mn)
                {
                    if (Id == ob.getId())
                        n.Remove(ob);
                }
            }
            else
                System.Console.WriteLine("Группа пуста!");
        }
        public void AddElement()
        {
            System.Console.WriteLine("Введите имя студента:");
            string n = Convert.ToString(Console.ReadLine());
            System.Console.WriteLine("Введите фамилию студента:");
            string sn = Convert.ToString(Console.ReadLine());
            System.Console.WriteLine("Введите отчество студента:");
            string ln = Convert.ToString(Console.ReadLine());
            System.Console.WriteLine("Введите идентификатор студента:");
            int Id = Convert.ToInt32(Console.ReadLine());
            System.Console.WriteLine("Введите возраст студента:");
            int ag = Convert.ToInt32(Console.ReadLine());
            System.Console.WriteLine("Введите среднюю успеваемость студента:");
            double av = Convert.ToDouble(Console.ReadLine());
            Student m = new Student(n, sn, ln, Id, ag, av);
            mn.Add(m);
        }
        public void SearchById()
        {
            int k = 0;
            System.Console.WriteLine("Введите идентификатор студента, которого необходимо найти:");
            int Id = Convert.ToInt32(Console.ReadLine());
            foreach (Student ob in mn)
                if (ob.getId() == Id){
                    ob.Show();
                    k = 1;
                    break;
                }
                else
                    k = 0;
        if(k == 0) System.Console.WriteLine("Студента с таким идентификатором нет в списке!");
        }
        public void SearchBySecondName()
        {
            int k = 0;
            System.Console.WriteLine("Введите фамилию студента, которого необходимо найти:");
            string sn = Convert.ToString(Console.ReadLine());
            foreach (Student ob in mn)
                if (ob.getSecondName() == sn)
                {
                    ob.Show();
                    k = 1;
                    break;
                }
                else
                    k = 0;
 
            if(k == 0)   System.Console.WriteLine("Студента с такой фамилией нет в списке!");
        }
        private System.Collections.Generic.List<Student> mn;
    } 
}

@LeniumSoft · 05.05.2014, 15:36

Я бы просто вместо:

C#
1
2
3
4
5
foreach (Student ob in mn)
                {
                    if (Id == ob.getId())
                        n.Remove(ob);
                }

сделал:

C#
1
2
3
4
5
foreach (Student ob in mn.ToList())
                {
                    if (Id == ob.getId())
                        n.Remove(ob);
                }

Новые блоги и статьи Все статьи Все блоги /
Реалии 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. . . .	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 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	05.05.2014, 14:27
	Доброго времени года! Не удаляйте в цикле foreach или замените foreach на какой-нибудь другой цикл. 0

@Cupko 658 / 595 / 171 Регистрация: 17.07.2012 Сообщений: 1,682 Записей в блоге: 1
	05.05.2014, 14:39
	_Vertigo_, Почитайте на MSDN, я даже не знаю, что вам прокомментировать. 1

Ошибка при использование цикла foreach

Решение