Можете ли вы удалить элемент из списка ‹›, просматривая его в C #

Изменить: чтобы уточнить, вопрос касается Silverlight, который, по-видимому, не поддерживает RemoveAll on List`T. Он доступен в полной структуре, CF, XNA версий 2.0+

Вы можете написать лямбду, которая выражает ваши критерии удаления:

bullets.RemoveAll(bullet => bullet.Offscreen());

Или вы можете выбрать те, которые вам нужны, вместо того, чтобы удалять те, которые вам не нужны:

bullets = bullets.Where(b => !b.OffScreen()).ToList();

Или используйте индексатор для перемещения назад по последовательности:

for(int i=bullets.Count-1;i>=0;i--)
{
    if(bullets[i].OffScreen())
    {
        bullets.RemoveAt(i);
    }
}

bullets.RemoveAll(bullet => bullet.Offscreen());

Изменить: чтобы эта функция работала как есть в Silverlight, добавьте в свой проект следующий метод расширения.

Как и List<T>.RemoveAll, этот алгоритм имеет вид O (N), где N - длина списка, в отличие от O (N * M), где M - количество элементов, удаленных из списка. Поскольку это метод расширения с тем же прототипом, что и метод RemoveAll, найденный во фреймворках, отличных от Silverlight, встроенный будет использоваться, когда он доступен, а этот - бесшовно для сборок Silverlight.

public static class ListExtensions
{
    public static int RemoveAll<T>(this List<T> list, Predicate<T> match)
    {
        if (list == null)
            throw new NullReferenceException();

        if (match == null)
            throw new ArgumentNullException("match");

        int i = 0;
        int j = 0;

        for (i = 0; i < list.Count; i++)
        {
            if (!match(list[i]))
            {
                if (i != j)
                    list[j] = list[i];

                j++;
            }
        }

        int removed = i - j;
        if (removed > 0)
            list.RemoveRange(list.Count - removed, removed);

        return removed;
    }
}

Попытка удалить его в цикле foreach вызовет исключение. Вам нужно перебрать его в обратном порядке с помощью цикла for.

for (int count = bullets.Count - 1; count >= 0; count--)
{
  if (bullets[count].Offscreen())
    {
        //bullets.Remove(bullets[count]);
        bullets.RemoveAt(count);
    }
}

Попробуй это:

bullets.RemoveAll(bullet => bullet.Offscreen());

Лучше либо создать список, который будет содержать элементы для удаления, а затем удалить элементы из списка:

List<Bullet> removedBullets = new List<Bullet>();

foreach(var bullet in bullets)
{
  if (bullet.OffScreen())
  {
   removedBullets.Add(bullet);
  }
}

foreach(var bullet in removedBullets)
{
  bullets.Remove(bullet);
}

Выполняйте итерацию цикла for вместо того, чтобы повторять цикл foreach. Это сработает.

Я сталкивался с этой проблемой раньше и писал в блоге об этом здесь.

Краткая версия заключается в том, что вы можете создать метод расширения под названием RemoveIf:

public void RemoveIf<T>(ICollection<T> collection, Predicate<T> match)
{
    List<T> removed = new List<T>();
    foreach (T item in collection)
    {
        if (match(item))
        {
            removed.Add(item); 
        }
    }

    foreach (T item in removed)
    {
        collection.Remove(item);
    }

    removed.Clear();
}

А затем просто вызывайте его со своим делегатом каждый раз, когда вам это нужно:

RemoveIf(_Entities.Item, delegate(Item i) { return i.OffScreen(); });