ComboBox ItemsSource changé => SelectedItem est ruiné

La norme ComboBox n'a pas que de la logique. Et comme vous l'avez mentionné SelectedItem devient null déjà après l'appel de Clear, de sorte que le ComboBox n'a aucune idée sur vous l'intention d'ajouter l'élément de même plus tard, et par conséquent, il ne fait rien pour le sélectionner. Cela étant dit, vous devrez mémoriser l'élément précédemment sélectionné manuellement et après que vous avez mis à jour, vous restauration de collection de la sélection manuellement. Habituellement, il est fait quelque chose comme ceci:

public void RefreshMyItems()
{
var previouslySelectedItem = SelectedItem;
MyItems.Clear();
foreach(var myItem in LoadItems()) MyItems.Add(myItem);
SelectedItem = MyItems.SingleOrDefault(i => i.Id == previouslySelectedItem.Id);
}

Si vous souhaitez appliquer le même comportement à tous les ComboBoxes (ou peut-être tous les Selector de contrôle), vous pouvez envisager de créer un Behavior(un attaché à la propriété ou mélange comportement). Ce comportement s'abonner à la SelectionChanged et CollectionChanged événements et les sauvegarder/restaurer l'élément sélectionné lorsque cela est approprié.

Malheureusement lors de la configuration de ItemsSource sur un Sélecteur d'objet il définit immédiatement SelectedValue ou SelectedItem null même si l'élément correspondant est à nouveau ItemsSource.

Peu importe si vous implémenter Equals.. fonctions ou si vous utilisez un implicitement type comparable pour votre SelectedValue.

Bien, vous pouvez enregistrer SelectedItem/Valeur avant le réglage de l'ItemsSource et de la restauration. Mais que faire si il y a une liaison sur SelectedItem/Valeur qui sera appelé à deux reprises:
la valeur null
restauration d'origine.

C'est une charge supplémentaire et même il peut causer certains un comportement indésirable.

Voici une solution que j'ai faite. Fonctionnera pour tout le Sélecteur d'objet. Juste SelectedValue de liaison avant le réglage de l'ItemsSource.

UPD: Ajout de try/finally pour protéger des exceptions dans les gestionnaires, a également ajouté null vérifier pour la liaison.

public static class ComboBoxItemsSourceDecorator
{
public static readonly DependencyProperty ItemsSourceProperty = DependencyProperty.RegisterAttached(
"ItemsSource", typeof(IEnumerable), typeof(ComboBoxItemsSourceDecorator), new PropertyMetadata(null, ItemsSourcePropertyChanged)
);
public static void SetItemsSource(UIElement element, IEnumerable value)
{
element.SetValue(ItemsSourceProperty, value);
}
public static IEnumerable GetItemsSource(UIElement element)
{
return (IEnumerable)element.GetValue(ItemsSourceProperty);
}
static void ItemsSourcePropertyChanged(DependencyObject element, 
DependencyPropertyChangedEventArgs e)
{
var target = element as Selector;
if (element == null)
return;
//Save original binding 
var originalBinding = BindingOperations.GetBinding(target, Selector.SelectedValueProperty);
BindingOperations.ClearBinding(target, Selector.SelectedValueProperty);
try
{
target.ItemsSource = e.NewValue as IEnumerable;
}
finally
{
if (originalBinding != null)
BindingOperations.SetBinding(target, Selector.SelectedValueProperty, originalBinding);
}
}
}

Voici un exemple de code XAML:

                <telerik:RadComboBox Grid.Column="1" x:Name="cmbDevCamera" DataContext="{Binding Settings}" SelectedValue="{Binding SelectedCaptureDevice}" 
SelectedValuePath="guid" e:ComboBoxItemsSourceDecorator.ItemsSource="{Binding CaptureDeviceList}" >
</telerik:RadComboBox>

De Test De L'Unité

Ici est une unité de cas de test prouvant qu'il fonctionne. Juste un commentaire le #define USE_DECORATOR de voir le test échoue lors de l'utilisation de la norme liaisons.

#define USE_DECORATOR
using System.Collections;
using System.Collections.Concurrent;
using System.Collections.Generic;
using System.Security.Permissions;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Controls.Primitives;
using System.Windows.Data;
using System.Windows.Threading;
using FluentAssertions;
using ReactiveUI;
using ReactiveUI.Ext;
using ReactiveUI.Fody.Helpers;
using Xunit;
namespace Weingartner.Controls.Spec
{
public class ComboxBoxItemsSourceDecoratorSpec
{
[WpfFact]
public async Task ControlSpec ()
{
var comboBox = new ComboBox();
try
{
var numbers1 = new[] {new {Number = 10, i = 0}, new {Number = 20, i = 1}, new {Number = 30, i = 2}};
var numbers2 = new[] {new {Number = 11, i = 3}, new {Number = 20, i = 4}, new {Number = 31, i = 5}};
var numbers3 = new[] {new {Number = 12, i = 6}, new {Number = 20, i = 7}, new {Number = 32, i = 8}};
comboBox.SelectedValuePath = "Number";
comboBox.DisplayMemberPath = "Number";
var binding = new Binding("Numbers");
binding.Mode = BindingMode.OneWay;
binding.UpdateSourceTrigger=UpdateSourceTrigger.PropertyChanged;
binding.ValidatesOnDataErrors = true;
#if USE_DECORATOR
BindingOperations.SetBinding(comboBox, ComboBoxItemsSourceDecorator.ItemsSourceProperty, binding );
#else
BindingOperations.SetBinding(comboBox, ItemsControl.ItemsSourceProperty, binding );
#endif
DoEvents();
var selectedValueBinding = new Binding("SelectedValue");
BindingOperations.SetBinding(comboBox, Selector.SelectedValueProperty, selectedValueBinding);
var viewModel = ViewModel.Create(numbers1, 20);
comboBox.DataContext = viewModel;
//Check the values after the data context is initially set
comboBox.SelectedIndex.Should().Be(1);
comboBox.SelectedItem.Should().BeSameAs(numbers1[1]);
viewModel.SelectedValue.Should().Be(20);
//Change the list of of numbers and check the values
viewModel.Numbers = numbers2;
DoEvents();
comboBox.SelectedIndex.Should().Be(1);
comboBox.SelectedItem.Should().BeSameAs(numbers2[1]);
viewModel.SelectedValue.Should().Be(20);
//Set the list of numbers to null and verify that SelectedValue is preserved
viewModel.Numbers = null;
DoEvents();
comboBox.SelectedIndex.Should().Be(-1);
comboBox.SelectedValue.Should().Be(20); //Notice that we have preserved the SelectedValue
viewModel.SelectedValue.Should().Be(20);
//Set the list of numbers again after being set to null and see that
//SelectedItem is now correctly mapped to what SelectedValue was.
viewModel.Numbers = numbers3;
DoEvents();
comboBox.SelectedIndex.Should().Be(1);
comboBox.SelectedItem.Should().BeSameAs(numbers3[1]);
viewModel.SelectedValue.Should().Be(20);
}
finally
{
Dispatcher.CurrentDispatcher.InvokeShutdown();
}
}
public class ViewModel<T> : ReactiveObject
{
[Reactive] public int SelectedValue { get; set;}
[Reactive] public IList<T> Numbers { get; set; }
public ViewModel(IList<T> numbers, int selectedValue)
{
Numbers = numbers;
SelectedValue = selectedValue;
}
}
public static class ViewModel
{
public static ViewModel<T> Create<T>(IList<T> numbers, int selectedValue)=>new ViewModel<T>(numbers, selectedValue);
}
///<summary>
///From http://stackoverflow.com/a/23823256/158285
///</summary>
public static class ComboBoxItemsSourceDecorator
{
private static ConcurrentDictionary<DependencyObject, Binding> _Cache = new ConcurrentDictionary<DependencyObject, Binding>();
public static readonly DependencyProperty ItemsSourceProperty = DependencyProperty.RegisterAttached(
"ItemsSource", typeof(IEnumerable), typeof(ComboBoxItemsSourceDecorator), new PropertyMetadata(null, ItemsSourcePropertyChanged)
);
public static void SetItemsSource(UIElement element, IEnumerable value)
{
element.SetValue(ItemsSourceProperty, value);
}
public static IEnumerable GetItemsSource(UIElement element)
{
return (IEnumerable)element.GetValue(ItemsSourceProperty);
}
static void ItemsSourcePropertyChanged(DependencyObject element,
DependencyPropertyChangedEventArgs e)
{
var target = element as Selector;
if (target == null)
return;
//Save original binding 
var originalBinding = BindingOperations.GetBinding(target, Selector.SelectedValueProperty);
BindingOperations.ClearBinding(target, Selector.SelectedValueProperty);
try
{
target.ItemsSource = e.NewValue as IEnumerable;
}
finally
{
if (originalBinding != null )
BindingOperations.SetBinding(target, Selector.SelectedValueProperty, originalBinding);
}
}
}
[SecurityPermission(SecurityAction.Demand, Flags = SecurityPermissionFlag.UnmanagedCode)]
public static void DoEvents()
{
DispatcherFrame frame = new DispatcherFrame();
Dispatcher.CurrentDispatcher.BeginInvoke(DispatcherPriority.Background, new DispatcherOperationCallback(ExitFrame), frame);
Dispatcher.PushFrame(frame);
}
private static object ExitFrame(object frame)
{
((DispatcherFrame)frame).Continue = false;
return null;
}
}
}

La véritable solution à ce problème est de ne pas enlever les éléments qui sont dans la nouvelle liste. C'est à dire. Ne pas effacer toute la liste, il suffit de supprimer ceux qui ne sont pas dans la nouvelle liste, puis ajouter ceux que la nouvelle liste a qui n'étaient pas dans l'ancienne liste.

Exemple.

Actuel De Zone De Liste Déroulante D'Éléments
Pomme, Orange, Banane

De Nouveaux Éléments De La Zone De Liste Déroulante
Pomme, Orange, Poire

Pour Remplir les nouveaux éléments
Supprimer la Banane et Ajouter la Poire

Maintenant le combo arc est toujours valable pour les éléments que vous pourriez avoir sélectionné et les éléments sont maintenant effacés s'ils ont été sélectionnés.

Après avoir perdu la moitié de ma tête et les poils et de casser mon clavier plusieurs fois,
je pense que pour le contrôle zone de liste déroulante, il est préférable de ne pas écrire le selectedItem,Selectedindex et ItemsSource de liaison de l'expression dans le code XAML comme nous ne peut pas vérifier si l'ItemsSource a changé, lors de l'utilisation de propriété ItemsSource de cours.

Dans la fenêtre ou de contrôle de l'utilisateur du constructeur j'ai défini la propriété ItemsSource de la zone de liste déroulante, puis dans le chargé de gestionnaire d'événement de la fenêtre ou de contrôle de l'utilisateur, j'ai défini l'expression de liaison et il fonctionne parfaitement. Si je mettrais ItemsSource expression de liaison dans le XAML, sans le "selectedItem", je ne trouve pas tout gestionnaire d'événements pour définir la SelectedItem expression de liaison, tout en empêchant la zone de liste déroulante source de mise à jour avec une référence null (selectedIndex = -1).