非効率的なLINQ Foreachステートメントタイムアウト

Question

私のforeachステートメントをループするときに、アプリケーションがタイムアウトエラーを返す問題があります。ご注意ください私は変更したり、私が照会しているテーブルのビューを作成することはできません。

IEnumerable<MyTable> TContextData; 
if (!string.IsNullOrEmpty(id) 
    || !string.IsNullOrEmpty(name) 
    || !string.IsNullOrEmpty(cell) 
    ) 
{ 
    TContextData = (from e in db.mytable 
        where (!String.IsNullOrEmpty(e.id)) 
        select e); 
} 
else 
{ 
    TContextData = null; 
} 

List<EachC> mDatalist = new List<EachC>(); 

if (TContextData != null) 
{ 
    foreach (var i in TContextData.ToList()) 
    { 
     EachMR EachM = new EachMR(); 

     EachM.Feature = i.TRuid; 
     EachM.Feature2 = i.TRuid2; 
     EachM.Feature3 = i.TRuid3; 
     EachM.Date = GetInnerTextfromXML(EachM.Feature3, "/Sections/Multiple/Functions/Group/Date"); 
     mdatalist.Add(EachM); 
    } 
} 

反復する行は約40Kです。これをより効率的にする方法はありますか？

エラーメッセージ：型「System.OutOfMemoryExceptionに」の例外がSystem.Data.Entity.dllで発生したが、ユーザーコードで処理されなかった

は私が

にコンテキストにタイムアウトを増加していました

((IObjectContextAdapter)this).ObjectContext.CommandTimeout = 0; 

しかし、それはまさにマスキングの問題ではありません。

Answer 1

正確なメッセージがなければ、タイムアウトがどこから来ているのかは分かりません。しかし、私はそれがSQLタイムアウトではないと思います。

実際にコードが実行しているのは、foreachで取得しようとしている各行のデータベースに対してクエリを実行することです。つまり、おそらく40,000クエリを実行している可能性があります。代わりに、データをメモリにロードして、.ToList()メソッドを使用してリスト上でコードを実行すると、クエリが1回実行されます。

IEnumerable TContextData; 

if (!string.IsNullOrEmpty(id) 
|| !string.IsNullOrEmpty(name) 
|| !string.IsNullOrEmpty(cell)) 
{ 
    TContextData = (from e in db.mytable where (!string.IsNullOrEmpty(e.id)) select e); 
} 

foreach(var i in TContextData.ToList()) 
{  
    //do stuff 
} 

また、私はすべてのコンテキストを持たずに話しています。あなたは.ToList()を助けることができるの削除EF、LinqToSQL、またはIEnumerableに変換DataReader ...

Answer 2

を使用している場合、答えは少し違うだろう。あなたのコードは、クエリーがリスト内で返すものすべてを不必要にバッファし、そのリストを反復処理します。 TContextDataをループするだけです。