Turms项目中用户关系查询的集合运算优化

在Turms即时通讯服务项目中，UserRelationshipService类负责处理用户关系相关的业务逻辑。近期发现了一个关于queryRelatedUserIds方法中集合运算的重要优化点，本文将详细分析这个问题及其解决方案。

问题背景

在用户关系服务中，经常需要查询与某个用户相关联的其他用户ID集合。queryRelatedUserIds方法就是用于实现这一功能的，它需要处理两个用户集合：一个是已经建立关系的用户集合，另一个是待查询的用户集合。

原实现分析

原代码实现中直接使用了CollectionUtils.containsAny方法来检查两个集合是否有交集。这种实现存在逻辑缺陷，因为它只判断是否有交集，而没有真正获取交集结果。这会导致返回的用户ID集合可能包含不在待查询集合中的用户。

技术原理

正确的实现应该使用集合的交集运算。集合交集指的是两个集合中都存在的元素。在Java中，可以通过以下方式实现：

1. 使用CollectionUtils.intersection方法（Apache Commons Collections）
2. 使用Java 8 Stream API的filter和contains组合
3. 使用retainAll方法（会修改原集合）

解决方案

最优解决方案是使用CollectionUtils.intersection方法，因为：

1. 它不会修改原集合
2. 它返回一个新的集合实例
3. 它已经过充分测试，性能可靠

具体实现应该先计算两个集合的交集，然后基于交集结果进行后续处理。这样可以确保返回的用户ID集合严格限定在待查询的用户集合范围内。

影响范围

这个优化影响所有依赖queryRelatedUserIds方法的业务逻辑，特别是：

1. 用户关系查询
2. 用户权限校验
3. 消息推送目标用户筛选

最佳实践

在处理类似集合运算时，开发者应该：

1. 明确区分"存在交集"和"获取交集"两种需求
2. 选择不修改原集合的方法
3. 考虑集合大小对性能的影响
4. 对空集合情况进行特殊处理

这个优化体现了在即时通讯系统中精确控制数据范围的重要性，特别是在处理用户关系这类核心业务逻辑时，每一个细节都可能影响系统的正确性和安全性。