web-dev-qa-db-ja.com

MVVM:ラジオボタンをビューモデルにバインドしますか?

編集:問題は.NET 4.0で修正されました。

IsCheckedボタンを使用して、ラジオボタンのグループをビューモデルにバインドしようとしました。他の投稿を確認した後、IsCheckedプロパティは単に機能しないようです。問題を再現する短いデモを作成しました。これを以下に示します。

ここに私の質問があります:MVVMを使用してラジオボタンをバインドする簡単で信頼できる方法はありますか?ありがとう。

追加情報:IsCheckedプロパティは、次の2つの理由で機能しません。

  1. ボタンが選択されると、グループ内の他のボタンのIsCheckedプロパティはfalseに設定されません。

  2. ボタンが選択されると、ボタンが最初に選択された後、独自のIsCheckedプロパティは設定されません。私は、最初のクリックでバインディングがWPFによって破壊されていると推測しています。

デモプロジェクト:以下は、問題を再現する簡単なデモのコードとマークアップです。 WPFプロジェクトを作成し、Window1.xamlのマークアップを次のものに置き換えます。

<Window x:Class="WpfApplication1.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" Loaded="Window_Loaded">
    <StackPanel>
        <RadioButton Content="Button A" IsChecked="{Binding Path=ButtonAIsChecked, Mode=TwoWay}" />
        <RadioButton Content="Button B" IsChecked="{Binding Path=ButtonBIsChecked, Mode=TwoWay}" />
    </StackPanel>
</Window>

Window1.xaml.csのコードを、ビューモデルを設定する次のコード(ハック)に置き換えます。

using System.Windows;

namespace WpfApplication1
{
    /// <summary>
    /// Interaction logic for Window1.xaml
    /// </summary>
    public partial class Window1 : Window
    {
        public Window1()
        {
            InitializeComponent();
        }

        private void Window_Loaded(object sender, RoutedEventArgs e)
        {
            this.DataContext = new Window1ViewModel();
        }
    }
}

次のコードをWindow1ViewModel.csとしてプロジェクトに追加します。

using System.Windows;

namespace WpfApplication1
{
    public class Window1ViewModel
    {
        private bool p_ButtonAIsChecked;

        /// <summary>
        /// Summary
        /// </summary>
        public bool ButtonAIsChecked
        {
            get { return p_ButtonAIsChecked; }
            set
            {
                p_ButtonAIsChecked = value;
                MessageBox.Show(string.Format("Button A is checked: {0}", value));
            }
        }

        private bool p_ButtonBIsChecked;

        /// <summary>
        /// Summary
        /// </summary>
        public bool ButtonBIsChecked
        {
            get { return p_ButtonBIsChecked; }
            set
            {
                p_ButtonBIsChecked = value;
                MessageBox.Show(string.Format("Button B is checked: {0}", value));
            }
        }

    }
}

問題を再現するには、アプリを実行してボタンAをクリックします。ボタンAのIsCheckedプロパティがtrueに設定されていることを示すメッセージボックスが表示されます。ボタンBを選択します。ボタンBのIsCheckedプロパティがtrueに設定されていることを示す別のメッセージボックスが表示されますが、ボタンAを示すメッセージボックスはありませんIsCheckedプロパティはfalseに設定されています-プロパティは変更されていません。

次に、ボタンAをもう一度クリックします。ウィンドウでボタンが選択されますが、メッセージボックスは表示されません。IsCheckedプロパティは変更されていません。最後に、ボタンBをもう一度クリックします-同じ結果です。ボタンが最初にクリックされた後、いずれかのボタンのIsCheckedプロパティはまったく更新されません。

60
David Veeneman

Jasonの提案から始めると、問題はリストからの単一の範囲選択になり、ListBoxに非常にうまく変換されます。その時点で、ListBoxコントロールにスタイルを適用して、RadioButtonリストとして表示するのは簡単です。

<ListBox ItemsSource="{Binding ...}" SelectedItem="{Binding ...}">
    <ListBox.ItemContainerStyle>
        <Style TargetType="{x:Type ListBoxItem}">
            <Setter Property="Template">
                <Setter.Value>
                    <ControlTemplate TargetType="{x:Type ListBoxItem}">
                        <RadioButton Content="{TemplateBinding Content}"
                                     IsChecked="{Binding RelativeSource={RelativeSource TemplatedParent}, Path=IsSelected}"/>
                    </ControlTemplate>
                </Setter.Value>
            </Setter>
        </Style>
    </ListBox.ItemContainerStyle>
</ListBox>
53
John Bowen

.NET 4のIsCheckedプロパティへのバインドを修正したようです。VS2008で壊れたプロジェクトはVS2010で動作します。

18
RandomEngy

この質問を今後調査する人のために、最終的に実装したソリューションを示します。これは、ジョンボウエンの答えに基づいて作成されたもので、この問題に対する最善の解決策として選択しました。

最初に、ラジオボタンをアイテムとして含む透明なリストボックスのスタイルを作成しました。次に、ボタンをリストボックスに配置するように作成しました。ボタンをアプリとしてデータとして読み込むのではなく、ボタンを固定するため、マークアップにハードコーディングしました。

ビューモデルでListButtonsという列挙型を使用してリストボックスのボタンを表し、各ボタンのTagプロパティを使用して、そのボタンに使用する列挙型値の文字列値を渡します。 。 ListBox.SelectedValuePathプロパティを使用すると、Tagプロパティを選択した値のソースとして指定でき、SelectedValueプロパティを使用してビューモデルにバインドします。文字列とその列挙値を変換するには値コンバーターが必要だと思っていましたが、WPFの組み込みコンバーターは問題なく変換を処理しました。

Window1.xamlの完全なマークアップは次のとおりです。

<Window x:Class="RadioButtonMvvmDemo.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">

    <!-- Resources -->
    <Window.Resources>
        <Style x:Key="RadioButtonList" TargetType="{x:Type ListBox}">
            <Setter Property="Background" Value="Transparent"/>
            <Setter Property="ItemContainerStyle">
                <Setter.Value>
                    <Style TargetType="{x:Type ListBoxItem}" >
                        <Setter Property="Margin" Value="5" />
                        <Setter Property="Template">
                            <Setter.Value>
                                <ControlTemplate TargetType="{x:Type ListBoxItem}">
                                    <Border BorderThickness="0" Background="Transparent">
                                        <RadioButton 
                                            Focusable="False"
                                            IsHitTestVisible="False"
                                            IsChecked="{TemplateBinding IsSelected}">
                                            <ContentPresenter />
                                        </RadioButton>
                                    </Border>
                                </ControlTemplate>
                            </Setter.Value>
                        </Setter>
                    </Style>
                </Setter.Value>
            </Setter>
            <Setter Property="Control.Template">
                <Setter.Value>
                    <ControlTemplate TargetType="{x:Type ListBox}">
                        <Border BorderThickness="0" Padding="0" BorderBrush="Transparent" Background="Transparent" Name="Bd" SnapsToDevicePixels="True">
                            <ItemsPresenter SnapsToDevicePixels="{TemplateBinding UIElement.SnapsToDevicePixels}" />
                        </Border>
                    </ControlTemplate>
                </Setter.Value>
            </Setter>
        </Style>
    </Window.Resources>

    <!-- Layout -->
    <Grid>
        <!-- Note that we use SelectedValue, instead of SelectedItem. This allows us 
        to specify the property to take the value from, using SelectedValuePath. -->

        <ListBox Style="{StaticResource RadioButtonList}" SelectedValuePath="Tag" SelectedValue="{Binding Path=SelectedButton}">
            <ListBoxItem Tag="ButtonA">Button A</ListBoxItem>
            <ListBoxItem Tag="ButtonB">Button B</ListBoxItem>
        </ListBox>
    </Grid>
</Window>

ビューモデルには、SelectedButtons列挙型を使用して選択されているボタンを表示するSelectedButtonという単一のプロパティがあります。プロパティは、ビューモデルに使用する基本クラスのイベントを呼び出し、PropertyChangedイベントを発生させます。

namespace RadioButtonMvvmDemo
{
    public enum ListButtons {ButtonA, ButtonB}

    public class Window1ViewModel : ViewModelBase
    {
        private ListButtons p_SelectedButton;

        public Window1ViewModel()
        {
            SelectedButton = ListButtons.ButtonB;
        }

        /// <summary>
        /// The button selected by the user.
        /// </summary>
        public ListButtons SelectedButton
        {
            get { return p_SelectedButton; }

            set
            {
                p_SelectedButton = value;
                base.RaisePropertyChangedEvent("SelectedButton");
            }
        }

    }
} 

私の本番アプリでは、SelectedButton setterはボタンが選択されたときに必要なアクションを実行するサービスクラスメソッドを呼び出します。

完全にするために、基本クラスを以下に示します。

using System.ComponentModel;

namespace RadioButtonMvvmDemo
{
    public abstract class ViewModelBase : INotifyPropertyChanged
    {
        #region INotifyPropertyChanged Members

        public event PropertyChangedEventHandler PropertyChanged;

        #endregion

        #region Protected Methods

        /// <summary>
        /// Raises the PropertyChanged event.
        /// </summary>
        /// <param name="propertyName">The name of the changed property.</param>
        protected void RaisePropertyChangedEvent(string propertyName)
        {
            if (PropertyChanged != null)
            {
                PropertyChangedEventArgs e = new PropertyChangedEventArgs(propertyName);
                PropertyChanged(this, e);
            }
        }

        #endregion
    }
}

お役に立てば幸いです!

10
David Veeneman

1つの解決策は、プロパティのセッターにあるラジオボタンのViewModelを更新することです。ボタンAがTrueに設定されている場合、ボタンBをfalseに設定します。

DataContextのオブジェクトにバインドするときのもう1つの重要な要素は、オブジェクトがINotifyPropertyChangedを実装する必要があることです。バインドされたプロパティが変更されると、イベントが発生し、変更されたプロパティの名前が含まれます。 (簡潔にするため、サンプルではヌルチェックを省略しています。)

public class ViewModel  : INotifyPropertyChanged
{
    public event PropertyChangedEventHandler PropertyChanged;

    protected bool _ButtonAChecked = true;
    public bool ButtonAChecked
    {
        get { return _ButtonAChecked; }
        set 
        { 
            _ButtonAChecked = value;
            PropertyChanged(this, new PropertyChangedEventArgs("ButtonAChecked"));
            if (value) ButtonBChecked = false;
        }
    }

    protected bool _ButtonBChecked;
    public bool ButtonBChecked
    {
        get { return _ButtonBChecked; }
        set 
        { 
            _ButtonBChecked = value; 
            PropertyChanged(this, new PropertyChangedEventArgs("ButtonBChecked"));
            if (value) ButtonAChecked = false;
        }
    }
}

編集:

問題は、最初にボタンBをクリックすると、IsChecked値が変更され、バインディングがフィードスルーされますが、ボタンAは、チェックされていない状態をButtonACheckedプロパティにフィードスルーしないことです。コードで手動で更新することにより、ButtonACheckedプロパティセッターは、次にボタンAがクリックされたときに呼び出されます。

3
Doug Ferguson

ここに別の方法があります

表示:

<StackPanel Margin="90,328,965,389" Orientation="Horizontal">
        <RadioButton Content="Mr" Command="{Binding TitleCommand, Mode=TwoWay}" CommandParameter="{Binding Content, RelativeSource={RelativeSource Mode=Self}, Mode=TwoWay}" GroupName="Title"/>
        <RadioButton Content="Mrs" Command="{Binding TitleCommand, Mode=TwoWay}" CommandParameter="{Binding Content, RelativeSource={RelativeSource Mode=Self}, Mode=TwoWay}" GroupName="Title"/>
        <RadioButton Content="Ms" Command="{Binding TitleCommand, Mode=TwoWay}" CommandParameter="{Binding Content, RelativeSource={RelativeSource Mode=Self}, Mode=TwoWay}" GroupName="Title"/>
        <RadioButton Content="Other" Command="{Binding TitleCommand, Mode=TwoWay}" CommandParameter="{Binding Content, RelativeSource={RelativeSource Mode=Self}}" GroupName="Title"/>
        <TextBlock Text="{Binding SelectedTitle, Mode=TwoWay}"/>
    </StackPanel>

ViewModel:

 private string selectedTitle;
    public string SelectedTitle
    {
        get { return selectedTitle; }
        set
        {
            SetProperty(ref selectedTitle, value);
        }
    }

    public RelayCommand TitleCommand
    {
        get
        {
            return new RelayCommand((p) =>
            {
                selectedTitle = (string)p;
            });
        }
    }
2
Enkosi

IsCheckedのバグについてはわかりません。ビューモデルに対して可能なリファクタリングの1つです。ビューには、一連のRadioButtonsで表される相互に排他的な状態がいくつかあり、一度に1つしか選択できません。ビューモデルには、可能な状態を表す1つのプロパティ(enumなど)があります:stateA、stateBなど

2
Jason Roberts

John Bowenの小さな拡張機能 answer :値がToString()を実装していない場合は機能しません。 RadioButtonのContentをTemplateBindingに設定する代わりに、次のようにContentPresenterをその中に入れるだけです。

<ListBox ItemsSource="{Binding ...}" SelectedItem="{Binding ...}">
    <ListBox.ItemContainerStyle>
        <Style TargetType="{x:Type ListBoxItem}">
            <Setter Property="Template">
                <Setter.Value>
                    <ControlTemplate TargetType="{x:Type ListBoxItem}">
                        <RadioButton IsChecked="{Binding RelativeSource={RelativeSource TemplatedParent}, Path=IsSelected}">
                            <ContentPresenter/>
                        </RadioButton>
                    </ControlTemplate>
                </Setter.Value>
            </Setter>
        </Style>
    </ListBox.ItemContainerStyle>
</ListBox>

この方法では、必要に応じてDisplayMemberPathまたはItemTemplateを追加で使用できます。 RadioButtonはアイテムを「ラップ」して、選択範囲を提供します。

2

これは古い質問であり、元の問題は.NET 4で解決されました。正直なところ、これは少し話題から外れています。

MVVMでRadioButtonsを使用したいほとんどの場合、enumの要素から選択することです。これには、boolプロパティのVM各ボタンのスペースと、それらを使用して全体enumプロパティは実際の選択を反映するため、非常に手間がかかりますので、再利用可能で実装が簡単で、ValueConvertersを必要としないソリューションを思いつきました。

ビューはほぼ同じですが、enumを所定の位置に配置したら、VM側で単一のプロパティ。

MainWindowVM

using System.ComponentModel;

namespace EnumSelectorTest
{
  public class MainWindowVM : INotifyPropertyChanged
  {
    public EnumSelectorVM Selector { get; set; }

    private string _colorName;
    public string ColorName
    {
      get { return _colorName; }
      set
      {
        if (_colorName == value) return;
        _colorName = value;
        RaisePropertyChanged("ColorName");
      }
    }

    public MainWindowVM()
    {
      Selector = new EnumSelectorVM
        (
          typeof(MyColors),
          MyColors.Red,
          false,
          val => ColorName = "The color is " + ((MyColors)val).ToString()
        );
    }

    public event PropertyChangedEventHandler PropertyChanged;
    protected virtual void RaisePropertyChanged(string propertyName)
    {
      PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }
  }
}

すべての作業を行うクラスは、DynamicObjectから継承します。外部から見ると、boolプロパティを作成しますenumXAMLからバインドできる「Is」、「IsRed」、「IsBlue」などの接頭辞が付いています。実際のenum値を保持するValueプロパティとともに。

public enum MyColors
{
  Red,
  Magenta,
  Green,
  Cyan,
  Blue,
  Yellow
}

EnumSelectorVM

using System;
using System.ComponentModel;
using System.Dynamic;
using System.Linq;

namespace EnumSelectorTest
{
  public class EnumSelectorVM : DynamicObject, INotifyPropertyChanged
  {
    //------------------------------------------------------------------------------------------------------------------------------------------
    #region Fields

    private readonly Action<object> _action;
    private readonly Type _enumType;
    private readonly string[] _enumNames;
    private readonly bool _notifyAll;

    #endregion Fields

    //------------------------------------------------------------------------------------------------------------------------------------------
    #region Properties

    private object _value;
    public object Value
    {
      get { return _value; }
      set
      {
        if (_value == value) return;
        _value = value;
        RaisePropertyChanged("Value");
        _action?.Invoke(_value);
      }
    }

    #endregion Properties

    //------------------------------------------------------------------------------------------------------------------------------------------
    #region Constructor

    public EnumSelectorVM(Type enumType, object initialValue, bool notifyAll = false, Action<object> action = null)
    {
      if (!enumType.IsEnum)
        throw new ArgumentException("enumType must be of Type: Enum");
      _enumType = enumType;
      _enumNames = enumType.GetEnumNames();
      _notifyAll = notifyAll;
      _action = action;

      //do last so notification fires and action is executed
      Value = initialValue;
    }

    #endregion Constructor

    //------------------------------------------------------------------------------------------------------------------------------------------
    #region Methods

    //---------------------------------------------------------------------
    #region Public Methods

    public override bool TryGetMember(GetMemberBinder binder, out object result)
    {
      string elementName;
      if (!TryGetEnumElemntName(binder.Name, out elementName))
      {
        result = null;
        return false;
      }
      try
      {
        result = Value.Equals(Enum.Parse(_enumType, elementName));
      }
      catch (Exception ex) when (ex is ArgumentNullException || ex is ArgumentException || ex is OverflowException)
      {
        result = null;
        return false;
      }
      return true;
    }

    public override bool TrySetMember(SetMemberBinder binder, object newValue)
    {
      if (!(newValue is bool))
        return false;
      string elementName;
      if (!TryGetEnumElemntName(binder.Name, out elementName))
        return false;
      try
      {
        if((bool) newValue)
          Value = Enum.Parse(_enumType, elementName);
      }
      catch (Exception ex) when (ex is ArgumentNullException || ex is ArgumentException || ex is OverflowException)
      {
        return false;
      }
      if (_notifyAll)
        foreach (var name in _enumNames)
          RaisePropertyChanged("Is" + name);
      else
        RaisePropertyChanged("Is" + elementName);
      return true;
    }

    #endregion Public Methods

    //---------------------------------------------------------------------
    #region Private Methods

    private bool TryGetEnumElemntName(string bindingName, out string elementName)
    {
      elementName = "";
      if (bindingName.IndexOf("Is", StringComparison.Ordinal) != 0)
        return false;
      var name = bindingName.Remove(0, 2); // remove first 2 chars "Is"
      if (!_enumNames.Contains(name))
        return false;
      elementName = name;
      return true;
    }

    #endregion Private Methods

    #endregion Methods

    //------------------------------------------------------------------------------------------------------------------------------------------
    #region Events

    public event PropertyChangedEventHandler PropertyChanged;
    protected virtual void RaisePropertyChanged(string propertyName)
    {
      PropertyChanged?.Invoke(this, new PropertyChangedEventArgs(propertyName));
    }

    #endregion Events
  }
}

変更に対応するには、上記のようにNotifyPropertyChangedイベントをサブスクライブするか、コンストラクターに匿名メソッドを渡すことができます。

そして最後にMainWindow.xaml

<Window x:Class="EnumSelectorTest.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"
    mc:Ignorable="d"
    Title="MainWindow" Height="350" Width="525">  
  <Grid>
    <StackPanel>
      <RadioButton IsChecked="{Binding Selector.IsRed}">Red</RadioButton>
      <RadioButton IsChecked="{Binding Selector.IsMagenta}">Magenta</RadioButton>
      <RadioButton IsChecked="{Binding Selector.IsBlue}">Blue</RadioButton>
      <RadioButton IsChecked="{Binding Selector.IsCyan}">Cyan</RadioButton>
      <RadioButton IsChecked="{Binding Selector.IsGreen}">Green</RadioButton>
      <RadioButton IsChecked="{Binding Selector.IsYellow}">Yellow</RadioButton>
      <TextBlock Text="{Binding ColorName}"/>
    </StackPanel>
  </Grid>
</Window>

他の誰かがこれが便利だと思うことを望んでいます。

1
Brown Bear

ラジオボタンのグループ名を追加する必要があります

   <StackPanel>
        <RadioButton Content="Button A" IsChecked="{Binding Path=ButtonAIsChecked, Mode=TwoWay}" GroupName="groupName" />
        <RadioButton Content="Button B" IsChecked="{Binding Path=ButtonBIsChecked, Mode=TwoWay}" GroupName="groupName" />
    </StackPanel>
1
ptsivakumar
<RadioButton  IsChecked="{Binding customer.isMaleFemale}">Male</RadioButton>
    <RadioButton IsChecked="{Binding customer.isMaleFemale,Converter=      {StaticResource GenderConvertor}}">Female</RadioButton>

以下はIValueConverterのコードです

public class GenderConvertor : IValueConverter
{
    public object Convert(object value, Type targetType, object parameter, System.Globalization.CultureInfo culture)
    {
        return !(bool)value;
    }

    public object ConvertBack(object value, Type targetType, object parameter, System.Globalization.CultureInfo culture)
    {
        return !(bool)value;
    }
}

これは私のために働いた。ラジオボタンのクリックに応じて、ビューとビューモデルの両方で値がバインドされました。 True->男性およびFalse->女性

0
jeevan

私はVS2015と.NET 4.5.1で非常に似た問題を抱えています

XAML:

                <ListView.ItemsPanel>
                    <ItemsPanelTemplate>
                        <UniformGrid Columns="6" Rows="1"/>
                    </ItemsPanelTemplate>
                </ListView.ItemsPanel>
                <ListView.ItemTemplate>
                    <DataTemplate >
                        <RadioButton  GroupName="callGroup" Style="{StaticResource itemListViewToggle}" Click="calls_ItemClick" Margin="1" IsChecked="{Binding Path=Selected,Mode=TwoWay}" Unchecked="callGroup_Checked"  Checked="callGroup_Checked">

....

このコードでわかるように、リストビューがあり、テンプレートのアイテムはグループ名に属するラジオボタンです。

プロパティSelectedをTrueに設定して新しいアイテムをコレクションに追加すると、チェックされた状態で表示され、残りのボタンはチェックされたままになります。

最初にcheckedbuttonを取得して手動でfalseに設定することで解決しますが、これはそれが行われるはずの方法ではありません。

コードビハインド:

`....
  lstInCallList.ItemsSource = ContactCallList
  AddHandler ContactCallList.CollectionChanged, AddressOf collectionInCall_change
.....
Public Sub collectionInCall_change(sender As Object, e As NotifyCollectionChangedEventArgs)
    'Whenever collection change we must test if there is no selection and autoselect first.   
    If e.Action = NotifyCollectionChangedAction.Add Then
        'The solution is this, but this shouldn't be necessary
        'Dim seleccionado As RadioButton = getCheckedRB(lstInCallList)
        'If seleccionado IsNot Nothing Then
        '    seleccionado.IsChecked = False
        'End If
        DirectCast(e.NewItems(0), PhoneCall).Selected = True
.....
End sub

`

0
Jose