Homebox项目中的搜索位置过滤器重复选择问题分析

Homebox是一款开源的家庭物品管理系统，在最新发布的v0.14.0版本中，用户报告了一个关于位置搜索过滤器的功能性问题。本文将深入分析该问题的技术原因及解决方案。

问题现象

在Homebox的搜索页面中，当用户通过位置过滤器选择多个位置时，系统会出现异常行为：除最后选择的一个位置外，其他所有已选位置都会在列表中重复显示。例如，用户先选择"卧室"，再选择"阁楼"后，"卧室"会在列表中显示两次。

技术分析

根本原因

该问题源于SearchFilter组件中的unselected计算属性实现方式。当前代码使用对象引用比较来判断项目是否已被选择：

const unselected = computed(() => {
  return props.options.filter(o => {
    if (searchFold.value.length > 0) {
      return o[props.display].toLowerCase().includes(searchFold.value) && !selected.value.includes(o);
    }
    return !selected.value.includes(o);
  });
});

这种实现方式在Vue的响应式系统中会导致比较失效，因为当对象被重新包装为响应式代理后，直接的对象引用比较将不再可靠。

解决方案

更健壮的实现应该基于对象的唯一标识属性进行比较，例如使用id字段：

const unselected = computed(() => {
  return props.options.filter(o => {
    if (searchFold.value.length > 0) {
      return o[props.display].toLowerCase().includes(searchFold.value) && selected.value.every(s => s['id'] !== o['id']);
    }
    return selected.value.every(s => s['id'] !== o['id']);
  });
});

架构考量

虽然上述解决方案能解决当前问题，但从组件设计的角度考虑，SearchFilter作为一个通用过滤组件，应当具备更强的适应性：

1. 不应假设所有过滤项都有id字段
2. 应允许调用方指定用于比较的字段名
3. 可考虑提供自定义比较函数的选项

建议的改进方向是添加一个prop，让父组件可以指定用于比较的字段名，例如：

props: {
  compareBy: {
    type: String,
    default: 'id'
  }
}

然后在比较逻辑中使用该字段：

selected.value.every(s => s[props.compareBy] !== o[props.compareBy])

影响范围

该问题主要影响以下场景：

• 搜索页面中使用位置过滤器
• 选择多个位置时
• 在过滤结果列表中查看未选择的项目

临时解决方案

对于无法立即升级系统的用户，可以采取以下临时措施：

1. 避免同时选择多个位置
2. 刷新页面后重新选择位置
3. 使用搜索框精确过滤而非多选

总结

Homebox的位置过滤器重复选择问题展示了在Vue响应式系统中对象比较的常见陷阱。通过改用基于唯一标识的比较方式，可以可靠地解决这一问题。对于通用组件设计，提供灵活的配置选项能够增强组件的复用性和健壮性。这类问题的解决不仅修复了当前缺陷，也为项目未来的可维护性奠定了基础。