我正在尝试解决我正在处理的一些现有代码的问题,其中 ObservableCollection 是从列表构建的,但是当项目从第一个列表中删除时(当它从头开始重新生成时(,它们不会从 ObservableCollection 中删除。该列表作为 DLL 的一部分生成,ObservableCollection 位于 exe 文件的一部分中,数据通过使用委托/事件/调用在两者之间传递。该列表指向一个类,该类包含一个索引,该索引可以唯一标识该类的每个实例。
代码如下所示:
.DLL
public delegate void EntryListUpdateDelegate(string sender, CarInfo car);
public event EntryListUpdateDelegate OnEntrylistUpdate;
List<CarInfo> _entryListCars = new List<CarInfo>();
case InboundMessageTypes.ENTRY_LIST:
{
_entryListCars.Clear();
var carEntryCount = br.ReadUInt16();
for (int i = 0; i < carEntryCount; i++)
{
_entryListCars.Add(new CarInfo(br.ReadUInt16()));
}
}
case InboundMessageTypes.ENTRY_LIST_CAR:
{
var carId = br.ReadUInt16();
var carInfo = _entryListCars.SingleOrDefault(x => x.CarIndex == carId);
// ... Set attributes for car e.g. carInfo.TeamName = ReadString(br);
OnEntrylistUpdate?.Invoke(ConnectionIdentifier, carInfo);
}
定期运行上述代码以重新创建 entryList 并使其保持最新。
CarInfo 结构如下所示:
public class CarInfo
{
public ushort CarIndex { get; }
....
}
.EXE
数据使用委托事件从 DLL 传递到 EXE,ObservableCollection 在 ViewModel 中创建,如下所示:
public ObservableCollection<CarViewModel> Cars { get; } = new ObservableCollection<CarViewModel>();
internal void RegisterNewClient(MyUdpRemoteClient newClient)
{
if (newClient.MsRealtimeUpdateInterval > 0)
{
// This client will send realtime updates, we should listen
newClient.MessageHandler.OnTrackDataUpdate += MessageHandler_OnTrackDataUpdate;
newClient.MessageHandler.OnEntrylistUpdate += MessageHandler_OnEntrylistUpdate;
newClient.MessageHandler.OnRealtimeUpdate += MessageHandler_OnRealtimeUpdate;
newClient.MessageHandler.OnRealtimeCarUpdate += MessageHandler_OnRealtimeCarUpdate;
}
_clients.Add(newClient.MessageHandler);
}
private void MessageHandler_OnEntrylistUpdate(string sender, CarInfo carUpdate)
{
CarViewModel vm = Cars.SingleOrDefault(x => x.CarIndex == carUpdate.CarIndex);
if (vm == null)
{
vm = new CarViewModel(carUpdate.CarIndex);
Cars.Add(vm);
}
vm.Update(carUpdate);
}
然后,在 xaml 文件中使用汽车,以在 CollectionViewSource 的 WPF GUI 的条目列表中显示汽车列表。
问题:
DLL 中的_entryListCars始终是更新和准确的,因为它每隔一段时间就会被清除和重新更新。出现的新车已成功添加到汽车可观察集合中,但断开连接/死亡的汽车不再是_entryListCars的一部分,永远不会从汽车可观察集合中删除。它只是越来越大,并继续显示死机/不活跃的汽车。
我的问题:
从汽车可观察收藏中删除死车/非活动车的最佳方法是什么?我想知道我是否能够使用 Except (list1.Except(list2((,然后删除 Cars ObservableCollection 中存在的所有不在 _entryListCars 列表中的汽车条目,使用 可能 包含在 CarIndex 和 Cars.Remove/RemoveAt/RemoveItem 中?实现此目的的最佳方法是什么,代码会是什么样子?
感谢您的帮助。
看着你的事件处理程序MessageHandler_OnEntrylistUpdate
命名会向我表明它需要一个列表作为事件参数,而不仅仅是一个新的CarInfo
。因此,为了保持这种命名,您可以"修复"此问题的一种方法是更改事件以将整个列表作为输入传递:
public event EventHandler<List<CarInfo>> OnEntrylistUpdate;
// ...
OnEntrylistUpdate.Invoke(_entryListCars);
然后在事件处理程序中(根据您的需要,您可能可以更智能地执行此操作(:
private void MessageHandler_OnEntrylistUpdate(string sender, List<CarInfo> updatedCarList)
{
var newCars = updatedCarList
.Where(uc => !Cars.Any(c => c.CarIndex == uc.CarIndex)
.Select(c => new CarViewModel(c.CarIndex))
.ToList();
var existingCars = Cars
.Where(c => updatedCarList.Any(uc => uc.CarIndex == c.CarIndex)
.ToList();
var removedCars = Cars
.Except(existingCars)
.ToList();
// Do whatever you need with your cars...
foreach (var car in newCars)
Cars.Add(car);
foreach (var car in removedCars)
Cars.Remove(car);
foreach (var car in newCars.Concat(existingCars))
car.Update(updatedCarList.Single(uc => uc.CarIndex == car.CarIndex));
}
话虽如此,我建议您研究是否可以避免每次都更新整个列表,而只是一次添加/删除/更新一辆车。这当然需要您的来源支持这一点。