绑定到ICollectionView的ComboBox显示不正确的SelectedItem

时间:2011-11-02 05:11:19

标签: silverlight mvvm combobox icollectionview

我在Silverlight 4.0中遇到了一对组合框的问题。

意图是具有从同一列表读取两个不同的组合框,但如果在一个所选择的任何项目将不会在其他显示(作为底层属性不允许相同)。

E.g。 (这只是示例代码,但同样代表它的工作原理)

<ComboBox ItemsSource="{Binding BackgroundColors}"
          SelectedItem="{Binding SelectedBackgroundColor, Mode=TwoWay}" />

<ComboBox ItemsSource="{Binding ForegroundColors}"
          SelectedItem="{Binding SelectedForegroundColor, Mode=TwoWay}" />

为了允许这种动态过滤,我的ViewModel中有2个不同的ICollectionView,每个组合框ItemsSource都绑定到了ICollectionView。每个ObservableCollection<T>都有相同private ObservableCollection<Color> _masterColorList; public ICollectionView BackgroundColors { get; } public ICollectionView ForegroundColors { get; } 的来源,但过滤器设置为过滤掉其他所选项目。

ICollectionView

当一个的SelectedItem在UI改变时,视图模型属性被更新,并作为该部分,相反的.Refresh()通过刷新public Color SelectedForegroundColor { get { return _selectedForegroundColor; } set { if (_selectedForegroundColor == value) return; _selectedForegroundColor = value; BackgroundColors.Refresh(); RaisePropertyChanged(() => SelectedForegroundColor); } }

例如

{{1}}

这允许过滤器重新运行并更改可供选择的内容。

这很好用,但是有一个问题:

假设我们的主列表中有3种颜色:

  • 蓝色
  • 绿色
  • 红色

组合框1(CB1)已选择蓝色 组合框2(CB2)已选择绿色

因此,组合框具有这些列表(选中粗体)

CB1

  • 蓝色
  • 红色

CB2

  • 绿色
  • 红色

如果我然后选择红色在CB1,我期望的红色将被从取出CB2和替换它。这种情况正确发生,但显示的值从绿色变为蓝色

底层绑定值不会更改且ICollectionView.CurrentItem正确,但显示显示错误的值。

我认为正在发生的事情是,因为格林在列表的早期,它正在与所展示的内容混淆。如果要对ICollectionView进行排序,也会发生这种情况。

我已经尝试重新提升变更组合框的属性更改事件通知。选择的项目,但这似乎不起作用。

之前有没有人见过这个问题,或者我有什么想法可以修复它?

1 个答案:

答案 0 :(得分:0)

组合框的结合至少存在5个严重问题。

我认为你遇到了

http://connect.microsoft.com/VisualStudio/feedback/details/523394/silverlight-forum-combobox-selecteditem-binding

这一个。

更新itemssource后,绑定将停止工作。

我使用了其中一种解决方案,这段代码为我解决了这个问题:

public class ComboBoxEx : ComboBox
    {
        #region Fields

        private bool _suppressSelectionChangedUpdatesRebind = false;

        #endregion

        #region Properties

        public static readonly DependencyProperty SelectedValueProperProperty =
            DependencyProperty.Register(
                "SelectedValueProper",
                typeof(object),
                typeof(ComboBoxEx),
                new PropertyMetadata((o, dp) => {
                                          var comboBoxEx = o as ComboBoxEx;
                                          if (comboBoxEx == null)
                                              return;

                                          comboBoxEx.SetSelectedValueSuppressingChangeEventProcessing(dp.NewValue);
                                      }));

        [Browsable(true), EditorBrowsable(EditorBrowsableState.Always)]
        public object SelectedValueProper
        {
            get { return GetValue(SelectedValueProperProperty); }
            set { SetValue(SelectedValueProperProperty, value); }
        }

        #endregion

        #region Constructor and Overrides

        public ComboBoxEx()
        {
            SelectionChanged += ComboBoxEx_SelectionChanged;
        }

        /// <summary>
        /// Updates the current selected item when the <see cref="P:System.Windows.Controls.ItemsControl.Items"/> collection has changed.
        /// </summary>
        /// <param name="e">Contains data about changes in the items collection.</param>
        protected override void OnItemsChanged(System.Collections.Specialized.NotifyCollectionChangedEventArgs e)
        {
            // Must re-apply value here because the combobox has a bug that 
            // despite the fact that the binding still exists, it doesn't 
            // re-evaluate and subsequently drops the binding on the change event
            SetSelectedValueSuppressingChangeEventProcessing(SelectedValueProper);
        }

        #endregion

        #region Events

        private void ComboBoxEx_SelectionChanged(object sender, SelectionChangedEventArgs e)
        {
            // Avoid recursive stack overflow
            if (_suppressSelectionChangedUpdatesRebind)
                return;

            if (e.AddedItems != null && e.AddedItems.Count > 0) {
                //SelectedValueProper = GetMemberValue( e.AddedItems[0] );
                SelectedValueProper = SelectedValue; // This is faster than GetMemberValue
            }
            // Do not apply the value if no items are selected (ie. the else)
            // because that just passes on the null-value bug from the combobox
        }

        #endregion

        #region Helpers

        /// <summary>
        /// Gets the member value based on the Selected Value Path
        /// </summary>
        /// <param name="item">The item.</param>
        /// <returns></returns>
        private object GetMemberValue(object item)
        {
            return item.GetType().GetProperty(SelectedValuePath).GetValue(item, null);
        }

        /// <summary>
        /// Sets the selected value suppressing change event processing.
        /// </summary>
        /// <param name="newSelectedValue">The new selected value.</param>
        private void SetSelectedValueSuppressingChangeEventProcessing(object newSelectedValue)
        {
            try {
                _suppressSelectionChangedUpdatesRebind = true;
                SelectedValue = newSelectedValue;
            }
            finally {
                _suppressSelectionChangedUpdatesRebind = false;
            }
        }

        #endregion
    }

它不是我的代码,而是来自与此错误相关的文章。