ASP.NET CoreのViewModelで、元のViewとDBを共有したい。

Question

こんにちは。
ASP.NET Coreについて初学者なので拙いところもあると思いますが、よろしくお願いします。

実現したいこと

具体例でお話しします。
例えばAnimalとPlantいうModelがあり、それを元にスキャフォールディングしたViewとControllerが存在しているとします。
この時データベースにはDbInitializerで追加した初期データか、実際にブラウザ上でCreateしたデータが入っています。

その後、AnimalとPlantという2つのModelを元にしてLivingViewModelを作成するとします。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;


namespace WebApplication.Models
{
    public class LivingViewModel
         
    {
        public IEnumerable<Animal> Animals { get; set; }
        public IEnumerable<Plant> Plants { get; set; }
    }
}

このとき、AnimalもしくはPlantのページで追加したデータや初期データをViewModelのページに反映させる、またLivingViewModelの方で編集したデータをAnimalもしくはPlantのページに反映させるControllerの書き方がわかりません。

以下AnimalのControllerです。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.Rendering;
using Microsoft.EntityFrameworkCore;
using WebApplication.Data;
using WebApplication.Models;

namespace WebApplication.Controllers
{
    public class AnimalsController : Controller
    {
        private readonly MyDbContext _context;

        public AnimalsController(MyDbContext context)
        {
            _context = context;
        }

        // GET: Animals
        public async Task<IActionResult> Index()
        {
            return View(await _context.Animals.ToListAsync());
        }

        // GET: Animals/Details/5
        public async Task<IActionResult> Details(int? id)
        {
            if (id == null)
            {
                return NotFound();
            }

            var Animal = await _context.Animals
                .FirstOrDefaultAsync(m => m.Id == id);
            if (Animal == null)
            {
                return NotFound();
            }

            return View(Animal);
        }

        // GET: Animals/Create
        public IActionResult Create()
        {
            return View();
        }

        // POST: Animals/Create
        // To protect from overposting attacks, please enable the specific properties you want to bind to, for 
        // more details see http://go.microsoft.com/fwlink/?LinkId=317598.
        [HttpPost]
        [ValidateAntiForgeryToken]
        public async Task<IActionResult> Create([Bind("Id,Name")] Animal Animal)
        {
            if (ModelState.IsValid)
            {
                _context.Add(Animal);
                await _context.SaveChangesAsync();
                return RedirectToAction(nameof(Index));
            }
            return View(Animal);
        }

        // GET: Animals/Edit/5
        public async Task<IActionResult> Edit(int? id)
        {
            if (id == null)
            {
                return NotFound();
            }

            var Animal = await _context.Animals.FindAsync(id);
            if (Animal == null)
            {
                return NotFound();
            }
            return View(Animal);
        }

        // POST: Animals/Edit/5
        // To protect from overposting attacks, please enable the specific properties you want to bind to, for 
        // more details see http://go.microsoft.com/fwlink/?LinkId=317598.
        [HttpPost]
        [ValidateAntiForgeryToken]
        public async Task<IActionResult> Edit(int id, [Bind("Id,Name")] Animal Animal)
        {
            if (id != Animal.Id)
            {
                return NotFound();
            }

            if (ModelState.IsValid)
            {
                try
                {
                    _context.Update(Animal);
                    await _context.SaveChangesAsync();
                }
                catch (DbUpdateConcurrencyException)
                {
                    if (!AnimalExists(Animal.Id))
                    {
                        return NotFound();
                    }
                    else
                    {
                        throw;
                    }
                }
                return RedirectToAction(nameof(Index));
            }
            return View(Animal);
        }

        // GET: Animals/Delete/5
        public async Task<IActionResult> Delete(int? id)
        {
            if (id == null)
            {
                return NotFound();
            }

            var Animal = await _context.Animals
                .FirstOrDefaultAsync(m => m.Id == id);
            if (Animal == null)
            {
                return NotFound();
            }

            return View(Animal);
        }

        // POST: Animals/Delete/5
        [HttpPost, ActionName("Delete")]
        [ValidateAntiForgeryToken]
        public async Task<IActionResult> DeleteConfirmed(int id)
        {
            var Animal = await _context.Animals.FindAsync(id);
            _context.Animals.Remove(Animal);
            await _context.SaveChangesAsync();
            return RedirectToAction(nameof(Index));
        }

        private bool AnimalExists(int id)
        {
            return _context.Animals.Any(e => e.Id == id);
        }
    }
}

AnimalやPlantのデータとは関係なくLivingViewModelに初期データを追加することは出来るのですが、相互に作用できるようにしたいです。

以下現在のViewModelのControllerです。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using Microsoft.AspNetCore.Mvc;
using Microsoft.EntityFrameworkCore;
using WebApplication.Data;
using WebApplication.Models;

namespace WebApplication.Controllers
{
    public class LivingViewModelsController : Controller
    {
        
        public IActionResult Index()
        {
            LivingViewModel myView = new LivingViewModel();
            myView.Animals = GetAnimals();
            myView.Plants = GetPlants();
            return View(myView);
        }

        private List<Animal> GetAnimals()
        {
            List<Animal> Animals = new List<Animal>();
            Animals.Add(new Animal { /*初期データ*/ });
            return Animals;

        }

        private List<Plant> GetPlants()
        {
            List<Plant> Plants = new List<Plant>();
            Plants.Add(new Plant { /*初期データ*/ });
            
            return Plants;
        }
    }
}

DbContext、DbInitializerは以下の通りです。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using DBAccessSample.Models;
using Microsoft.EntityFrameworkCore;

namespace WebApplication.Data
{
    public class MyDbContext : DbContext
    {
        public MyDbContext(DbContextOptions<MyDbContext> options) : base(options) { }

        public DbSet<Animal> Animals { get; set; }

        public DbSet<Plant> Plants { get; set; }
    }
}

sing System;
using System.Collections.Generic;
using System.Linq;
using System.Threading.Tasks;
using WebApplication.Models;
using Microsoft.EntityFrameworkCore;

namespace WebApplication.Data
{
    public class DbInitializer
    {
        public static void Initialize(MyDbContext context)
        {
            context.Database.EnsureCreated();

            if (context.ProgressTables.Any())
            {
                return;   // DB has been seeded
            }

            //Animal
            var animals = new Animal[]
            {
                new Animal{
                  //初期データ
                 }

            };
            foreach (var tmp in animals)
            {
                context.Animals.Add(tmp);
            }
            context.SaveChanges();
           
            //Plant
            var plants = new Plant[]
            {
                new Plant{
                  //初期データ
                 }

            };
            foreach (var tmp in plants)
            {
                context.Plants.Add(tmp);
            }
            context.SaveChanges();
        }
    }
}

疑問点が不明瞭で分かりにくいかと思いますが、よろしくお願いします。

参考URL

https://tech-blog.cloud-config.jp/2019-09-10-create-a-netcore-webapp-loosely-and-easily-part1/
https://qiita.com/KktkiY/items/f28528916e97310262e0

補足情報（FW/ツールのバージョンなど）

Visual Studio 2017 Community
Microsoft.AspNetCore.All 2.2.8
Windows10 Pro 1909

Comments

[退会済みユーザー]

> AnimalもしくはPlantのページで追加したデータや初期データをViewModelのページに反映させる、またLivingViewModelの方で編集したデータをAnimalもしくはPlantのページに反映させる

意味が分かりません。もう少し詳しく具体例など挙げて説明できませんか？

[K-actus]

AnimalのページにあるCreate Newのリンクからデータを作成した際に、AnimalのModelを継承しているはずのViewModelのページに追加したデータが反映されません。
その追加したデータが反映されるようなViewModelのControllerの書き方を教えてほしいです。

[退会済みユーザー]

「AnimalのページにあるCreate Newのリンク」とか言われても、質問に書いてないことなので分かりません。「AnimalのModelを継承しているはずのViewModelのページ」というのも質問に書いてないことなので分かりません。

[退会済みユーザー]

回答が必要なら、回答者が分からないと言っていることに対して追加情報を出して分かってもらえるようにする努力をしませんか。

[K-actus]

すみません返信が遅れました。
CreateNewのリンクはDbContextとModelでスキャフォールディングしたら自動で生成される認識なんですけど共通ではないんですか？
ViewModelのページについては2つ目の参考リンクを元にほぼそのまま実装しました。

[退会済みユーザー]

> CreateNewのリンクはDbContextとModelでスキャフォールディングしたら自動で生成される認識なんですけど共通ではないんですか？

自動生成されたコードの少なくとも Controller はコピペして質問欄に書いてください。回答者の想像力に期待しないできちんと質問に書いてください。想像で答えて、想像が違っていると、混乱を招くばかりです。ここ Teratail には回答者の想像の斜め上のことをしている質問者さんが多いのです。

「AnimalのModelを継承しているはずのViewModelのページ」というのも質問に書いてないので分かりません。こちらもコードをアップして説明してください。

[BluOxy]

> AnimalもしくはPlantのページで追加したデータや初期データをViewModelのページに反映させる、またLivingViewModelの方で編集したデータをAnimalもしくはPlantのページに反映させるControllerの書き方

Animal Plant と ViewModel をEF上でリレーションさせれば良いと思います。
もっとも、そうなるとViewModelという命名は不適切になると思いますが…。

[退会済みユーザー]

> 以下現在のViewModelのControllerです。

の LivingViewModelsController の Index では、実際には Animals.Add(new Animal { /*初期データ*/ }); と Plants.Add(new Plant { /*初期データ*/ }); のデータを表示したいのではなくて、

> 以下AnimalのControllerです。

の AnimalsController コードの Index で表示される Animal のレコード一覧の Table と、もう一つ他にあると思われる PlantController の Index で表示される Plant のレコード一覧の Table を、参考にしている記事 https://qiita.com/KktkiY/items/f28528916e97310262e0 にあるように並べて表示したいということですか？

そして、表示するだけでなく、LivingViewModelsController に DELETE, UPDATE, CREATE 機能を持たせたいと言ってますか？

[K-actus]

前半部分はその通りです。

>そして、表示するだけでなく、LivingViewModelsController に DELETE, UPDATE, CREATE 機能を持たせたいと言ってますか？
LivingViewModelsController にCreate, Update, Delete機能を持たせるのはあったら嬉しいくらいなので、
難しそうでしたらReadだけで構いません。

Answer 1

以下現在のViewModelのControllerです。

の LivingViewModelsController の Index では、実際には Animals.Add(new Animal { /初期データ/ }); と Plants.Add(new Plant { /初期データ/ }); のデータを表示したいのではなくて、

以下AnimalのControllerです。

の AnimalsController コードの Index で表示される Animal のレコード一覧の Table と、もう一つ他にあると思われる PlantController の Index で表示される Plant のレコード一覧の Table を、参考にしている記事 https://qiita.com/KktkiY/items/f28528916e97310262e0 にあるように並べて表示したいということだそうですので・・・

LivingViewModelsController のコードを以下のようにすればよさそうです。これが質問者さんのやりたいことと異なる場合は、どう異なるか下のコメント欄に書いてください。

namespace WebApplication.Controllers
{
    public class LivingViewModelsController : Controller
    {
        private readonly MyDbContext _context;

        public LivingViewModelsController(MyDbContext context)
        {
            _context = context;
        }

        public async Task<IActionResult> Index()
        {
            LivingViewModel myView = new LivingViewModel();
            myView.Animals = await _context.Animals.ToListAsync();
            myView.Plants = await _context.Plant.ToListAsync();
            return View(myView);
        }
    }
}

表示するだけでなく、LivingViewModelsController に DELETE, UPDATE, CREATE 機能を持たせるというのは、並べて表示する Table に、AnimalsController の Index、PlantController の Index で表示される Table と全く同じ Create, Edit, Delete, Detail のリンクを持たせて、AnimalsController, PlantController に飛ばしてそれで行うようにしてはいかがですか？

Comments

[退会済みユーザー]

最初に書いたコードが間違っていて（コンストラクタの名前、await が欠落）訂正しましたので注意してください。

[K-actus]

該当ページを開こうとするとSqlException: Invalid object name 'Plants'. のエラーが出ます。
LivingViewModelについてDbContextを修正する必要はありますか？

[退会済みユーザー]

それだけの情報では分からないですね。どこでどのようにするとそのエラーが出るかも書いてないし、ホントに情報不足です。回答者はスレッドに書いてあること以外は知り得ません。よく考えて質問してください。

[退会済みユーザー]

質問のコメント欄にも書きましたが、回答が必要なら、回答者が分からないと言っていることに対して追加情報を出して分かってもらえるようにする努力をしませんか。

LivingViewModelsController のコードを書き直したのであろうと想像してますが、そのコードを見せてもらわないと、あなたが何をしたかなんてのは分かりません。

これも質問のコメント欄に書きましたが、回答者の想像力に期待しないできちんと質問に書いてください。想像で答えて、想像が違っていると、混乱を招くばかりです。ここ Teratail には回答者の想像の斜め上のことをしている質問者さんが多いのです。

なお、質問者さんが書き直した LivingViewModelsController のコードを質問欄にアップする際は、元のコードを書き直すのではなく、追記する形で書いてください。このスレッドの Q&A の話につじつまが合わなくなって、あとで検索などでこのスレッドを訪れる人には訳が分からなくなりますので。

[K-actus]

返信が遅れて申し訳ありません。

LivingViewModelsControllerのコードは、
myView.Plants = await _context.Plant.ToListAsync();
の右辺をPlantからPlantsにした以外SurferOnWwwさんが載せてくれたコード通りに修正しました。
DbContextも質問文から修正していません。

[退会済みユーザー]

エラーは await _context.Animals.ToListAsync() で出るのですよね？

そうであればAnimalsController の Index メソッドの await _context.Animals.ToListAsync() でも同じエラーが出るはずなのですが？

でも、質問を読む限り AnimalsController の Index メソッドではエラーにならず、期待通り Animals のリストが表示されるように思えます。

とすると、質問に書いてないどこか別のところで何かの違うことをやっていて、それが原因だとしか思えないです。どこか違わないか探してみてください。

[K-actus]

すみません、解決しました。
エラーが出てたのはPlantsの方で、原因はテーブルが存在してないからでした。
ありがとうございました。

Answer 2

リレーションシップ を構築してください。

public class Living
{
    public int LivingId { get; set; }
    public List<Animal> Animals { get; set; }
    public List<Plant> Plants { get; set; }
}

public class Animal
{
    public int AnimalId { get; set; }
    public string Name { get; set; }

    public int LivingId { get; set; }
    public Living Living { get; set; }
}

public class Plant
{
    public int PlantId { get; set; }
    public string Name { get; set; }

    public int LivingId { get; set; }
    public Living Living { get; set; }
}

public class MyDbContext : DbContext
{
    public MyDbContext(DbContextOptions<MyDbContext> options) : base(options) { }
    public DbSet<Living> Livings { get; set; }
    public DbSet<Animal> Animals { get; set; }
    public DbSet<Plant> Plants { get; set; }
}

上記をコードファーストでマイグレーションしてください。

リレーションシップを構築し、関連データの読み込みを行うことによって、例えば AnimalController から特定の Animal エンティティを変更したとき、対象のエンティティはリレーションしている Living エンティティの Animals プロパティから参照できるので、変更後のエンティティを Living エンティティからも取得できるようになります。

具体的には上記エンティティに対してLINQ to EntitiesのIncludeメソッドを利用することでプロバイダーから関連のデータを一括で読み込めます。

Comments

[退会済みユーザー]

質問者さんのやりたいことは、AnimalsController コードの Index で表示される Animal のレコード一覧の Table と、もう一つ他にあると思われる PlantController の Index で表示される Plant のレコード一覧の Table を、参考にしている記事 https://qiita.com/KktkiY/items/f28528916e97310262e0 にあるように並べて表示したいということだそうですので、上記の回答はその目的を果たすための直接の回答になってないと思いますが、そこはちょっと置いといて・・・

> 例えば AnimalController から特定の Animal エンティティを変更したとき、対象のエンティティはリレーションしている Living エンティティの Animals プロパティから参照できるので、変更後のエンティティを Living エンティティからも取得できるようになります。

そこは具体的にどのようにするのでしょうか？

.NET Framework の Enitity Framework ですと、ナビゲーションプロパティで遅延読み込みが可能なので、Living クラスの Animals プロパティに virtaul を付与しておけばプロパティにアクセスしたときに DB から関連データを読み込むことができるのですが、回答のコードのプロパティには virtual がないです。

virtual を付与して遅延読み込みを有効にしても、今度は例えば View でループを回してナビゲーションプロパティにアクセスすると、N+1 問題が発生するという別の問題も考えなければならなそうです。

さらに、Core では遅延読み込みはデフォルトで無効なので（質問者さんのアプリは Core 2.2 だそうです）、Living の Animals プロパティが指す List<Animal> オブジェクトはあらかじめ初期化してその参照をプロパティに代入しておかなければならないのですが、そこのところはどうするのでしょう。

[BluOxy]

はい、複数TableをViewModel経由で並べて表示する方法ではなく、新規のTableを定義してそれを直接表示する方法になります。Livingクラスの Animals, PlantsをそれぞれRazor構文で表示することで、やりたいことは結果的に満たせると思っています。

> 具体的にどのようにするのでしょうか？

ご指摘の通り、関連データの取得方法を具体的に回答していませんでした。
いくつかあるので、方法が記載されているMicrosoftの記事を添付します。回答に書いたコードでは、Includeメソッドを通して取得できる認識です。（検証はしていません）
https://docs.microsoft.com/ja-jp/ef/core/querying/related-data/

※ソースコードについては、Microsoftの記事にあるBlob, Post クラスにvirtualが無かったので、まずはそれ等にならうよう愚直に実装しました。

[退会済みユーザー]

回答の説明では具体的にどのようにするのか少なくとも自分には分からなかったです。

> 回答に書いたコードでは、Includeメソッドを通して取得できる認識です。

そういう話であれば、上の回答に書いてあった、

> 例えば AnimalController から特定の Animal エンティティを変更したとき、対象のエンティティはリレーションしている Living エンティティの Animals プロパティから参照できるので、変更後のエンティティを Living エンティティからも取得できるようになります。

の説明の前に、Linq のメソッドではこれこれこういう形で Include 使った式で一括で SQL Server からデータを読み込んで Living オブジェクトを生成し・・・と言うことを書いておいていただけたらと思います。

> ソースコードについては、Microsoftの記事にあるBlob, Post クラスにvirtualが無かったので、まずはそれ等にならうよう愚直に実装しました。

Entity Framework Core では遅延読み込みはデフォルトで無効なので virtual を付与しても意味がないからでしょう。（参考にされた記事の「遅延読み込み」のリンク先では遅延読み込みを有効にして virtual を付与しています）

[BluOxy]

ありがとうございます。追記しました。