LINQ: Comment concaténer une liste d'entiers en délimitée par des virgules de la chaîne?

C'est probablement quelque chose de stupide je l'ai raté, mais j'essaie de concaténer une liste d'entiers au lieu d'additionner avec:

integerArray.Aggregate((accumulator, piece) => accumulator+"," + piece)

Le compilateur se plaint à propos de l'argument de l'erreur. Est-il un habile moyen de le faire sans avoir à passer par une boucle?

InformationsquelleAutor Haoest | 2010-05-26
  1. 66

    La version de .NET? Dans la version 4.0, vous pouvez utiliser string.Join(",",integerArray). En 3.5, je serais tenté d'utiliser string.Join(",",Array.ConvertAll(integerArray,i=>i.ToString())); (en supposant qu'il est un tableau). Sinon, soit faire il un tableau, ou d'utiliser StringBuilder.

    • ne serait-il pas toujours être la meilleure perf sage d'utiliser un StringBuilder dans ce cas?
    • Curieusement, la Jointure est plus performant que le StringBuilder. J'ai fait un test, et avec un tableau de 1 000 000 de taille, de Rejoindre a pris 207ms, tandis que StringBuilder a pris 264ms. Je soupçonne que c'est parce que la Jointure est optimisé pour la tâche, alors que le SB est plus optimisé pour le cas général.
    • Si je n'avais pas un Array type pour commencer, est-il une différence de vitesse entre Array.ConvertAll(list.ToArray(),...) contre IEnumerable.Select(...).ToArray()?
    • la principale différence est qu'il pourraient avoir besoin d'un intermédiaire de tableaux et de bloquer les copies alors qu'il trouve la longueur. Mais dans la plupart des cas, vous serez amende.
    • La classe StringBuilder sans doute d'accélérer considérablement si vous avez créé avec une assez grande capacité initiale.
    InformationsquelleAutor Marc Gravell
  2. 15

    L'erreur que vous obtenez est parce que vous n'avez pas d'utiliser le remplacement de Aggregate qui vous permet de spécifier la valeur de départ. Si vous ne spécifiez pas de la graine, il utilise le type de la collection.

    integerArray.Aggregate("", (accumulator, piece) => accumulator + "," + piece);
    • +1 parce que vous avez répondu à la question. Cependant, votre code a O(n^2) les performances. Pas cool.
    • maintenant, comptez le nombre de chaînes concernées...
    • n'est rien mais une boucle foreach qui définit la valeur d'une variable. - C'est O(n) non O(n^2)?
    • Le moteur d'exécution de 'accumulateur + ..." est proportionnelle à la longueur de la valeur dans l'accumulateur. La longueur de l'accumulateur chaîne grandit avec chaque invocation. Le code est d'effectuer un O(n) opération n fois. Voilà comment il est O(n^2).
    • parce que la graine est "", ce serait de retourner une chaîne de caractères avec la première valeur d'une virgule , par exemple ,1,2,3,4 . Une solution serait de revenir integerArray.First() + integerArray.Skip(1).Aggregate("", (accumulator, piece) => accumulator + "," + piece);
    • le plus simple moyen de se débarrasser de l'extra ajouté virgule ::: integerArray.D'agrégation( "",(x, y) => string.Concat(x,",", y)).Substring(1)

    InformationsquelleAutor Samuel
  3. 14

    Vous voudrez probablement utiliser Chaîne de caractères.Rejoindre.

    string.Join(",", integerArray.Select(i => i.ToString()).ToArray());

    Si vous êtes en utilisant .Net 4.0, vous n'avez pas besoin de passer par les tracas de la chosification d'un tableau. et peut juste faire

     string.Join(",", integerArray);
    • Slick! Vraiment utile.
    InformationsquelleAutor 48klocs
  4. 2

    Juste pour ajouter une autre alternative à @Marc

    var list = string.Join( ",", integerArray.Select( i => i.ToString() ).ToArray() );
    InformationsquelleAutor tvanfosson