WPFComboBoxをカスタムリストにバインドする

2009年02月18日に質問されました。  ·  閲覧回数 583.5k回  ·  ソース

Geoff Bennett picture
2009年02月18日

SelectedItem / SelectedValueを更新していないように見えるComboBoxがあります。

ComboBox ItemsSourceは、一連のRAS電話帳エントリをCollectionViewとしてリストするViewModelクラスのプロパティにバインドされます。 次に、 SelectedItemまたはSelectedValue両方をViewModelの別のプロパティに(別々の時間に)バインドしました。 データバインディングによって設定された値をデバッグするためにsaveコマンドにMessageBoxを追加しましたが、 SelectedItem / SelectedValueバインディングが設定されていません。

ViewModelクラスは次のようになります。

public ConnectionViewModel
{
    private readonly CollectionView _phonebookEntries;
    private string _phonebookeEntry;

    public CollectionView PhonebookEntries
    {
        get { return _phonebookEntries; }
    }

    public string PhonebookEntry
    {
        get { return _phonebookEntry; }
        set
        {
            if (_phonebookEntry == value) return;
            _phonebookEntry = value;
            OnPropertyChanged("PhonebookEntry");
        }
    }
}

_phonebookEntriesコレクションは、コンストラクターでビジネスオブジェクトから初期化されています。 ComboBoxXAMLは次のようになります。

<ComboBox ItemsSource="{Binding Path=PhonebookEntries}"
    DisplayMemberPath="Name"
    SelectedValuePath="Name"
    SelectedValue="{Binding Path=PhonebookEntry}" />

ComboBoxに表示される実際の文字列値のみに関心があり、オブジェクトの他のプロパティには関心がありません。これは、VPN接続を確立するときにRASに渡す必要がある値であるため、 DisplayMemberPathSelectedValuePathは、どちらもConnectionViewModelのNameプロパティです。 コンボボックスは、であるDataTemplateに適用ItemsControlそのDataContextのビューモデルのインスタンスに設定されているウィンドウに。

ComboBoxはアイテムのリストを正しく表示し、UIで問題なく選択できます。 ただし、コマンドからメッセージボックスを表示すると、PhonebookEntryプロパティには、ComboBoxから選択された値ではなく、初期値がまだ含まれています。 他のTextBoxインスタンスは正常に更新され、MessageBoxに表示されています。

ComboBoxのデータバインディングで何が欠けていますか? 私はたくさんの検索をしましたが、間違っていることは何も見つからないようです。


これは私が見ている動作ですが、私の特定のコンテキストでは何らかの理由で機能していません。

CollectionViewのConnectionViewModelsを持つMainWindowViewModelがあります。 MainWindowView.xamlファイルのコードビハインドで、DataContextをMainWindowViewModelに設定しました。 MainWindowView.xamlには、ConnectionViewModelsのコレクションにバインドされたItemsControlがあります。 ComboBoxと他のいくつかのTextBoxを保持するDataTemplateがあります。 TextBoxは、 Text="{Binding Path=ConnectionName}"を使用してConnectionViewModelのプロパティに直接バインドされます。

public class ConnectionViewModel : ViewModelBase
{
    public string Name { get; set; }
    public string Password { get; set; }
}

public class MainWindowViewModel : ViewModelBase
{
    // List<ConnectionViewModel>...
    public CollectionView Connections { get; set; }
}

XAMLコードビハインド:

public partial class Window1
{
    public Window1()
    {
        InitializeComponent();
        DataContext = new MainWindowViewModel();
    }
}

次にXAML:

<DataTemplate x:Key="listTemplate">
    <Grid>
        <ComboBox ItemsSource="{Binding Path=PhonebookEntries}"
            DisplayMemberPath="Name"
            SelectedValuePath="Name"
            SelectedValue="{Binding Path=PhonebookEntry}" />
        <TextBox Text="{Binding Path=Password}" />
    </Grid>
</DataTemplate>

<ItemsControl ItemsSource="{Binding Path=Connections}"
    ItemTemplate="{StaticResource listTemplate}" />

TextBoxはすべて正しくバインドされ、データは問題なくそれらとViewModelの間を移動します。 動作していないのはComboBoxだけです。

PhonebookEntryクラスに関するあなたの仮定は正しいです。

私が行っている仮定は、DataTemplateによって使用されるDataContextがバインディング階層を介して自動的に設定されるため、 ItemsControl内の各アイテムに明示的に設定する必要がないことです。 それは私には少しばかげているように思えます。


上記の例に基づいて、問題を実証するテスト実装を次に示します。

XAML:

<Window x:Class="WpfApplication7.Window1"
    xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
    xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
    Title="Window1" Height="300" Width="300">
    <Window.Resources>
        <DataTemplate x:Key="itemTemplate">
            <StackPanel Orientation="Horizontal">
                <TextBox Text="{Binding Path=Name}" Width="50" />
                <ComboBox ItemsSource="{Binding Path=PhonebookEntries}"
                    DisplayMemberPath="Name"
                    SelectedValuePath="Name"
                    SelectedValue="{Binding Path=PhonebookEntry}"
                    Width="200"/>
            </StackPanel>
        </DataTemplate>
    </Window.Resources>
    <Grid>
        <ItemsControl ItemsSource="{Binding Path=Connections}"
            ItemTemplate="{StaticResource itemTemplate}" />
    </Grid>
</Window>

コードビハインド

namespace WpfApplication7
{
    /// <summary>
    /// Interaction logic for Window1.xaml
    /// </summary>
    public partial class Window1 : Window
    {
        public Window1()
        {
            InitializeComponent();
            DataContext = new MainWindowViewModel();
        }
    }

    public class PhoneBookEntry
    {
        public string Name { get; set; }
        public PhoneBookEntry(string name)
        {
            Name = name;
        }
    }

    public class ConnectionViewModel : INotifyPropertyChanged
    {

        private string _name;

        public ConnectionViewModel(string name)
        {
            _name = name;
            IList<PhoneBookEntry> list = new List<PhoneBookEntry>
                                             {
                                                 new PhoneBookEntry("test"),
                                                 new PhoneBookEntry("test2")
                                             };
            _phonebookEntries = new CollectionView(list);
        }
        private readonly CollectionView _phonebookEntries;
        private string _phonebookEntry;

        public CollectionView PhonebookEntries
        {
            get { return _phonebookEntries; }
        }

        public string PhonebookEntry
        {
            get { return _phonebookEntry; }
            set
            {
                if (_phonebookEntry == value) return;
                _phonebookEntry = value;
                OnPropertyChanged("PhonebookEntry");
            }
        }

        public string Name
        {
            get { return _name; }
            set
            {
                if (_name == value) return;
                _name = value;
                OnPropertyChanged("Name");
            }
        }
        private void OnPropertyChanged(string propertyName)
        {
            if (PropertyChanged != null)
                PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
        }
        public event PropertyChangedEventHandler PropertyChanged;
    }

    public class MainWindowViewModel
    {
        private readonly CollectionView _connections;

        public MainWindowViewModel()
        {
            IList<ConnectionViewModel> connections = new List<ConnectionViewModel>
                                                          {
                                                              new ConnectionViewModel("First"),
                                                              new ConnectionViewModel("Second"),
                                                              new ConnectionViewModel("Third")
                                                          };
            _connections = new CollectionView(connections);
        }

        public CollectionView Connections
        {
            get { return _connections; }
        }
    }
}

その例を実行すると、私が話している動作が得られます。 TextBoxは、編集するとバインディングを正常に更新しますが、ComboBoxは更新しません。 私がやったことは、親のViewModelを導入することだけなので、非常に混乱します。

私は現在、DataContextの子にバインドされたアイテムがその子をDataContextとして持っているという印象の下で働いています。 これを何らかの方法で解決するドキュメントが見つかりません。

つまり、

ウィンドウ-> DataContext = MainWindowViewModel
..アイテム-> DataContext.PhonebookEntriesにバインド
.... Item-> DataContext = PhonebookEntry(暗黙的に関連付けられています)

それが私の仮定をよりよく説明するかどうかはわかりません(?)。


私の仮定を確認するには、TextBoxのバインディングを次のように変更します。

<TextBox Text="{Binding Mode=OneWay}" Width="50" />

これにより、TextBoxバインディングルート(DataContextと比較しています)がConnectionViewModelインスタンスであることがわかります。

回答

Kjetil Watnedal picture
2009年02月18日
196

DisplayMemberPathとSelectedValuePathを "Name"に設定したので、パブリックプロパティNameを持つクラスPhoneBookEntryがあると想定します。

DataContextをConnectionViewModelオブジェクトに設定しましたか?

私はあなたのコードをコピーし、いくつかの小さな変更を加えました、そしてそれはうまくいくようです。 ビューモデルのPhoneBookEntyプロパティを設定すると、コンボボックスで選択したアイテムが変更されます。また、コンボボックスで選択したアイテムを変更すると、ビューモデルのPhoneBookEntryプロパティが正しく設定されます。

これが私のXAMLコンテンツです:

<Window x:Class="WpfApplication6.Window1"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        Title="Window1" Height="300" Width="300">
<Grid>
    <StackPanel>
        <Button Click="Button_Click">asdf</Button>
        <ComboBox ItemsSource="{Binding Path=PhonebookEntries}"
                  DisplayMemberPath="Name"
                  SelectedValuePath="Name"
                  SelectedValue="{Binding Path=PhonebookEntry}" />
    </StackPanel>
</Grid>
</Window>

そして、これが私のコードビハインドです:

namespace WpfApplication6
{

    /// <summary>
    /// Interaction logic for Window1.xaml
    /// </summary>
    public partial class Window1 : Window
    {
        public Window1()
        {
            InitializeComponent();
            ConnectionViewModel vm = new ConnectionViewModel();
            DataContext = vm;
        }

        private void Button_Click(object sender, RoutedEventArgs e)
        {
            ((ConnectionViewModel)DataContext).PhonebookEntry = "test";
        }
    }

    public class PhoneBookEntry
    {
        public string Name { get; set; }

        public PhoneBookEntry(string name)
        {
            Name = name;
        }

        public override string ToString()
        {
            return Name;
        }
    }

    public class ConnectionViewModel : INotifyPropertyChanged
    {
        public ConnectionViewModel()
        {
            IList<PhoneBookEntry> list = new List<PhoneBookEntry>();
            list.Add(new PhoneBookEntry("test"));
            list.Add(new PhoneBookEntry("test2"));
            _phonebookEntries = new CollectionView(list);
        }

        private readonly CollectionView _phonebookEntries;
        private string _phonebookEntry;

        public CollectionView PhonebookEntries
        {
            get { return _phonebookEntries; }
        }

        public string PhonebookEntry
        {
            get { return _phonebookEntry; }
            set
            {
                if (_phonebookEntry == value) return;
                _phonebookEntry = value;
                OnPropertyChanged("PhonebookEntry");
            }
        }

        private void OnPropertyChanged(string propertyName)
        {
            if (PropertyChanged != null)
                PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
        }
        public event PropertyChangedEventHandler PropertyChanged;
    }
}

編集: Geoffsの2番目の例は機能していないようですが、これは私には少し奇妙に思えます。 ConnectionViewModelのPhonebookEntriesプロパティをReadOnlyCollectionタイプ

たぶん、CollectionViewに問題がありますか? 出力コンソールに警告が表示されました。

System.Windows.Data警告:50:CollectionViewを直接使用することは完全にはサポートされていません。 基本的な機能は機能しますが、非効率的ですが、高度な機能では既知のバグが発生する可能性があります。 これらの問題を回避するには、派生クラスの使用を検討してください。

Edit2(.NET 4.5): DropDownListのコンテンツは、DisplayMemberPathではなくToString()に基づくことができますが、DisplayMemberPathは、選択および表示されたアイテムのメンバーのみを指定します。

Roy picture
2010年01月05日
77

データをComboBoxにバインドするには

List<ComboData> ListData = new List<ComboData>();
ListData.Add(new ComboData { Id = "1", Value = "One" });
ListData.Add(new ComboData { Id = "2", Value = "Two" });
ListData.Add(new ComboData { Id = "3", Value = "Three" });
ListData.Add(new ComboData { Id = "4", Value = "Four" });
ListData.Add(new ComboData { Id = "5", Value = "Five" });

cbotest.ItemsSource = ListData;
cbotest.DisplayMemberPath = "Value";
cbotest.SelectedValuePath = "Id";

cbotest.SelectedValue = "2";

ComboDataは次のようになります。

public class ComboData
{ 
  public int Id { get; set; } 
  public string Value { get; set; } 
}
CyberMonk picture
2009年10月06日
24

最初は同じ問題のように見えましたが、NHibernate / WPFの互換性の問題が原因であることが判明しました。 この問題は、WPFがオブジェクトの同等性をチェックする方法が原因で発生しました。 SelectedValueプロパティとSelectedValuePathプロパティのオブジェクトIDプロパティを使用して、自分のものを機能させることができました。

<ComboBox Name="CategoryList"
          DisplayMemberPath="CategoryName"
          SelectedItem="{Binding Path=CategoryParent}"
          SelectedValue="{Binding Path=CategoryParent.ID}"
          SelectedValuePath="ID">

詳細については、Chesterのブログ投稿、 WPF ComboBox-SelectedItem、SelectedValue、およびSelectedValuePath withNHibernateを参照してください。

phifi picture
2016年10月11日
1

SelectedItemが更新されないという同様の問題がありました。

私の問題は、選択したアイテムがリストに含まれているアイテムと同じインスタンスではないことでした。 そのため、MyCustomObjectのEquals()メソッドをオーバーライドし、これら2つのインスタンスのIDを比較して、ComboBoxに同じオブジェクトであることを通知するだけで済みました。

public override bool Equals(object obj)
{
    return this.Id == (obj as MyCustomObject).Id;
}