WPFで二つのクラスの内容を一つのDataContextで表示

Question

前提・実現したいこと

C#のWPFで二つのクラス（PrintTextとPrintFilename）の内容を一つのDataContextで両方とも表示したいです。具体的な動きとしては、実行直後にOKという文字を表示し、Dataボタンをクリックして選択したファイル名を表示したいです。

発生している問題・エラーメッセージ

二つのクラスを一つのViewModelにまとめてDataContextに設定しましたが、何も表示されません。

該当のソースコード

MainWindow.xaml

<Window.DataContext>
        <view:MainWindowViewModel/>
    </Window.DataContext>
    <Grid>
        <StackPanel Orientation="Vertical">
            <Button x:Name="Data_Button" Content="Data" Click="Data_Button_Click" Margin="5" Width="75"/>
            <TextBlock x:Name="TextBlock_Result" Text="{Binding Path=printText.Result}"/>
            <TextBlock x:Name="TextBlock_Filename" Text="{Binding filename.Filename}"/>
        </StackPanel>
    </Grid>

MainWindow.xaml.cs

public partial class MainWindow : Window
    {
        OpenFileDialog oFileDialog;

        public MainWindow()
        {
            InitializeComponent();

            var printText = new PrintText();
            printText.Result = "OK";

            var viewModel = new MainWindowViewModel();
            DataContext = viewModel;

            MessageBox.Show(printText.Result);
        }

        private void Data_Button_Click(object sender, RoutedEventArgs e)
        {
            oFileDialog = new OpenFileDialog
            {
                Filter = "Excel (*.xlsx)|*.xlsx"
            };

            if (oFileDialog.ShowDialog() == false)
            {
                return;
            }

            var filename = new PrintFilename();
            filename.Filename = oFileDialog.FileName;
            MessageBox.Show(filename.Filename);
        }
    }

Model.PrintFilename.cs

class PrintFilename
    {
        public string Filename { get; set; }
    }

Model.PrintText.cs

class PrintText
    {
        public string Result { get; set; }
    }

ViewModel.MainWindowViewModel.cs

 class MainWindowViewModel
    {
        public PrintText PrintText { get; private set; }
        public PrintFilename PrintFilename { get; private set; }

        public MainWindowViewModel()
        {
            PrintText = new PrintText();
            PrintFilename = new PrintFilename();
        }
    }

試したこと

MessageBoxで値がちゃんと入っていることは確認しました。

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

Visual Studio 2019 Community, C# 7.3
他にも必要な情報があれば補足します。

Answer 1

1点目

xamlの間違いを直します。

<TextBlock x:Name="TextBlock_Result" Text="{Binding Path=printText.Result}"/>
<TextBlock x:Name="TextBlock_Filename" Text="{Binding filename.Filename}"/>

↓

<TextBlock x:Name="TextBlock_Result" Text="{Binding PrintText.Result}"/>
<TextBlock x:Name="TextBlock_Filename" Text="{Binding PrintFilename.Filename}"/>

バインディングのエラーは出力ウィンドウに出るだけで、特に止まったりしないので気が付きにくいです。

System.Windows.Data Error: 40 : BindingExpression path error: 'printText' property not found on 'object' ''MainWindowViewModel' (HashCode=64554036)'. BindingExpression:Path=printText.Result; DataItem='MainWindowViewModel' (HashCode=64554036); target element is 'TextBlock' (Name='TextBlock_Result'); target property is 'Text' (type 'String')
System.Windows.Data Error: 40 : BindingExpression path error: 'filename' property not found on 'object' ''MainWindowViewModel' (HashCode=64554036)'. BindingExpression:Path=filename.Filename; DataItem='MainWindowViewModel' (HashCode=64554036); target element is 'TextBlock' (Name='TextBlock_Filename'); target property is 'Text' (type 'String')

のようなエラーが出ているはずです。

2点目

バインディングを使って表示したいのでしょうから、変更通知（INotifyPropertyChanged）の実装が必要になります（ViewModelBase・BindableBase・Observable等名前はいろいろだったりしますが、中身は大体同じで定型コードです）

MainWindowViewModelのPrintText・PrintFilenameを丸ごと入れ替えるような想定であればこんな感じになります。

using System.ComponentModel;
using System.Runtime.CompilerServices;
using System.Windows;
using Microsoft.Win32;

namespace Questions233114
{
    class PrintFilename
    {
        public string Filename { get; set; }
    }

    class PrintText
    {
        public string Result { get; set; }
    }

    class MainWindowViewModel : Observable
    {
        public PrintText PrintText { get => _PrintText; set => Set(ref _PrintText, value); }
        private PrintText _PrintText;
        public PrintFilename PrintFilename { get => _PrintFilename; set => Set(ref _PrintFilename, value); }
        private PrintFilename _PrintFilename;

        public MainWindowViewModel()
        {
            PrintText = new PrintText();
            PrintFilename = new PrintFilename();
        }
    }

    abstract class Observable : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged;
        protected bool Set<T>(ref T storage, T value, [CallerMemberName] string propertyName = null)
        {
            if(Equals(storage, value)) return false;

            storage = value;
            OnPropertyChanged(propertyName);
            return true;
        }
        protected void OnPropertyChanged([CallerMemberName] string propertyName = null)
            => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    public partial class MainWindow : Window
    {
        private readonly MainWindowViewModel vm;
        public MainWindow()
        {
            InitializeComponent();

            vm = new MainWindowViewModel();
            DataContext = vm;

            var printText = new PrintText { Result = "OK" };
            vm.PrintText = printText;

            MessageBox.Show(vm.PrintText.Result);
        }

        private void Data_Button_Click(object sender, RoutedEventArgs e)
        {
            var oFileDialog = new OpenFileDialog { Filter = "Excel (*.xlsx)|*.xlsx" };

            if(oFileDialog.ShowDialog() == false)
            {
                return;
            }

            var filename = new PrintFilename { Filename = oFileDialog.FileName };
            vm.PrintFilename = filename;

            MessageBox.Show(vm.PrintFilename.Filename);
        }
    }
}

Filename・Resultを直接変えるのであればこんな感じになります。

using System.ComponentModel;
using System.Runtime.CompilerServices;
using System.Windows;
using Microsoft.Win32;

namespace Questions233114
{
    class PrintFilename : Observable
    {
        public string Filename { get => _Filename; set => Set(ref _Filename, value); }
        private string _Filename;
    }

    class PrintText : Observable
    {
        public string Result { get => _Result; set => Set(ref _Result, value); }
        private string _Result;
    }

    class MainWindowViewModel
    {
        public PrintText PrintText { get; private set; }
        public PrintFilename PrintFilename { get; private set; }

        public MainWindowViewModel()
        {
            PrintText = new PrintText();
            PrintFilename = new PrintFilename();
        }
    }

    abstract class Observable : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged;
        protected bool Set<T>(ref T storage, T value, [CallerMemberName] string propertyName = null)
        {
            if(Equals(storage, value)) return false;

            storage = value;
            OnPropertyChanged(propertyName);
            return true;
        }
        protected void OnPropertyChanged([CallerMemberName] string propertyName = null)
            => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    public partial class MainWindow : Window
    {
        private readonly MainWindowViewModel vm;

        public MainWindow()
        {
            InitializeComponent();
            vm = new MainWindowViewModel();
            DataContext = vm;

            vm.PrintText.Result = "OK";

            MessageBox.Show(vm.PrintText.Result);
        }

        private void Data_Button_Click(object sender, RoutedEventArgs e)
        {
            var oFileDialog = new OpenFileDialog { Filter = "Excel (*.xlsx)|*.xlsx" };

            if(oFileDialog.ShowDialog() == false)
            {
                return;
            }

            vm.PrintFilename.Filename = oFileDialog.FileName;

            MessageBox.Show(vm.PrintFilename.Filename);
        }
    }
}

どちらのパターンもあり得ますが、どちらがいいかはこれだけだとちょっと判断できません。

Comments

[Q10]

ベストアンサーを差し上げます。
まず、XAMLのBinding Pathですが、インスタンス名じゃなくてクラス名を書くんですね。
次回から出力ウィンドウを確認します。

次にC#の方ですが、変更通知が必要なんですね。
正直、自分のコードはあと一歩で完成かと思っていましたが、まだまだ遠かったです。
①「PrintText・PrintFilenameを丸ごと入れ替え」と②「Filename・Resultを直接変える」で比較してみました。今回は変更するプロパティが一つしかなかったのでコーディングは②が楽ですが、他にも変更するプロパティがあって、それらすべてを変更するなら①なんでしょうね。TN8001さんのコードを参考にしていろいろ組んでみます。C#は奥が深いですね。ありがとうございました！

[TN8001]

> クラス名を書くんですね。
いえ インスタンス名（変数名）です 同名なので分かりにくかったですが
class MainWindowViewModel { public PrintText Text { get; set; } }
class PrintText { public string Result { get; set; } }
だったとしたら

{Binding Path=Text.Result}か {Binding Text.Result}です
DataContext（MainWindowViewModel）の Text（PrintText）の Result（string）

あとプロパティでないとバインディングできないのを忘れてよくハマりますｗ（{ get; set; }の書き忘れ）

①②の使い分けですが
最近はModelにINotifyPropertyChangedの実装を許して②の上で、状況によっては①②両方みたいなことが多いかもしれません（値の変更のコードを書いてみて楽なほうでいいと思いますｗ）

お約束？のコードが多く面倒に思うかもしれませんが、バインディングで表示が自動的に追従するのはそれを上回る便利さです
最低限のコードを作ってプロジェクトテンプレートにしておくといいと思います
https://docs.microsoft.com/ja-jp/visualstudio/ide/how-to-create-project-templates?view=vs-2019

ReactiveProperty https://blog.okazuki.jp/entry/2018/03/24/135318
を使うのもいいですね

[Q10]

なるほど、インスタンス名でしたか。元々私がそれらを同名にしていたのがいけなかったですね。
{ get; set; }も忘れないようにします。

INotifyPropertyChangedは定型文なのかもしれないですが、私にはスラスラと書ける自信が無いです。プロジェクトテンプレートは「毎回この文書いているな」と思い始めたら挑戦してみます。ReactivePropertyだと短く書けそうですね。

また近いうちに質問すると思います。
ありがとうございました！