質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.76%

  • WPF

    656questions

    Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

  • .NET Framework

    425questions

    .NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

  • MVVM

    71questions

    MVVM(Model View ViewModel)は構築上のデザインパターンで、表現ロジック(ViewModel)によってデータ(Model)からページ(View)を分離させます。

WPFのMVVMでViewModelでModelを丸ごと公開するのは良いことか?

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 309

silatori

score 6

 分からないこと

現在WPFのアプリを作成するにあたって、以下のような実装を考えています。

Model.cs
class Model : BindableBase
{
    // 通知を実装したstringのプロパティ
    // 通知を実装したintのプロパティ
}
ViewModel.cs
class ViewModel
{
    public ViewModel()
    {
        this._model = new Model();
    }

    private Model _model;

    public Model Model
    {
        get{return this._model;}
    }
}


こうすることで、ViewからModel.[Modelのプロパティ名]とすることで同期することが出来ると思います。  

しかし、MVVMについて調べているとViewModelはModelのプロパティをひとつずつラップしていくような作りでないとMVVMとは言えないという旨の情報をあちこちで見かけます。それはなぜなのでしょうか?

具体的にはLivetの作者さんの資料を読みましたが(リンクは避けます)、私の理解力が足らず、このパターンが良くない理由が分かりませんでした。

ご教授願います。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

+1

シンプルに、それだと、VM と M の責務の分離ができていない
表示用に必要なものを追加できない
VMは、本当に、表示用。

C#のpublicが フィールがいいのか、プロパティがいいのか、という議論と同じように、
直接だと、モデルの変更は、どこから変更されたのかが、追うのが難しくなります。

モデルとそのまま表示するくらいの用途なら、わざわざやらなくてもいいことではありますが。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/24 00:19

    ご回答ありがとうございます。
    確かに、責務の分離はできていないなと感じました。

    具体的なVMの責務はVMはMを購読して値を受け取ってVに公開し、MでVMの値を使うときは引数の形で渡す。という認識で間違いないでしょうか。

    確かにこれならMの更新はMしか行わないのでわかりやすいと感じました。

    キャンセル

+1

ご提示されているコードでは

  • ViewModelA
  • ViewModelB
  • ModelA

とあった場合、ModelAの情報をViewModelAとViewModelBの両方が参照したい場合、ViewModelA/Bそれぞれの中でModelAのインスタンスが生成されてしまいませんか。すなわちViewModelAのModelAの情報と、ViewModelBのModelAの情報は異なってしまうように見受けられます。

プロパティをラップしないとMVVMとは言えないのソースはよくわかりませんでしたが、少なくともModelの情報を複数のViewModelで参照することが出来ないので、提示されたソースコードはよろしくないのではないでしょうか。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/24 00:24

    ご回答ありがとうございます。
    ご指摘いただいた内容は、今回のコードでは確かにその通りです。

    しかし、この問題はVMでMを生成せずにシングルトンにすることで解消するように思えます。

    こうなった場合でも、VMでそのMをVに向けて丸ごと公開してもよいか、というところがわかりませんでした。

    提示すべきコードが質問に適さないものであったために、うまく意図を伝えられませんでした。申し訳ございません。

    キャンセル

  • 2018/05/24 10:25

    なるほどです。
    丸ごと公開してよいのか?という話は、kiichi54321さんのおっしゃる通り責務の分割というところに帰結するでしょう。

    また実務的な話でいうと、大規模な開発の場合はModelを開発する人とViewやViewModelを開発する人は別の人々である事が多いです。その場合、Modelのインタフェースが定まっていないとViewとViewModelが開発できない事になってしまいます。

    あくまでViewはViewModelに依存し、その中だけで完結できるのが望ましいのではないかと思います。

    キャンセル

  • 2018/05/24 21:54

    確かに、このやり方だとViewのコントロールに合うようなModelを考えることになるので分担できませんね!
    データ駆動の考えにも反していたことに気づきませんでした。VMはMをモデル化したものという説明もいまなら理解できます。どうやらVM-Mの関係を深く考えすぎていました。本質はV-VMなのですね。

    キャンセル

+1

ViewModelはViewModelの責務としてModelのプロパティをそのまま公開するならそれはそれでOKだと思います。
その場合は、ViewModelが公開している公開名がたまたま Model.[Modelのプロパティ名] と同じなだけです。
公開名にドットが入ってるか否かの違いしかありません。
公開名自体は「共有すべき仕様」です。
もちろん ViewModel側はModelの変更に合わせて「共有すべき仕様」を更新する責務があります。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/05/24 21:43

    ご回答ありがとうございます。
    あくまでも"VMのModelプロパティ"にバインドしているから良いという考えでしょうか。

    やっていることは自動プロパティのようなもので、融通は利かないがかろうじてVMの責務から外れていないと考えると納得できるような気がしました。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.76%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • WPF

    656questions

    Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

  • .NET Framework

    425questions

    .NET Framework は、Microsoft Windowsのオペレーティングシステムのために開発されたソフトウェア開発環境/実行環境です。多くのプログラミング言語をサポートしています。

  • MVVM

    71questions

    MVVM(Model View ViewModel)は構築上のデザインパターンで、表現ロジック(ViewModel)によってデータ(Model)からページ(View)を分離させます。