Meteor项目中Underscore 1.6.0版本的`_.intersection`函数问题分析

在Meteor项目的开发过程中，开发者发现了一个与Underscore.js库相关的功能性问题。具体表现为Underscore 1.6.0版本中的_.intersection函数返回了不符合预期的结果。

问题背景

_.intersection是Underscore.js库中一个常用的集合操作函数，用于计算多个数组的交集。在Meteor 2.15版本中集成的Underscore 1.6.0版本中，该函数的实现存在逻辑错误。

问题代码分析

问题出现在函数的实现逻辑中。当前Meteor中的实现代码如下：

_.intersection = function(array) {
  var rest = slice.call(arguments, 1);
  return _.filter(_.uniq(array), function(item) {
    return _.every(rest, function(other) {
      return _.contains(other, item) >= 0;
    });
  });
};

关键问题在于_.contains(other, item) >= 0这一比较操作。_.contains函数实际上返回的是一个布尔值（true或false），表示元素是否存在于数组中。然而代码中却将其与0进行比较，这显然是不合理的。

正确实现对比

在Underscore.js官方仓库的1.6.0版本中，正确的实现应该是：

return _.contains(other, item);

不需要进行任何数值比较，直接使用_.contains返回的布尔值即可。

问题影响

这个错误会导致_.intersection函数的返回结果不正确。具体表现为：

1. 当元素存在于比较数组中时，_.contains返回true（相当于1），比较1 >= 0结果为true
2. 当元素不存在于比较数组中时，_.contains返回false（相当于0），比较0 >= 0结果也为true

这会导致函数实际上无法正确过滤出真正的交集元素，而是返回了第一个数组中的所有唯一元素。

解决方案

Meteor项目团队已经注意到了这个问题，并在进行Underscore库的版本升级工作。修复方案很简单，只需移除不必要的>= 0比较操作即可。

开发者建议

对于使用Meteor 2.15版本的开发者，如果项目中依赖_.intersection函数的正确性，可以考虑以下临时解决方案之一：

1. 在项目中覆盖Underscore的_.intersection实现
2. 等待官方发布修复版本
3. 使用其他方法实现交集功能

这个问题也提醒我们在集成第三方库时，需要仔细检查关键函数的实现是否正确，特别是在版本升级过程中。