¿Cómo muevo los artículos de una lista a otra lista en C#?

Question

¿Cuál es la forma preferible para transferir algunos elementos (no todos) de una lista a otra?

Lo que estoy haciendo es el siguiente:

var selected = from item in items 
       where item.something > 10 
       select item; 

otherList.AddRange(selected); 

items.RemoveAll(item => selected.Contains(item)); 

Con el fin de tener el más rápido mejor código/no existe, hay una manera mejor?

Answer 1

Me gustaría probar la respuesta de @ Mehrdad, y tal vez la prueba contra éste también ...

var selected = items.Where(item => item.Something > 10).ToList(); 
selected.ForEach(item => items.Remove(item)); 
otherList.AddRange(selected); 

Answer 2

sugiero:

var selected = items.Where(item => item.Something > 10).ToList(); 
items = items.Except(selected).ToList(); 
otherList.AddRange(selected); 

Answer 3

Eso es bastante mala actuación sabia - que en realidad enumera una consulta n veces (para n elementos en items). Sería mejor si construyes (por ejemplo) un HashSet<T> de los elementos para manipular.

Para dar un ejemplo sencillo simplemente con int valores:

var items = new List<int> { 1, 2, 3, 4, 5, 6 }; 
    var otherList = new List<int>(); 
    var selected = new HashSet<int>(items.Where(
     item => item > 3)); 
    otherList.AddRange(selected); 
    items.RemoveAll(selected.Contains); 

Answer 4

RemoveAll pasa a través cada elemento y enumera todos los valores de su lista seleccionada cada vez. Esto tomará más tiempo de lo que debería ...

lo que he hecho es poner la condición directamente en el parámetro RemoveAll:

items.RemoveAll(item => item.something > 10); 

Si usted hace esto y no cambia el resto de su código habría duplicación de código, lo cual no es bueno. Yo haría lo siguiente para evitarlo:

Func<ItemType, bool> selectedCondition = (item => item.something > 10); 

otherList.AddRange(items.Where(selectedCondition)); 

items.RemoveAll(new Predicate<ItemType>(selectedCondition)); 

Answer 5

¿Qué tal una partición:

int[] items = { 5, 4, 1, 3, 9, 8, 6, 7, 2, 0 }; 
var partition = items.ToLookup(x => x > 5); 
var part1 = partition[true]; 
var part2 = partition[false];