工具借用系统中修改Reservation时工具可用性判断的优化咨询

在工具借用应用的预约编辑流程中，用户会修改同一工具的预约时间，常见情况是新取件时间（pickup_at）早于当前预约的取件时间，和/或新归还时间（return_at）晚于当前预约的归还时间。

举个具体案例：
当前预约信息：

{ 
  "tool": "impact wrench", 
  "pickup_at": "05-06-2023 8:00 AM", 
  "return_at": "05-07-2023 6:00 PM" 
}

用户希望修改为：

{
  "tool": "impact wrench", 
  "pickup_at": "05-05-2023 8:00 AM", 
  "return_at": "05-08-2023 2:00 PM" 
}

目前采用的冲突检测逻辑是分两次查询，分别检查「新取件到当前取件」「当前归还到新归还」这两个时间段是否存在其他预约冲突：

a = Reservation.where("tool_id = ? and pickup_at <= ? and return_at >= ?", 
       t_id, new_pickup_at, current_pickup_at).any?

b = Reservation.where("tool_id = ? and pickup_at <= ? and return_at >= ?", 
       t_id, current_return_at, new_return_at).any?

tool_available = a & b

请问有没有更优、更高效的实现方式？

可以将两次数据库查询合并为一次，同时排除当前正在编辑的预约本身（避免误判自身为冲突），直接检查**整个新预约时间段（new_pickup_at 至 new_return_at）**内是否存在其他冲突的预约。

核心逻辑 #

两个时间段存在冲突的本质是：对方的取件时间早于我的归还时间，且对方的归还时间晚于我的取件时间。针对编辑场景，必须排除当前预约ID，否则会和自己的原有记录冲突。

优化后代码示例 #

# 假设当前预约的ID为 current_reservation_id
conflict_exists = Reservation.where(
  "tool_id = ? AND id != ? AND pickup_at < ? AND return_at > ?",
  t_id, current_reservation_id, new_return_at, new_pickup_at
).any?

tool_available = !conflict_exists

优势说明 #

1. 减少数据库IO：从两次查询合并为一次，降低了数据库交互开销
2. 逻辑更全面：直接覆盖所有可能的冲突场景，无需拆分时间段检查，避免遗漏边界情况
3. 代码更简洁：逻辑清晰易懂，维护成本更低

数据库特定优化（可选） #

如果使用支持时间范围操作的数据库（如PostgreSQL），可以利用内置的重叠运算符进一步简化语法：

# PostgreSQL 专属写法，利用 OVERLAPS 运算符检测时间范围重叠
conflict_exists = Reservation.where(
  tool_id: t_id,
  id: !current_reservation_id,
  "(pickup_at, return_at) OVERLAPS (?, ?)" => [new_pickup_at, new_return_at]
).any?

内容的提问来源于stack exchange，提问作者dr.