V2rayN 加载所有节点去重时不要影响原来的各分组订阅

问题

上一篇修改加载节点列表后自动去重, 最后发现一个问题. 点击 "所有" 以后, 调用节点去重功能, 然后再回到单独的分组订阅里面, 有节点被去掉了.

举个例子,
如果订阅A里面有节点 1 2 3 4 5, 订阅B里面有节点3 4 5 6 7. 那么全部里面是1 2 3 4 5 3 4 5 6 7, 再去重.
再去看订阅A和订阅B里面就会有节点被去掉了, 比如, 订阅B里面节点 3 4 5 没有了, 只剩节点 6 7 了.

说具体一点, 本来订阅节点是这样的, 订阅A里面有节点 1 2 3 4 5, 订阅B里面有节点3 4 5 6 7.

那么点击 "所有" 之后, 我们会得到 1 2 3 4 5 3 4 5 6 7 . 再去重就会得到这样:

然后再去分别查看订阅A 和订阅B, 就成了这样:

原理

查阅代码, 作者是把所有的订阅的节点都放在一个总的节点列表里面 (实际上是放在一个数据库里面), 也记载了这个节点是属于哪个分组订阅的.

界面上需要显示的时候, 就根据你选择的哪个分组, 还有你设置的服务器过滤器, 从这个总列表中把一部分节点过滤出来.

去重操作时, 那些节点就从这个总列表里被删掉了. 当然之后再选择单独分组的时候, 就加载不出来了.

思路

所以我们的思路就是, 当加载节点自动去重的时候, 不要操作底层的那个总列表.

上一篇教程的最后, 我们的修改成果是这样:

我们进到 RefreshServers 函数的内部去看看.

Ctrl + F12 或右键菜单转到实现.

里面大致是这么几部分.

RefreshServers从数据库读出lstModel，处理后赋值给lstProfile

RefreshServers末尾调用RefreshServersMenu，更新系统栏菜单

所以我们要做的就是在读取了节点列表 lstModel 之后, 进行去重处理, 再执行后面的步骤去显示.

具体实践

我们怎么做呢? 把 SubSelectedChanged 里面 RefreshServers(); 这个函数调用删除, 换成上面我们分析的这些代码, 然后再在中间插写一段给 lstModel 做节点去重的逻辑吗?

从逻辑上说, 这样是可行的, 也是正确的.

但是在编程开发的工程实践中, 这是糟糕的, 是代码中坏味道, 是代码重复. 以后如果需要修改(不管是增强开发, 还是修改BUG)都会带来霰弹枪手术的问题.

抱歉上面这一句话掉个书袋, 我是为了给有兴趣的朋友引个路, 介绍一下好的编程学习资料.

换个说法来解释一下. 首先我们要认识到一点, 没有代码是不会在未来修改的. 一是人不可能不犯错误, 二是程序的功能会发展增强.

如果我们把相同功能的代码到处拷贝, 过两个月你发现有BUG需要修改, 你还记得都拷贝到哪里去了吗? 如果有一个地方没有修改到, 就等于在程序中遗漏了BUG没有修改全. 就算你记得, 有10个地方要改, 如果改动比较大, 不是一两行的事, 你怎么保证每个地方都改正确了呢? 不会有地方手一抖, 脑子一抽, 少写了一行吗?

所以在编程开发的工程实践中, 如果有相同或相似功能的代码段, 我们总是将它们合理的安排到一个函数里面, 需要用到的时候, 以这个函数的名字来调用就可以了.

你可以 Shift + F12 搜索一下 RefreshServers 的所有引用, 看看有多少地方用到它, 如果每个地方都是把这一大段代码复制粘贴, 后果不敢想象.

重构

我们要做的事情是先将 RefreshServers 进行重构.

幸运的是, VS 工具提供了方便的操作.

我们选中第一行的代码, 右键菜单 - 快速操作和重构

然后选择提取方法

给提取出的函数取个名字, LoadProfilelist

再选中剩下的代码, 同样的提取方法, 取个名字 DisplayProfilelist

哦, 对了, 编程的世界里, 方法和函数这两个术语常常就是指一段可以重用的代码, 用一个名字来指示. 我个人觉得, 如果不是在某个考试的考卷上答题的话, 你可以认为它们就是同一个东西.

重构的作用就是调整程序的结构, 但是应该不影响程序本身的功能. 所以我们现在就测试一下.

可以把刚刚添加的调用原来的节点去重函数 RemoveDuplicateServer 先屏蔽掉. 用 // 设置为注释.

保存, 编译, 开始调试(或执行).

测试, 功能和修改之前完全一样.

还记得我们本来的修改思路吗? 要在获取节点列表和显示节点列表之间插入节点去重的逻辑.

所以现在我们把 RefreshServers 换成我们刚刚重构出来的两个函数 LoadProfilelist 和 DisplayProfilelist.

这一步修改应该不影响功能. 如果你不放心, 那就编译, 执行, 测试一下.

添加 lstModel 节点去重的逻辑

之前我们使用的 RemoveDuplicateServer 会修改底层保存的节点数据, 不能用了, 我们干脆删掉.

我们原计划是在读取节点列表和显示节点列表之间插入一下节点去重的逻辑.

那么这个逻辑应该怎么写呢?

我们可以学习一下原来的节点去重逻辑是怎么写的.

找到 RemoveDuplicateServer 函数的实现, 是这样的.

结合英文单词的意思猜一猜, 我们应该继续看 DedupServerList 函数的实现.

Ctrl + F12, 转到实现, 内容逻辑如下:

我们要学习的就是中间去重的逻辑部分.

所以我们这么写

List<ProfileItemModel> lstKeep = new();
foreach (ProfileItemModel item in lstModel)
{
  if (!lstKeep.Exists(i => CompareProfileItem(i, item, false)))
  {
    lstKeep.Add(item);
  }
}

然后再把 lstKeep 用于显示.