Wpf コンボボックスで選択値をbindingしたい

Question

WPFで作成した画面へのBindingでのデータの入出力を勉強し始めました。

ComboBoxのBindingで分からないことがあります。

やりたいことは、コンボボックスの初期値をSelectedIndexを指定するのではなく、
Value値（DicValue）のBindingで設定したいのと、
選択した結果をTwowayで（DicValue）反映させたいです。

他のサイトで調べたところ、SelectedValueで初期値を設定できる記載があったのですが、
うまく初期値として設定されないのと、選択値変更時、
DicValueに変更後の"key"列が入ると思ったのですが設定されません。

<Window x:Class="WpfApp1.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:WpfApp1"
        mc:Ignorable="d"
        Title="MainWindow" Height="450" Width="800">
    <Grid>

        <StackPanel Height="40" Background="Aqua">
        <ComboBox x:Name="cmb1" ItemsSource="{Binding ComboDic,Mode=OneWay}"
                  SelectedValue="{Binding DicValue,Mode=TwoWay}"
                  SelectedValuePath="key"
                  DisplayMemberPath="Value"
                  SelectionChanged="cmb1_Changed"
                  />
         </StackPanel>

    </Grid>
</Window>

namespace WpfApp1
{
    public partial class MainWindow : Window
    {
        /* コンボボックスでバインドテスト*/
        public Dictionary<string, string> ComboDic { get; set; }
        public string DicValue { get; set; }


        public string ListSelectedItem { get; set; }

        public MainWindow()
        {
            /* DictionaryでBinding */
            ComboDic = new Dictionary<string, string>()
            {
                {"10","りんご"},
                {"20","みかん"},
                {"30","ぶどう"},
            };

            InitializeComponent();

            /* 初期値　みかん */
            DicValue = "20";

            DataContext = this;
        }

        private void cmb1_Changed(object sender, SelectionChangedEventArgs e)
        {
            var item = this.DicValue;
        }
    }
}

Answer 1

DicValueに変更後の"key"列が入ると思ったのですが設定されません。

「XAML バインド エラー」ウィンドウにエラーが出ています。

重大度レベル	データ コンテキスト	バインド パス	ターゲット	ターゲット型	説明	ファイル	行	プロジェクト
エラー	KeyValuePair`2	key	ComboBox.NoTarget	Object	型 KeyValuePair`2 のオブジェクトに key プロパティが見つかりません。	\mainwindow.xaml	13	Questions355574

「XAML バインド エラー」ウィンドウが見つからなければ、「出力」ウィンドウでもいいです。

System.Windows.Data Error: 40 : BindingExpression path error: 'key' property not found on 'object' ''KeyValuePair`2' (HashCode=54645477)'. BindingExpression:Path=key; DataItem='KeyValuePair`2' (HashCode=54645477); target element is 'ComboBox' (Name=''); target property is 'NoTarget' (type 'Object')

keyが見つからないと言っています。そのためDicValueとバインドできていません。
指定するのはKeyです。

選択した結果をTwowayで（DicValue）反映させたいです。

TwoWayの意味が初期値の反映とＵＩからの変更のみであれば、今のまま（自動プロパティ）でいいです。
コードからも変更したい場合は、PropertyChangedを発砲する必要があります。

<Window
  x:Class="Questions355574.MainWindow"
  xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
  xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
  Width="800"
  Height="450">
  <StackPanel>
    <ComboBox
      DisplayMemberPath="Value"
      ItemsSource="{Binding ComboDic}"
      SelectedValue="{Binding DicValue}"
      SelectedValuePath="Key"
      SelectionChanged="cmb1_Changed" />
    <Button Click="Button_Click" Content="みかん" />
  </StackPanel>
</Window>

using System.Collections.Generic;
using System.ComponentModel;
using System.Diagnostics;
using System.Runtime.CompilerServices;
using System.Windows;
using System.Windows.Controls;

namespace Questions355574
{
    public partial class MainWindow : Window, INotifyPropertyChanged
    {
        public Dictionary<string, string> ComboDic { get; }

        public string DicValue { get; set; }
        //public string DicValue { get => _DicValue; set => Set(ref _DicValue, value); }
        //private string _DicValue;

        public MainWindow()
        {
            InitializeComponent();

            ComboDic = new Dictionary<string, string>
            {
                { "10", "りんご" },
                { "20", "みかん" },
                { "30", "ぶどう" },
            };
            DicValue = "20";

            DataContext = this;
        }

        private void cmb1_Changed(object sender, SelectionChangedEventArgs e)
        {
            Debug.WriteLine(DicValue);
        }

        private void Button_Click(object sender, RoutedEventArgs e)
        {
            // DicValue { get; set; }のほうではこの変更が反映されない！
            // こういうことをしたい場合は、PropertyChangedを発砲する必要がある（コメントされているDicValueのほうを使う）
            DicValue = "20";
        }


        public event PropertyChangedEventHandler PropertyChanged;
        protected void Set<T>(ref T storage, T value, [CallerMemberName] string propertyName = null)
        {
            if (Equals(storage, value)) return;
            storage = value;
            OnPropertyChanged(propertyName);
        }
        protected void OnPropertyChanged(string propertyName) => PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
}

Comments

[kota190]

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

ご指摘いただいた箇所を修正することにより、意図した動作となりました。
また、発砲のケースについても提示いただいたコードにより動作を確認することができました。

１点質問があります。
初歩的な質問で申し訳ありません。
Bindingするデータは、publicで宣言する必要があるのでしょうか。
（private変数に変更するとうまく動作しませんでした。）

[TN8001]

> Bindingするデータは、publicで宣言する必要があるのでしょうか。

今回はMainWindow が MainWindow 内のプロパティにバインドしているためそう見えても不思議ではありません。

しかし通常は ViewModel とバインドします。
xamlでのバインド（ItemsSource="{Binding ComboDic}"）をコードで表現するとこうなります。
var binding = new Binding("ComboDic") { Source = this, };
BindingOperations.SetBinding(ComboBox, ComboBox.ItemsSourceProperty, binding);

Sourceに自身を入れようがViewModelを入れようがそこに差はありません。
このアプリ（Questions355574.exe）から見たら、外部のアセンブリ BindingOperations（PresentationFramework.dll）がバインドを処理します。

そのため「public プロパティしか探さない」という設計にしたのだと思います。

技術的にはinternalやprivateでもできるはず（リフレクションベースなので）ですが、できるようにする必然性も特に感じませんね^^;

[c# - Binding to an internal property? - Stack Overflow](https://stackoverflow.com/questions/19704149/binding-to-an-internal-property

[kota190]

ありがとうございます。
ViewModelについて、これから勉強したいと思います。