Я пытаюсь использовать шаблон 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





Поскольку у вас уже есть элемент 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.
Кроме того, я немного рефакторил код. Очень важно иметь хорошее имя для переменных. И если выглядит немного неправильно, но у меня нет вашей модели.
Да, из-за использования foreach, извините за это. Второй вариант не сработал? Если вы используете его, представление будет обновляться только один раз. Вместо того, чтобы каждый раз использовать RemoveAt.
Если вы не хотите создавать новую исходную коллекцию, вы можете удалить элементы из существующей. Просто убедитесь, что вы не вызываете метод 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>. Просто убедитесь, что вы вызываете уведомление о свойстве, когда свойство установлено.
У меня работает нормально. Спасибо.
Ваш первый вариант был первым, который я попробовал сам, но он создает исключение. В любом случае, я благодарю вас, ответ mm8 отлично сработал для меня.