WPF的SelectedValue不是从视图模型属性

Question

更新我在我的主窗口下面的下拉框：

<ComboBox SelectedValue="{Binding LanguageId}" SelectedValuePath="Id" DisplayMemberPath="Name" ItemsSource="{Binding Languages}"/> 

在我的ViewModel

我有以下两个属性设置：

public List<Language> Languages 
{ 
    get 
    { 
    return new List<Language>() 
    { 
    new Language { Id = 0, Name = ml.ml_string(100, "Language1") }, 
    new Language { Id = 1, Name = ml.ml_string(101, "Language2") }, 
    new Language { Id = 2, Name = ml.ml_string(102, "Language3") }, 
    new Language { Id = 3, Name = ml.ml_string(103, "Language4") } 
    }; 
    } 
} 

public int LanguageId 
{ 
    get 
    { 
    return _languageId; 
    } 
    set 
    { 
    _languageId = value; 
    NotifyPropertyChanged("Languages"); 
    NotifyPropertyChanged(); 
    } 
} 

所以我想是要在组合框中选择一种语言后通知我的语言属性，但是当前当我这样做时，它根本不显示任何值（请参阅图片）： combobox error 我该如何解决此问题？

Answer 1

选定的项目必须被视为等于集合中的项目。但是，您始终返回Languages的新对象集合，因此当选择更改时，您将ComboBox.ItemsSource设置为不包含刚选择的对象的新集合。因此，ComboBox将所选项目值丢弃为无效。正如你发现的那样，使用SelectedValuePath不能解决这个问题。

你可以通过重写Equals(Language)上Language通过保持你的Languages围绕永久和重用他们解决这个问题，或者（更好）。

如果你需要翻译Name属性，你可以这样做with a value converter。

我建议不要覆盖Equals()的原因是，在C＃中，你习惯于假设比较引用类型的两个实例意味着Object.ReferenceEquals()，并且打破该假设会导致错误。我从经验讲。

由于我们不再使用Name属性，我们可以只改变Languages到语言ID的List<int> - 在这种情况下，我们可以把它只读的，从来就不愿去重新创建列表。我在这里完成了。

public class LanguageNameConverter : IMultiValueConverter 
{ 
    public object Convert(object[] values, Type targetType, object parameter, CultureInfo culture) 
    { 
     if (values[0] is int) 
     { 
      var itemLanguageId = (int)values[0]; 
      var displayLanguageId = (int)values[1]; 

      // Do stuff here to get the translated name of the item language 
      var itemLanguageNameTranslated = $"Name of language {itemLanguageId} in language {displayLanguageId}"; 

      return itemLanguageNameTranslated; 
     } 
     else return values[0]?.GetType(); 
    } 

    public object[] ConvertBack(object value, Type[] targetTypes, object parameter, CultureInfo culture) 
    { 
     throw new NotImplementedException(); 
    } 
} 

XAML：

<ComboBox 
    SelectedValue="{Binding LanguageId}" 
    ItemsSource="{Binding Languages}" 
    > 
    <ComboBox.ItemTemplate> 
     <DataTemplate> 
      <Label> 
       <Label.Content> 
        <MultiBinding 
         Converter="{StaticResource LanguageNameConverter}" 
         > 
         <Binding Path="." /> 
         <Binding 
          Path="DataContext.LanguageId" 
          RelativeSource="{RelativeSource AncestorType=ComboBox}" 
          /> 
        </MultiBinding> 
       </Label.Content> 
      </Label> 
     </DataTemplate> 
    </ComboBox.ItemTemplate> 
</ComboBox> 

如果您更愿意保持Language类Languages，刚恢复SelectedValuePath="Id"的组合框，并设置第一Binding的Path在MultiBinding来代替Id的.。

<ComboBox 
    SelectedValue="{Binding LanguageId}" 
    SelectedValuePath="Id" 
    ... 

...

<MultiBinding 
    Converter="{StaticResource LanguageNameConverter}" 
    > 
    <Binding Path="Id" /> 
    <Binding 
     Path="DataContext.LanguageId" 
     RelativeSource="{RelativeSource AncestorType=ComboBox}" 
     /> 
</MultiBinding> 

---

等号清除溶液

这里的Equals()解决方案;我们尝试了它，但是在选择更改之后，保​​留旧的SelectedItem并且旧的名称以先前选择的语言存在问题。这是因为Equals()覆盖的事情。所以我们使用了多值转换器，它不需要任何解决方法或有趣的业务。

只是做事情WPF希望的方式，没有人受伤。这是规则。

public class Language 
{ 
    public int Id { get; set; } 
    public String Name { get; set; } 

    public override bool Equals(object obj) 
    { 
     if (obj is Language) 
     { 
      return ((Language)obj).Id == Id; 
     } 
     return base.Equals(obj); 
    } 

    public override int GetHashCode() 
    { 
     return Id.GetHashCode(); 
    } 
} 

如果你选择了这个方案，你还需要一个递归后卫添加到您的LanuageId属性：

private int _languageId; 
public int LanguageId 
{ 
    get 
    { 
     return _languageId; 
    } 
    set 
    { 
     if (_languageId != value) 
     { 
      _languageId = value; 
      OnPropertyChanged("Languages"); 
      OnPropertyChanged(); 
     } 
    } 
} 

更换ComboBox.ItemsSource现在将导致实际SelectedItem改变到一个新的对象实例，其中更新SelectedValue，这将导致LanguageId再次被设置 - 它已经具有相同的值。如果你省略了递归守护，LanguageId的set块会再次提升PropertyChanged为Languages，这将会更新SelectedItem等等 - 并且在堆栈溢出之前我们已经进入比赛。