【.NET Core】EFCoreから取得したリストを別のリスト条件で絞込みしたい

Question

前提

Visual Studio2019 Community
ASP.NET Core MVC
.NET 5.0

初学者です。上記の開発環境で日報アプリを作成しています。
以下のようなテーブル構成です。

DBから取得した日報リストの絞り込み方法が分からないため、ご助言頂ければと思い、質問させていただきます。

実現したいこと

コントローラーにgroupIDが渡されているとき、グループに紐づけられたメンバーの日報を取得したいです。

public async Task<IActionResult> Index(int? groupID )
{
    var reports = _context.Reports;  //groupID
    if(groupID!=null){
        var group = _context.Groups.FindAsync(groupID);
        //groupIDから紐づけたユーザーの絞り込み処理
    }
    reports.AsQueryable();

    return View();
}

※using、コンストラクタなど、質問に直接関係のない箇所は省略しています。

試したこと

上記ユーザーの絞り込み処理の箇所に、以下のように書いたのですが、絞り込めていません。

    reports.Where(s=>s.ReportUser.Members == group.Members);

補足情報

各テーブル(Model)を載せておきます。

//日報テーブル
public class Report
    {
        public int ReportID { get; set; }

        public string ReportUserId { get; set; }
        public ReportUser ReportUser { get; set; }

        public bool IsSubmit { get; set; }
    }

//ユーザーテーブル
public class ReportUser:IdentityUser
    {
        public string Name { get; set; }
        
        public ICollection<Report> Reports { get; set; }
        public ICollection<Member> Members { get; set; }
    }

//メンバーテーブル
public class Member
    {
        public int MemberID { get; set; }

        public int GroupID { get; set; }
        public Group Group { get; set; }

        public string ReportUserId { get; set; }
        public ReportUser ReportUser { get; set; }
    }

//グループテーブル
public class Group
    {
        public int GroupID { get; set; }

        public int CompanyID { get; set; }
        public Company Company { get; set; }

        public string GroupName { get; set; }

        public ICollection<Member> Members { get; set; }
    }

不足している情報があれば、追記しますので、よろしくお願い致します。

Comments

[SurferOnWww]

ReportViewController/Index のコードが意味不明ですが説明してもらえませんか？

コードの各クラスが質問の図のどのテーブルに該当するか書いてください。

ReportUser クラスが継承している IdentityUser とは何ですか？

アプリはどこまで自分で作ったのですか？　他人が作ったあまり理解できてないコードをいじっているように見えますけど・・・　

[akarinrin_]

回答ありがとうございます。

＞ReportViewController/Index のコードが意味不明ですが説明してもらえませんか？
var user = await _userManager.GetUserAsync(User);//質問とは関係のないコードでしたので、削除します。
var reports = _context.Reports; 日報テーブルから日報全てを取得
if(groupID!=null){
var group = _context.Groups.FindAsync(groupID);　//グループテーブルからgroupIDで検索
//groupに紐づいたユーザーの絞り込み処理　
}
reports.AsQueryable();　//Queryableで取得

ReportViewControllerのIndexメソッドの中身が混沌としているので、今回の質問の回答に必要と思われる箇所のみピックアップしています。

＞コードの各クラスが質問の図のどのテーブルに該当するか書いてください。
追記します。

＞ReportUser クラスが継承している IdentityUser とは何ですか？
こちらを参考にしました。
Microsoft.AspNetCore.Identity名前空間のIdentityUserを継承しています。
https://www.c-sharpcorner.com/article/registration-razor-page-in-asp-net-core-razor-pages-web-application/

全て自分で作っています。

[SurferOnWww]

> ReportViewControllerのIndexメソッドの中身が混沌としているので、今回の質問の回答に必要と思われる箇所のみピックアップしています。

やりたいことは「グループに紐づけられたメンバーの日報を取得」ですよね。質問に書かれたコードにはそれが全く書かれてないようですが？

> Microsoft.AspNetCore.Identity名前空間のIdentityUserを継承しています。

ASP.NET Core Identity を使ったユーザー認証を実装しているということのようですが、それと今回の質問とどう関係があるのでしょうか？　今回の質問の範囲では関係ないから気にしなくて良いのですか？

ナビゲーションプロパティが定義されているようですが、それを使ってグループに属するユーザーを得ることはできないのですか？

[akarinrin_]

引き続き、ありがとうございます。

＞やりたいことは「グループに紐づけられたメンバーの日報を取得」ですよね。質問に書かれたコードにはそれが全く書かれてないようですが？
回答の方に追記させて頂きました。

＞今回の質問の範囲では関係ないから気にしなくて良いのですか？
はい、モデルの相関を示す上で記載したため、無視して頂いて大丈夫です。

[SurferOnWww]

私の回答のコメントの私の質問にもタイムリーに答えてもらえませんか？　時間が経つと忘れてしまいます。

Answer 1

質問のコメントに返事がないので勝手に想像して答えておきます。

質問の図でははっきりしませんが、質問者さんの「ユーザー」、「メンバー」、「グループ」は、以下の記事の図の Student、Enrollment、Course とほぼ同じで、ナビゲーションプロパティもほぼ同じように実装されているのではないですか。

データ モデルを作成する
https://docs.microsoft.com/ja-jp/aspnet/core/data/ef-mvc/intro?view=aspnetcore-5.0#create-the-data-model

であれば、ナビゲーションプロパティをたどって、特定のグループに属するユーザーを取得できるはずです。

上に紹介した記事の Student、Enrollment、Course を使った場合、特定の Course に属する Student は以下のようにして取得できます。Student = ユーザー、Enrollment = メンバー、Course = グループと思ってください。

質問者さんのケースで「ユーザー」の先にある「日報」も、同様に「ユーザー」のナビゲーションプロパティをたどって取得できると思います。

Comments

[akarinrin_]

質問が不明確で大変失礼しました。
日報を取得したいと書いていますが、正しくは日報リスト()を取得したいです。

＞ナビゲーションプロパティもほぼ同じように実装されているのではないですか。
その通りです。公式チュートリアルを元に実装しました。

＞特定のグループに属するユーザーを取得できるはず
var reportlist = new List<Report>();
foreach(var member in group.Members)
{
reportlist.AddRange(member.ReportUser.Reports.ToList());
}
というようにナビゲーションプロパティを辿って日報リストを取得する方法は分かるのですが、
グループに属した人の日報リストをforeachなど使わずにまとめて、取得する条件式の書き方はないということでしょうか。

[SurferOnWww]

何を言ってるのか分かりません。

> コントローラーにgroupIDが渡されているとき、グループに紐づけられたメンバーの日報を取得したいです。

という話ではなかったの？　その手段の話だったの？　「日報リストをforeachなど使わずにまとめて、取得する条件式の書き方」って何ですか？

[SurferOnWww]

こちらの質問にもタイムリーに答えてもらえませんか？　時間が経つと忘れてしまいます。

[akarinrin_]

回答が遅くなり申し訳ございません。
>日報リストをforeachなど使わずにまとめて、取得する条件式の書き方
ナビゲーションプロパティを辿る際、
GroupIDからグループを取得して辿る方法と
日報リストを全取得→ユーザー→メンバー→グループと辿り、グループで絞り込みすることで取得する方法の2種類あると思ったのですが、後者の取得のことを指しています。

[SurferOnWww]

> ナビゲーションプロパティを辿る際、GroupIDからグループを取得して辿る方法と日報リストを全取得→ユーザー→メンバー→グループと辿り、グループで絞り込みすることで取得する方法の2種類あると思ったのですが、後者の取得のことを指しています。

依然として意味が分かりませんが・・・

基本的に、groupID で指定した特定の Gruop に属する複数の ReportUser が書いた複数の Report を List<Report> として取得したいと想像してます。もし、違うなら具体的にどう違うのか書いてください。

想像が合っているとして・・・

私が回答で書いたように Include, ThenInclude を使って、Group にぶら下がっている Member, ReportUser, Report のすべてが取得できていれば（あなたの書いたコードでは Include, ThenInclude を使ってないのでそうは思えないのですが）、あとは取得できた Gruop をどう調理してその配下にある複数の Report を List<Report> として取得するかという話になるはずです。

それには、あなたが考えた foreach を使うとか SelectMany を使って望むことができるはず。これは前者の方法「GroupIDからグループを取得して辿る方法」になると思いますが、それがベストだと思います。

あなたの言う後者の方法はまるで今までと話が違うのでは？　「日報リストを全取得」して SQL 文で言うと WHERE ユーザーID IN (...) とかで（Linq では Contains を使って） List<Report> を取得できると思いますが、何のメリットもないと思いますけど。WHERE ユーザーID IN (...) の ... を取得するために SELECT クエリを投げなければならないですよね。デメリットしかなさそうですけど。

[akarinrin_]

＞あなたが考えた foreach を使うとか SelectMany を使って望むことができるはず。これは前者の方法「GroupIDからグループを取得して辿る方法」になると思いますが、それがベストだと思います。
ありがとうございました。
こちらの回答で納得しました。