Как удалить объекты из списка, привязанного к ObservableCollection в C# MVVM

Я пытаюсь использовать шаблон MVVM в С#. Поэтому у меня есть класс клиентов:

public class Customer
{
    public string CustumerNumber { get; set; }
    public string CustomerName { get; set; }
}

Я заполняю список клиентами из базы данных:

public class CustomerList
{
    public static List<Customer> customerlist = new List<Customer>();

    public static List<Customer> GetCustomer()
    {
    // Get data from database
    }
}

Моя модель просмотра:

class ViewModel : BaseViewModel
{
    public ObservableCollection<Customer> Customers { get; set; }
    public string CustomerSearch { get; set; }

    public ViewModel()
    {
         Customers = new ObservableColletion<Customers>(CustomerList.GetCustomer());
    }
}

Я связал клиентов в WPF-ListBox:

<ListBox ItemsSource = "{Binding Customers}"
DisplayMemberPath = "CustomerName"/>

Допустим, у меня есть 10 объектов CustomerName в ListBox. Есть TextBox, содержащий строку. Теперь я хочу удалить все объекты в ListBox, которые не содержат строку. Я решил проблему в ViewModel следующим образом:

public void SearchCustomer()
{
    foreach (Customer item in Customers)
    {
        if (item.Customers.ToUpper().Contains(CustomerSearch.ToUpper()) == false)
        {
            this.Customers = new ObservableCollection<Customer>(CustomerList.RemoveItemsFromView(item));
        }
    }
}

Это правильно? Мне это кажется неправильным, потому что каждый раз, когда цикл удаляет элемент, я создаю новый ObservableCollection вместо того, чтобы манипулировать существующим. Есть ли более профессиональный способ решить эту задачу?

Для PropertyChangeEvent я использую FodyWeaver

Стоит ли изучать PHP в 2026-2027 годах?
Стоит ли изучать PHP в 2026-2027 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
2
0
62
3
Перейти к ответу Данный вопрос помечен как решенный

Ответы 3

Поскольку у вас уже есть элемент Customer, вы можете просто сделать это внутри своего if:

this.Customers.Remove(item) 

Вы должны использовать метод Remove из Customers, как вы делаете с CustomerList. Или обновить Customers после цикла:

Первый вариант

foreach (Customer customer in Customers)
{
    if (!customer.Customers.ToUpper().Contains(CustomerSearch.ToUpper()))
    {
        CustomerList.RemoveItemsFromView(customer);
        Customers.Remove(customer);
    }
}

Второй вариант

foreach (Customer customer in Customers)
{
    if (!customer.Customers.ToUpper().Contains(CustomerSearch.ToUpper()))
    {
        CustomerList.RemoveItemsFromView(customer);
    }
}

this.Customers = new ObservableCollection<Customer>(CustomerList);

Я не могу предложить вам больше вариантов без наличия исходного кода CustomerList и RemoveItemsFromView.

Кроме того, я немного рефакторил код. Очень важно иметь хорошее имя для переменных. И если выглядит немного неправильно, но у меня нет вашей модели.

Ваш первый вариант был первым, который я попробовал сам, но он создает исключение. В любом случае, я благодарю вас, ответ mm8 отлично сработал для меня.

Mati 14.02.2019 17:25

Да, из-за использования foreach, извините за это. Второй вариант не сработал? Если вы используете его, представление будет обновляться только один раз. Вместо того, чтобы каждый раз использовать RemoveAt.

Oscar Vicente Perez 14.02.2019 17:53
Ответ принят как подходящий

Если вы не хотите создавать новую исходную коллекцию, вы можете удалить элементы из существующей. Просто убедитесь, что вы не вызываете метод Remove в цикле foreach.

Это должно работать:

for (int i = Customers.Count - 1; i >= 0; i--)
{
    Customer item = Customers[i];
    if (item.Customers.ToUpper().Contains(CustomerSearch.ToUpper()) == false)
    {
        Customers.RemoveAt(i);
    }
}

Если вы сбрасываете свойство коллекции каждый раз, когда хотите добавить или удалить элемент, вы также можете использовать List<T>. Просто убедитесь, что вы вызываете уведомление о свойстве, когда свойство установлено.

У меня работает нормально. Спасибо.

Mati 14.02.2019 17:29

Другие вопросы по теме