3


1

linqを使用するときにネストされたforeachループを取り除く

ネストされたforeachループを頻繁に使用するlinq式を常に作成しています。 以下は、私が話していることの簡単な例です。ここにいる誰かがこの低効率コードを単一のlinq式に凝縮する方法を教えてくれたら本当に感謝しています。

データベースコンテキスト(db)には、Blog、Tag、Junc_Tag_Blogの3つのテーブルがあります。 ジャンクションテーブルは、タグの付いたブログのレコードを保存するだけです。

とにかく、ここに私の厄介なコードがあります:

public static Collection GetByTag(string tagName)
{
    // Get the tag record.
    var tag = (from t in db.Tags
              where t.Name == tagName
              select t).Single();

    // Get the list of all junction table records.
    var tagJunc = from tj in db.Junc_Tag_Blogs
                  where tj.Fk_Tag_Id == tag.Id
                  select tj;

    // Get a list of all blogs.
    var blogs = from b in db.BlogPosts
                select b;

    // Work out if each blog is associated with given tag.
    foreach(var blog in blogs)
    {
        foreach(var junc in tagJunc)
        {
            if(blog.Id == junc.Fk_Blog_Id)
            {
                // We have a match! - do something with this result.
            }
        }
    }
}

このコードのクリーンアップを手伝ってくれる人に感謝します!

4 Answer


4


データベースがあなたのための一致を見つけることを可能にするクエリを構築できます。

List blogs =
(
  from t in tag
  where t.Name == tagName
  from tj in t.Junc_Tag_Blogs
  let b = tj.Blog
  select b
).ToList();


1


var blogsWithGivenTag =
    from blog in db.BlogPosts
    where blog.BlogTags.Any(bt => bt.Tag.Name == tagName)
    select blog;


0


LINQとの内部結合の実行について読んでください。


0


ブログを辞書に入れ、ブログIDでジャンクションをグループ化し、グループをループできます。

var blogDict = blogs.ToDictionary(b => b.Id);

foreach(var group in tagJunk.GroupBy(j => j.Fk_Blog_Id)) {
  if (blogDict.ContainsKey(group.Key)) {
    var blog = blogDict[group.Key];
    foreach (var junction in group) {
      // here you have the blog and the junction
    }
  }
}

これもネストされたループですが、各ブログでは、すべてのジャンクションではなく、そのブログに実際に属するジャンクションのみをループします。