Comment déplacer des éléments d'une liste vers une autre liste en C#?

Question

Quel est le moyen préférable pour transférer certains éléments (pas tous) d'une liste à une autre.

Ce que je fais est la suivante:

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

otherList.AddRange(selected); 

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

Dans l'intérêt d'avoir le plus rapide/meilleur code il y a, est-il une meilleure façon?

Answer 1

Je vais essayer @ la réponse de Mehrdad, et peut-être tester contre celui-ci aussi ...

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

Answer 2

Je propose:

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

Answer 3

C'est sage tout à fait mauvaise performance - il en fait une fois énumère n de requête (pour n articles dans items). Il serait préférable que vous construisiez (par exemple) un HashSet<T> des éléments à manipuler.

Pour donner un exemple simple avec juste int valeurs:

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 va creux chaque élément et énumère toutes les valeurs de votre liste sélectionnée à chaque fois. Cela prendra plus longtemps que prévu ...

Ce que je ferais est mis l'état directement dans le paramètre RemoveAll:

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

Si vous faites cela et ne modifiez pas le reste de votre code il y aurait duplication de code, ce qui n'est pas bon. Je fais ce qui suit pour l'éviter:

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

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

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

Answer 5

Que diriez-vous d'une partition:

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];