meilleure façon de le faire foreach en utilisant linq

Question

Hey là, J'ai cette boucle foreach qui ne fonctionne pas à 100%. Fondamentalement, je suis en train de sortir une chaîne. Mon problème est que je ne veux pas sb.Append (","); ajouter le dernier enregistrement de la boucle. Y at-il un moyen facile d'utiliser linq pour résoudre ce problème?

sb.Append("Readings:["); 
        foreach (var reading in lake.Reading) 
        { 
         sb.Append("["); 
         sb.Append("\""); 
         sb.Append(reading.DateTime.ToString("dd-MMM-yy")); 
         sb.Append("\""); 
         sb.Append(","); 
         sb.Append(reading.Level); 
         sb.Append("]"); 
       //only insert this line if its the not the last record  sb.Append(","); 
        } 
       sb.Append("]"); 

Answer 1

Vous pouvez utiliser comme normal pour boucle

for (int i = 0; i < lake.Reading.count; i ++)

Answer 2

créer un tableau de lake.Reading avec le ToArray () méthode et changer le foreach en un. Il suffit de tester votre index par rapport au nombre d'éléments dans le tableau.

sb.Append("Readings:["); 
Array list = late.Reading.ToArray(); 
       for (int i=0;i<list.Length;i++) 
       { 
        var reading = list[i]; 
        sb.Append("["); 
        sb.Append("\""); 
        sb.Append(reading.DateTime.ToString("dd-MMM-yy")); 
        sb.Append("\""); 
        sb.Append(","); 
        sb.Append(reading.Level); 
        sb.Append("]"); 
        if (i!=list.Length-1) 
         sb.Append(","); 
       } 
      sb.Append("]"); 

Answer 3

pas une approche LINQ, mais vous pouvez le faire:

sb.Append("Readings:["); 

bool isFirst = true; 
foreach (var reading in lake.Reading) 
{ 
    if(isFirst == false) 
    { 
     sb.Append(","); 
    } 
    isFirst = false; 

    sb.Append("["); 
    sb.Append("\""); 
    sb.Append(reading.DateTime.ToString("dd-MMM-yy")); 
    sb.Append("\""); 
    sb.Append(","); 
    sb.Append(reading.Level); 
    sb.Append("]"); 
} 
sb.Append("]"); 

Même avec Linq vous devez vérifier si vous êtes soit sur le premier élément ou le dernier.

Answer 4

Vous devriez regarder en utilisant String.Join() ou String.Concat(). Il rend les listes (séparées par des virgules) comme des chaînes beaucoup plus faciles. Et fonctionne bien avec LINQ.

var combine = from r in lake.Reading 
       select String.Format("[\"{0:dd-MMM-yy}\",{1}]", r.DateTime, r.Level); 
var str = String.Format("Readings:[{0}]", String.Join(",", combine)); 

Answer 5

Si vous voulez être concis, vous pouvez en fait utiliser linq. Mais événement si c'est une solution, ce n'est peut-être pas le meilleur. Quelque chose comme ça devrait fonctionner:

lake.Reading.Aggregate(new StringBuilder(), 
    (acc, value) => { return acc.AppendFormat("[{0},{1}],",value.Reading,value.Level); }, 
    acc => acc.Length > 1 ? (acc.Remove(acc.Length-1, 1).ToString()):string.Empty 
); 

Mais string.join ferait l'affaire mieux que je pense

Answer 6

J'ai résolu ces trois façons.

• Coupez la substance indésirable après la boucle.

• Utilisez un "commencé" bool, par exemple,

 
    // Warning -- Air code fragment 
    bool started = false; 
    foreach (...) { 
     if (!started) 
      started = true; 
     else 
      sb.Append(","); 
     // Rest of the loop 
    } 

• créé un wrapper pour StringBuilder qui fait le truc "a commencé".

Answer 7

Je généralement utiliser quelque chose comme:

string s = lake.Reading 
    .Select(r => "[\"" + r.DateTime.ToString("dd-MMM-yy") + "\"," + r.Level + "]") 
    .Aggregate((n, m) => n + "," + m); 

Answer 8

Ce n'est pas vraiment LINQ qui fait la chose particulière que vous demandez, mais string.Join:

var formattedReadings = 
    from reading in lake.Reading 
    select string.Format("[\"{0}\",{1}]", 
     reading.DateTime.ToString("dd-MMM-yy"), 
     reading.Level); 

sb.Append("Readings:["); 
sb.Append(string.Join(",", formattedReadings)); 
sb.Append("]");