Question

I am trying to bind a combobox in WPF like this,

<ComboBox Width="350" Margin="5" IsEditable="True" ItemsSource="{Binding ComboboxItems}" DisplayMemberPath="Name">

public List<ExpandoObject> ComboboxItems
{
    get
    {
        return comboboxItems;
    }
}

If I set the list like this in my view model,

comboboxItems.Clear();

foreach (ExpandoObject comboboxItem in repository.LoadComboboxItems())
{
    comboboxItems.Add(comboboxItem);
}

NotifyPropertyChanged(this, x => x.ComboboxItems);

The NotifyPropertyChanged seems to work because a breakpoint on the ComboboxItems is hit, but then the combobox list does not update on the GUI. Snoop shows no binding errors or anything like that.

The first time the above list is updated it seems to work, so it can’t be anything to do with using an ExpandoObject I don’t think.

UPDATE:

Using an observable collection works, but I would like to know if I have a setter in a viewmodel like this which binds to a control on the GUI,

public string Database
{
    get
    {
        return importData.Database;
    }
    set
    {
        importData.Database = value;
        NotifyPropertyChanged(this, x => x.Database);

        comboboxItems.Clear();

        foreach (ExpandoObject comboboxItem in repository.LoadComboboxItems())
        {
            comboboxItems.Add(comboboxItem);
        }

        NotifyPropertyChanged(this, x => x.ComboboxItems);
   }
}

Is that setter being run on a background thread? The reason I ask is will the setter block the GUI if it takes a while to load the items from the database?

This is where I went wrong the first time trying to use an ObservableCollection, by running the code in the setter on a background thread using BackgroundWorker. Updating the ObservableCollection caused an exception under those conditions.

Answer 1

I think it will work if you use an ObservableCollection<> instead of a List<>. Unless you use an ObservableCollection, xaml will not know that the contents of the list changed.

To answer the second part of your question, if you’re trying to set the ObservableCollection using a BackgroundWorker directly, you will get an exception. One of the ways to get around the exception is to set the ObservableCollection using BeginInvoke

One thing to note: you don’t need to call NotifyPropertyChanged(this, x => x.ComboboxItems); in your setter. This is because the property isn’t changing; the property is a collection and the collection contents are changing. ObservableCollection will notify subscribers that the contents have changed.