war 发布后页面不更新_解决 WPF 绑定集合后数据变动界面却不更新的问题（使用 ObservableCollection）...

解决 WPF 绑定集合后数据变动界面却不更新的问题

独立观察员 2020 年 9 月 9 日

在 .NET Core 3.1 的 WPF 程序中打算用 ListBox 绑定显示一个集合(满足需求即可，无所谓什么类型的集合)，以下是 Xaml 代码(瞟一眼就行，不是本文讨论重点)：

<ListBox ItemsSource="{Binding SipRegistrations, Mode=OneWay}" SelectedValue="{Binding SelectedAccountBinding, Mode=OneWayToSource}">    <ListBox.ItemTemplate>        <DataTemplate>            <TextBlock Text="{Binding SIPAccount.SIPUsername}">TextBlock>        DataTemplate>    ListBox.ItemTemplate>ListBox>

ViewModel 中有一个目标集合，当前是一个 List。

属性变动通知有两种实现方式，一是使用 PropertyChanged.Fody，二是使用自定义绑定基类 BindableBase，如下图。

下面主要谈论数据变动(集合增加内容)后，前台的界面却没有更新的问题。具体来说就是，List.Add 之后，第一次有效果，但后面就没效果了，界面始终只显示一条数据。

原始(无效果)：

SipRegistrations.RemoveAll(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername); // 移除重复项(如果有的话)SipRegistrations.Add(binding); // 添加新项

猜想是因为 List 的引用并没有变化，所以被认为该属性没有改变，进而也就没有变动通知。

其实这种需要变动通知的情况，推荐使用的是 ObservableCollection：

但是本人之前使用 ObservableCollection 没有成功过，反而是使用 List 是可以的，所以还是先看看用 List 怎么解决吧。

变体一(调试时有几率有效果)：

// 添加联系人到集合并处理界面绑定；SipRegistrations.RemoveAll(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername);List tempList = SipRegistrations;    // 临时集合；SipRegistrations = new List();       // 目标集合先置为空；tempList.Add(binding);                                  // 临时集合添加新项；SipRegistrations = tempList;                            // 临时集合赋值给目标集合；

变体一通过临时变量做中转，强制让目标集合(的引用)发生改变，但结果是只在调试时以很小的概率成功过。

由于这部分代码是在异步逻辑里，所以有可能是在多线程环境，而 List 不是线程安全的，所以有了以下加锁版本的变体二。

变体二(无效果，应该是和变体一类似)：

#region 成员/// /// 加锁对象/// private object _lockObj = new object();#endregion// 加锁；lock (_lockObj){    // 添加联系人到集合并处理界面绑定；    SipRegistrations.RemoveAll(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername);    List tempList = SipRegistrations;    SipRegistrations = new List();    tempList.Add(binding);    SipRegistrations = tempList;}

加了锁还是不行(不过锁还是需要的)，又想到，既然调试的时候有几率成功，那么是不是和代码运行速度有关呢？于是在目标集合置空和重新赋值之间加了个线程休眠，竟然真的可以，也就是以下的变体三。

变体三(有效果)：

lock (_lockObj){    // 添加联系人到集合并处理界面绑定；    SipRegistrations.RemoveAll(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername);    List tempList = SipRegistrations;    SipRegistrations = new List();        Thread.Sleep(500); // 关键代码；        tempList.Add(binding);    SipRegistrations = tempList;}

好了，以上就是解决方法了。

接下来再尝试一下 ObservableCollection 吧：

#region 成员/// /// 加锁对象/// private object _lockObj = new object();public ObservableCollection SipRegistrations { get; set; } = new ObservableCollection();#endregionlock (_lockObj){    SipRegistrations.Remove(SipRegistrations.FirstOrDefault(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername));    SipRegistrations.Add(binding);          // 情况一    //SipRegistrations.Append (binding);     // 情况二}Console.WriteLine($" 注册联系人 [{binding.RegisteredContact}] 为 [{sipAccount.SIPUsername}].");

这个有两种情况(都不能成功)：

情况一，使用 Add 方法，结果是执行完 Add 方法后就返回了，后面的方法不再执行(不知道为什么)，界面上是有几率能添加一条。

情况二，使用 Append 方法，执行完 Append 后倒是可以继续执行后面的代码，但是界面上一条也出现不了。

后记：本文主要是抛砖引玉，大家有什么更好的方法，或者能解释文中所描述现象的原理，请不吝赐教。

项目地址：https://gitee.com/DLGCY_GB28181/SimpleSIPServer

---

更新(20200909)：

经过在 https://dotnet9.com/ 站长的技术讨论群的讨论，决定还是要使用 ObservableCollection。加上在网上搜到了文章《WPF ViewModel 中对 ObservableCollection 集合操作》，所以最终代码为：

#region 成员/// /// 加锁对象/// private object _lockObj = new object();/// /// 集合对象/// public ObservableCollection SipRegistrations { get; set; } = new ObservableCollection();#endregionlock (_lockObj){    ThreadPool.QueueUserWorkItem(delegate    {        SynchronizationContext.SetSynchronizationContext(new System.Windows.Threading.DispatcherSynchronizationContext(Application.Current.Dispatcher));        SynchronizationContext.Current?.Post(pl =>        {            SipRegistrations.Remove(SipRegistrations.FirstOrDefault(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername));            SipRegistrations.Add(binding);        }, null);    });}Console.WriteLine($" 注册联系人 [{binding.RegisteredContact}] 为 [{sipAccount.SIPUsername}].");

甚至还可以简化：

Application.Current.Dispatcher.Invoke(delegate{    SipRegistrations.Remove(SipRegistrations.FirstOrDefault(x => x.SIPAccount.SIPUsername == sipAccount.SIPUsername));    SipRegistrations.Add(binding);});