がこのスニペットから重複を除去するのにアドバイスをお探しのコード

Question

のこのスニペットから重複を削除します。

foreach (Car car in carList) { 
    DataRow row = NewRow(); 

    StringBuilder sbConfigurations = new StringBuilder();  
    foreach (ConfigurationItem configurationItem in car.Configurations) 
     sbConfigurations.AppendFormat("{0}: {1}\n", configurationItem.Name, configurationItem.Value); 

    row["configurations"] = sbConfigurations; 


    StringBuilder sbOptionals = new StringBuilder(); 
    foreach (OptionalItem optionalItem in car.Optionals) 
     sbOptionals.AppendFormat("{0}, ", optionalItem.Name); 

    row["optionals"] = sbOptionals; 

    Rows.Add(row); 
} 

EDIT：これは単純化文脈で、より多くのがあるかもしれないリスト加入これら

Answer 1

私は多くの重複がない同意するが、おそらくLINQの拡張 にこの「直訳は」あなたは（ブラウザタイプなので、テストしていません - まだ）探しているものです。

foreach (Car car in carList) { 
    DataRow row = NewRow(); 

    row["configurations"] = car.Configurations.Aggregate(new StringBuilder(), (a,i) => a.AppendFormat("{0}: {1}\n", i.Name, i.Value)); 
    row["optionals"] = car.Optionals.Aggregate(new StringBuilder(), (a,i) => a.AppendFormat("{0}, ", i.Name)); 

    Rows.Add(row); 
} 

また、あなたは多少読みやすく、それを書くことができstringbuildersなし（効率的/？）：ラムダを抽出

はラインを短く：

Func<Car, string> nameValue = car => string.Format("{0}: {1}\n", car.Name, car.Value); 

foreach (var car in carList) { 
    var row = new Dictionary<string, string>(); 

    row["configurations"] = string.Join("\n", car.Configurations.Select(nameValue)); 
    row["optionals"]  = string.Join(", ", car.Optionals.Select(i => i.Name)); 

    list.Add(row); 
} 

_{注 C＃4.0前に、逆さま問題を回しについてstring.Join}

Answer 2

Iドンのような本当にあなたが "複製"が必要であると言っているものを取り除くとは思わない。同一のコードが2回出現するのではなく、同じようなコードが2回出現します。これは一般的であり、心配することはありません。

Answer 3

方法の2番目のパラメータに関する追加.ToArray()の呼び出しを必要とします。関数をそれぞれのオブジェクトに対してDataRow形式を理解させる代わりに、各オブジェクトにDataRow形式を理解させるようにしてください。あなたは何のためにobject.ToString()を使用していない場合は、ConfigurationItemを作ることができるとOptionalItemobject.ToString()を実装：

class ConfigurationItem 
{ 
    public string override ToString() 
    { 
     return string.Format("{0}: {1}\n", Name, Value); 
    } 
} 

class OptionalItem 
{ 
    public string override ToString() 
    { 
     return string.Format("{0}, ", Name); 
    } 
} 

今、あなたは、オブジェクトのすべてのタイプのための単一のループを使用することができます：あなたがobject.ToString()が必要な場合は

string BuildDataRowString(IEnumerable collection) 
{ 
    var sb = new StringBuilder(); 
    foreach (var o in collection) sb.Append(o.ToString()); 
    return sb.ToString(); 
} 

row["configurations"] = car.Configurations.BuildDataRowString(); 
row["optionals"] = car.Optionals.BuildDataRowString(); 

を他の目的で、「DataRowフォーマット」のカスタムフォーマットを追加することができます：

class ConfigurationItem : IFormattable 
{ 
    public string override ToString(string format, IFormatProvider formatProvider) 
    { 
     if (format == "D") { 
      return string.Format(formatProvider, "{0}: {1}\n", Name, Value); 
     } 
     return this.ToString(); // otherwise format as default 
    } 
} 

class OptionalItem : IFormattable 
{ 
    public string override ToString(string format, IFormatProvider formatProvider) 
    { 
     if (format == "D") { 
      return string.Format(formatProvider, "{0}, ", Name); 
     } 
     return this.ToString(); // otherwise format as default 
    } 
} 

string BuildDataRowString(this IEnumerable e, string format) 
{ 
    StringBuilder sb = new StringBuilder(); 
    foreach (var o in e) sb.AppendFormat("{0:D}", o); 
    return sb.ToString(); 
}