Question

This really seems like a bug to me, but perhaps some databinding gurus can enlighten me? (My WinForms databinding knowledge is quite limited.)

I have a ComboBox bound to a sorted DataView. When the properties of the items in the DataView change such that items are resorted, the SelectedItem in my ComboBox does not keep in-sync. It seems to point to someplace completely random. Is this a bug, or am I missing something in my databinding?

Here is a sample application that reproduces the problem. All you need is a Button and a ComboBox:

public partial class Form1 : Form
{
    private DataTable myData;

    public Form1()
    {
        this.InitializeComponent();

        this.myData = new DataTable();
        this.myData.Columns.Add("ID", typeof(int));
        this.myData.Columns.Add("Name", typeof(string));
        this.myData.Columns.Add("LastModified", typeof(DateTime));
        this.myData.Rows.Add(1, "first", DateTime.Now.AddMinutes(-2));
        this.myData.Rows.Add(2, "second", DateTime.Now.AddMinutes(-1));
        this.myData.Rows.Add(3, "third", DateTime.Now);

        this.myData.DefaultView.Sort = "LastModified DESC";
        this.comboBox1.DataSource = this.myData.DefaultView;
        this.comboBox1.ValueMember = "ID"; 
        this.comboBox1.DisplayMember = "Name";
    }

    private void saveStuffButton_Click(object sender, EventArgs e)
    {
        DataRowView preUpdateSelectedItem = (DataRowView)this.comboBox1.SelectedItem;
        // OUTPUT: SelectedIndex = 0; SelectedItem.Name = third
        Debug.WriteLine(string.Format("SelectedIndex = {0:N0}; SelectedItem.Name = {1}", this.comboBox1.SelectedIndex, preUpdateSelectedItem["Name"]));

        this.myData.Rows[0]["LastModified"] = DateTime.Now;

        DataRowView postUpdateSelectedItem = (DataRowView)this.comboBox1.SelectedItem;
        // OUTPUT: SelectedIndex = 2; SelectedItem.Name = second
        Debug.WriteLine(string.Format("SelectedIndex = {0:N0}; SelectedItem.Name = {1}", this.comboBox1.SelectedIndex, postUpdateSelectedItem["Name"]));

        // FAIL!
        Debug.Assert(object.ReferenceEquals(preUpdateSelectedItem, postUpdateSelectedItem));
    }
}

To clarify:

• I understand how I would fix the simple application above–I only included that to demonstrate the problem. My concern is how to fix it when the updates to the underlying data rows could be happening anywhere (on another form, perhaps.)
• I would really like to still receive updates, inserts, deletes, etc. to my data source. I have tried just binding to an array of DataRows severed from the DataTable, but this causes additional headaches.

Answer 1

The only promising solution I see at this time is to bind the combo box to a detached data source and then update it every time the “real” DataView changes. Here is what I have so far. Seems to be working, but (1) it’s a total hack, and (2) it will not scale well at all.

In form declaration:

private DataView shadowView;

In form initialization:

this.comboBox1.DisplayMember = "Value";
this.comboBox1.ValueMember = "Key";
this.shadowView = new DataView(GlobalData.TheGlobalTable, null, "LastModified DESC", DataViewRowState.CurrentRows);
this.shadowView.ListChanged += new ListChangedEventHandler(shadowView_ListChanged);
this.ResetComboBoxDataSource(null);

And then the hack:

private void shadowView_ListChanged(object sender, ListChangedEventArgs e)
{
    this.ResetComboBoxDataSource((int)this.comboBox1.SelectedValue);
}

private void ResetComboBoxDataSource(int? selectedId)
{
    int selectedIndex = 0;
    var detached = new KeyValuePair<int, string>[this.shadowView.Count];
    for (int i = 0; i < this.shadowView.Count; i++)
    {
        int id = (int)this.shadowView[i]["ID"];
        detached[i] = new KeyValuePair<int, string>(id, (string)this.shadowView[i]["Name"]);
        if (id == selectedId)
        {
            selectedIndex = i;
        }
    }
    this.comboBox1.DataSource = detached;
    this.comboBox1.SelectedIndex = selectedIndex;
}

Must detach event handler in Dispose:

this.shadowView.ListChanged -= new ListChangedEventHandler(shadowView_ListChanged);