Сохранить изменения в связанном DataGridView с активным фильтром

Редко бывает хорошей идеей читать и записывать значения ячеек самостоятельно. Использование DataBinding обычно является более простым методом.

Кажется, вы уже используете BindingSource, но по какой-то причине вы читаете измененную строку из отображаемых данных, а не из исходного BindingSource.

Я не уверен, что разумно обновлять базу данных, как только оператор переходит к новой строке. Что, если он не знает значения для заполнения и хочет прокрутить вниз, чтобы проверить другую строку? Рассмотрите возможность изменения интерфейса таким образом, чтобы оператор мог указать, что он закончил редактирование всех данных. Например кнопка. Но это немного не ваш вопрос.

Если вы поместите свои данные в BindingList и назначите эти данные источнику данных вашего DataGridView, то каждое изменение данных в DataGridView будет автоматически обновляться в BindingList. Используйте событие BindingList.ListChanged, чтобы получить уведомление, или используйте вышеупомянутую кнопку перед обработкой измененных данных.

BindingList<Customer> DisplayedCustomers {get; } = new BindingList<Customer>();

public MyForm()
{
    this.InitializeComponents();

    this.DataGridView1.DataSource = this.DisplayedCustomers();
    this.BindingListView.ListChanged += BindingListChanged;
}

Отображается пустой DataGridView. Чтобы заполнить его Клиентами:

private InitializeDataGridView()
{
    IEnumerable<Customer> customers = this.QueryCustomers();
    foreach (var customer in Customers)
        this.Customers.Add(customer);
}

Это автоматически покажет всех клиентов в вашем DataGridView. Увы, вы получите событие один раз для каждого клиента. Если вы этого не хотите, попробуйте создать новый BindingList и назначить его источнику данных DataGridView. Не забудьте отписаться от старого BindingList и подписаться на событие нового BindingList.

private void BindingListChanged(object sender, ListChangedEventArgs e)
{
     BindingList<Customer> bindingList = (BindingList<Customer>)sender;
     // use e.ListChangedType to detect what has changed:
     // item added / removed / moved to a new location / value changed?

     switch (e.ListChangedType)
     {
         case ListChangedType.ItemChanged:
             var changedCustomer = bindingList[e.NewIndex];
             ProcessChangedCustomer(changeCustomer);
             break;
         case ...
    }
}

Преимущество использования BindingList вместо того, чтобы возиться со значениями ячеек DataGridView, заключается в том, что вы отделяете данные от того, как эти данные передаются оператору. Это упрощает повторное использование (показывать его в ComboBox вместо DataGridView? Продолжайте, изменения минимальны), упрощает модульное тестирование: BindingList также работает, если нет формы; и, следовательно, легче поддерживать и изменять.

И последний совет: чтобы упростить фильтрацию по свойствам (показывать только тех клиентов, которые живут в Нью-Йорке) и упростить сортировку, рассмотрите возможность использования пакета nuget Equin.ApplicationFramework.BindingListView

List<Customer> customers = GetCustomers();
BindingListView<Customer> view = new BindingListView<Customer>(customers);
dataGridView1.DataSource = view;

И Бинго: бесплатная сортировка щелчком мыши по заголовку столбца. Он даже запоминает, следует ли сортировать по возрастанию или по убыванию, и показывает символ сортировки (стрелка, указывающая направление сортировки).

Фильтрация:

void FilerCustomersByCity(string city)
{
    view.ApplyFilter(delegate(Customer) customer => customer.City == city);
}