Qual è il modo più semplice per foreach attraverso un elenco <T> rimozione di oggetti indesiderati?

Question

Nella mia applicazione, _collection è una lista da cui ho bisogno di rimuovere tutti gli oggetti Utente che non corrispondono ai criteri .

Tuttavia, il codice seguente ottiene un invalido errore operazione nella sua seconda iterazione poiché il _collection stessa è stata modificata:

foreach (User user in _collection) 
{ 
    if (!user.IsApproved()) 
    { 
     _collection.Remove(user); 
    } 
} 

ho potuto creare un'altra raccolta List e copiarli avanti e indietro, ma poi ho avere il problema dei tipi di riferimento non clonati, ecc.

C'è un modo per fare quanto sopra più elegantemente della copia di _collection su un'altra variabile di lista?

Answer 1

_collection.RemoveAll(user => !user.IsApproved()); 

Se sei ancora sul 2.0:

_collection.RemoveAll(delegate(User u) { return !u.IsApproved(); }); 

A proposito, se non si vuole toccare la lista originale, è possibile ottenere un altro elenco delle approvato utenti con:

_collection.FindAll(user => user.IsApproved()); 

Answer 2

È sempre possibile iniziare dall'indice superiore e scorrere verso il basso verso 0:

for (int i = _collection.Count - 1; i >= 0; i--) 
{ 
    User user = _collection[i]; 
    if (!user.IsApproved()) 
    { 
     _collection.RemoveAt(i); 
    } 
} 

risposta di Mehrdad sembra maledettamente elegante, però.

Answer 3

Ogni volta che c'è una possibilità che una collezione sarà modificato in un ciclo, optare per un ciclo for invece. La soluzione offerta da Mehrdad è adorabile e sicuramente vale la pena provare!

Ecco il codice che trovo utile quando si tratta di collezioni modificabili:

for(int index=0;index < _collection.Count; index++) 
{ 
    if (!_collection[index].IsApproved) 
    { 
     _collection.RemoveAt(index); 
     index--; 
    } 
}