使用MVVM绑定ComboBox的SelectedItem

Question

我在使用ComboBox时遇到了SelectedItem的问题。

<ComboBox Name="cbxSalesPeriods"
        ItemsSource="{Binding SalesPeriods}"
        DisplayMemberPath="displayPeriod"
        SelectedItem="{Binding SelectedSalesPeriod}"
        SelectedValuePath="displayPeriod"
        IsSynchronizedWithCurrentItem="True"/>

如果我打开下拉框，我可以看到值。

如果我选择项目，则所选项目不会显示。

有人有想法吗？

在我的ViewModel中，我有这两个属性：

public ObservableCollection<SalesPeriodVM> SalesPeriods { get; private set; }

private SalesPeriodVM selectedSalesPeriod;
public SalesPeriodVM SelectedSalesPeriod
{
    get { return selectedSalesPeriod; }

    set 
    {
        if (selectedSalesPeriod != value)
        {
            selectedSalesPeriod = value;
            RaisePropertyChanged("SelectedSalesPeriod");
        }
    }
}

以下是该类的一些属性：

public SalesPeriodVO Vo
{
    get { return period; }
}

public int Year
{
    get { return period.Year; }
    set
    {
        if (period.Year != value)
        {
            period.Year = value;
            RaisePropertyChanged("Year");
        }
    }
}

public int Month
{
    get { return period.Month; }
    set
    {
        if (period.Month != value)
        {
            period.Month = value;
            RaisePropertyChanged("Month");
        }
    }
}

public string displayPeriod { 
    get
    {
        return this.ToString();
    }
}

public override string ToString()
{
    return String.Format("{0:D2}.{1}", Month, Year);
}

---

编辑： 如果我移除DisplayMemberPath属性，会发生以下情况：

- René Winter

---

你应该创建一个沙盒应用程序。只留下一个组合框和你的视图模型，移除其他所有内容。然后尝试重现此问题。你可能无法做到。然后比较这两个应用程序，看看哪些差异可能导致这种行为。我怀疑问题不在你提供的代码中。还要注意你的组合框有一个名称。查看代码后台文件，看看是否有一些代码干扰了选择。 - Nikita B

Answer 1

看起来你在ComboBox上不必要地设置了属性。你可以删除DisplayMemberPath和SelectedValuePath这些有不同用途的属性。也许你可以查看此处的文章，了解这些属性的解释。请尝试以下方法:

<ComboBox Name="cbxSalesPeriods"
    ItemsSource="{Binding SalesPeriods}"
    SelectedItem="{Binding SelectedSalesPeriod}"
    IsSynchronizedWithCurrentItem="True"/>

此外，使用displayPeriod属性是无意义的，因为WPF框架会自动调用ToString方法来显示那些没有显式设置DataTemplate的对象。

---

更新>>>

由于我看不到您的所有代码，所以无法告诉您哪里出了问题。相反，我可以为您提供一个完整的工作示例，以展示如何实现您想要的功能。我从您的类中删除了无意义的displayPeriod属性和SalesPeriodVO属性，因为我对它一无所知...也许这就是你的问题的原因？ 试试这个：

public class SalesPeriodV
{
    private int month, year;

    public int Year
    {
        get { return year; }
        set
        {
            if (year != value)
            {
                year = value;
                NotifyPropertyChanged("Year");
            }
        }
    }

    public int Month
    {
        get { return month; }
        set
        {
            if (month != value)
            {
                month = value;
                NotifyPropertyChanged("Month");
            }
        }
    }

    public override string ToString()
    {
        return String.Format("{0:D2}.{1}", Month, Year);
    }

    public virtual event PropertyChangedEventHandler PropertyChanged;
    protected virtual void NotifyPropertyChanged(params string[] propertyNames)
    {
        if (PropertyChanged != null)
        {
            foreach (string propertyName in propertyNames) PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
            PropertyChanged(this, new PropertyChangedEventArgs("HasError"));
        }
    }
}

然后，我向视图模型中添加了两个属性：

private ObservableCollection<SalesPeriodV> salesPeriods = new ObservableCollection<SalesPeriodV>();
public ObservableCollection<SalesPeriodV> SalesPeriods
{
    get { return salesPeriods; }
    set { salesPeriods = value; NotifyPropertyChanged("SalesPeriods"); }
}
private SalesPeriodV selectedItem = new SalesPeriodV();
public SalesPeriodV SelectedItem
{
    get { return selectedItem; }
    set { selectedItem = value; NotifyPropertyChanged("SelectedItem"); }
}

然后使用您的值初始化集合：

SalesPeriods.Add(new SalesPeriodV() { Month = 3, Year = 2013 } );
SalesPeriods.Add(new SalesPeriodV() { Month = 4, Year = 2013 } );

然后仅将这两个属性绑定到ComboBox：

<ComboBox ItemsSource="{Binding SalesPeriods}" SelectedItem="{Binding SelectedItem}" />

就是这样... 你只需要这些就能得到一个完美的工作示例。你应该看到项目的显示来自于ToString方法，没有你的displayPeriod属性。希望你可以从这个代码示例中找到你的错误。

- Sheridan

---

这并没有回答问题。在给定的情境中，设置那些属性不应该导致所描述的行为。 - Nikita B

@NikitaBrizhak 我承认我的原始回答并没有为用户提供完整的解决方案。然而，在回答了许多这样的问题后，我意识到很少能够立即提供答案，原因有很多，比如缺乏信息或者提问者的理解。因此，我已经养成了分阶段回答这些问题的习惯，你刚刚看到的是其中的第一阶段。也许在将来，你可以给有声望的用户以怀疑的态度，并将那些评论留给新用户？ - Sheridan

@Sheridan，我只是指出，虽然你的建议从代码审查者的角度来看是有效的，但它们并没有试图解决手头的问题。这样做没有什么坏处，对吧？如果我的评论听起来有冒犯之嫌，我很抱歉，我并不是那个意思。 - Nikita B

没有问题，我完全接受了你的观点。也许我应该在回答中加上免责声明，说明这还不是正式的回答，但是我已经厌倦了一遍又一遍地打同样的东西。我只是想说，也许你应该相信这个网站上有声誉良好的成员会“做正确的事情”，但我并没有任何负面意思。 - Sheridan

@Sheridan，“也许将来您可以相信有声望的用户，而将那些评论留给新用户？”-> 很有趣 :-) - Matt

Answer 2

我曾遇到过类似的问题，当我在下拉框中选择某个选项时，SelectedItem绑定没有更新。我的问题在于我必须为绑定设置UpdateSourceTrigger=PropertyChanged。

<ComboBox ItemsSource="{Binding SalesPeriods}" 
          SelectedItem="{Binding SelectedItem, UpdateSourceTrigger=PropertyChanged}" />

- Sakarias Stenbäck

---

这个对我有用，赞！ - Digit Plays

Answer 3

<!-- xaml code-->
    <Grid>
        <ComboBox Name="cmbData"    SelectedItem="{Binding SelectedstudentInfo, Mode=OneWayToSource}" HorizontalAlignment="Left" Margin="225,150,0,0" VerticalAlignment="Top" Width="120" DisplayMemberPath="name" SelectedValuePath="id" SelectedIndex="0" />
        <Button VerticalAlignment="Center" Margin="0,0,150,0" Height="40" Width="70" Click="Button_Click">OK</Button>
    </Grid>



        //student Class
        public class Student
        {
            public  int Id { set; get; }
            public string name { set; get; }
        }

        //set 2 properties in MainWindow.xaml.cs Class
        public ObservableCollection<Student> studentInfo { set; get; }
        public Student SelectedstudentInfo { set; get; }

        //MainWindow.xaml.cs Constructor
        public MainWindow()
        {
            InitializeComponent();
            bindCombo();
            this.DataContext = this;
            cmbData.ItemsSource = studentInfo;

        }

        //method to bind cobobox or you can fetch data from database in MainWindow.xaml.cs
        public void bindCombo()
        {
            ObservableCollection<Student> studentList = new ObservableCollection<Student>();
            studentList.Add(new Student { Id=0 ,name="==Select=="});
            studentList.Add(new Student { Id = 1, name = "zoyeb" });
            studentList.Add(new Student { Id = 2, name = "siddiq" });
            studentList.Add(new Student { Id = 3, name = "James" });

              studentInfo=studentList;

        }

        //button click to get selected student MainWindow.xaml.cs
        private void Button_Click(object sender, RoutedEventArgs e)
        {
            Student student = SelectedstudentInfo;
            if(student.Id ==0)
            {
                MessageBox.Show("select name from dropdown");
            }
            else
            {
                MessageBox.Show("Name :"+student.name + "Id :"+student.Id);
            }
        }

- Zoyeb Shaikh