Une autre question Linqification

Question

Dites que j'ai une liste d'objets, et l'objet a une propriété de chaîne. Je veux obtenir une seule liste séparée par des virgules de la valeur de chaque propriété de chaîne de chaque objet dans la liste. est ici 1 façon de le faire (sans LINQ)

 

StringBuilder result = new StringBuilder() 
foreach(myObject obj in myList) 
{ 
    result.Append(obj.TheString); 
    result.Append(", "); 
} 
// then trim the trailing ", " and call ToString() on result, etc, etc... 
 

Voici mon premier coup linqification. Y a-t-il un meilleur moyen?

 

string result = string.Join(", ", myList.Select(myObj => myObj.TheString).ToArray()); 
 

C'est une ligne de code, mais il ne semble pas très efficace pour moi - itérer la liste juste pour construire un tableau, juste pour itérer le tableau et construire une chaîne ... ouf!

Y a-t-il un meilleur moyen?

Answer 1

Si vous voulez efficace, utilisez Enumerable.Aggregate avec StringBuilder:

string result = myList.Aggregate(new StringBuilder(), 
           (sb, o) => sb.Append(o.TheString).Append(", ")) 
         .ToString(); 

Le problème initial est que String.Join veut un tableau. En .NET 4, il y aura une surcharge qui prendra IEnumerable<string> (et je m'attends à ce qu'il soit implémenté comme ci-dessus).

Answer 2

J'aime cette méthode d'extension pour joindre des chaînes. C'est essentiellement la même technique que vous utilisez, mais enveloppé dans une méthode d'extension. Je ne le recommanderais pas pour les grands ensembles puisque l'efficacité n'était pas l'objectif. L'avantage pour moi est expressivité (très linqy) et pratique pour les petits ensembles:

[Test] 
public void Should_make_comma_delimited_list() 
{ 
    var colors = new List<HasColor> 
    { 
     new HasColor { Color = "red" }, 
     new HasColor { Color = "green" }, 
     new HasColor { Color = "blue" } 
    }; 

    var result = colors.Implode(x => x.Color, ", "); 

    Assert.That(result, Is.EqualTo("red, green, blue")); 
} 

public class HasColor 
{ 
    public string Color { get; set; } 
} 

public static class LinqExtensions 
{ 
    public static string Implode<T>(this IEnumerable<T> list, Func<T, string> func, string separator) 
    { 
     return string.Join(separator, list.Select(func).ToArray()); 
    } 
} 

Answer 3

est ici une autre façon (result est un StringBuilder):

myList.ForEach(
    myObj => 
    { 
     if (result.Length != 0) 
      result.Append(","); 
     result.Append(myObj.TheString); 
    } 
    ); 

Answer 4

Utilisez string.join, il est assez bon.

Optimisez lorsque profiler vous dira de le faire.

La lisibilité de la version avec StringBuilder est médiocre et vous obtenez toujours la virgule de fin.