LINQ式の実行

Question

以下のLINQ式を使用してデータテーブルを作成しようとしていますが、これは問題なく動作しますが、ハックのように感じます。これを行うより良い方法はありますか？

var Records = <Dictionary object with some data.> 

DataTable objDataTable = null; 
objDataTable.Columns.Add("Column1", typeof (string)); 
objDataTable.Columns.Add("Column2", typeof (string)); 

// Have to perform .Count() to execute the linq expression in order to 
// add datarows in datatable 

Records.Keys.Select(rec => objDataTable.Rows.Add(new object[] {rec, Records[rec]})).Count(); 

私は文を実行するために、最後に.Count()までに持っている上に、あなたが見ることができるように

。先に私は .ToArray()をやっていたが、私は .Count()が安くなると思った。

いずれにしても、これはあまり読みにくくはありません。

Answer 1

はい - だけ完璧に読めるし、このために適したforeachループを使用します - 特にあなたが LINQのは、あなたが常にがそうすることを強要されなければならないという意味ではありません使用することができるからといって

foreach(var rec in Records.Keys) 
{ 
    objDataTable.Rows.Add(new object[] {rec, Records[rec]}); 
} 

を副作用のあるコードがある場合

Answer 2

代わりForEachを使用してください：あなたは常にあなたが反復処理の各キーの値を調べるよう

Records.Keys.ForEach(rec => objDataTable.Rows.Add(new object[] {rec, Records[rec]})); 

Answer 3

を見て、私はちょうど辞書自体を列挙言うと思います：ここで

foreach(var kvp in Records) 
{ 
    objDataTable.Rows.Add(new object[] {kvp.Key, kvp.Value}); 
} 

kvpKeyValuePair<string, string>です。私の目には、のエントリを辞書に入れて何かしたいことが明確になります。