ActiveStorage中rescue StandardError未捕获RecordNotFound异常的原因排查

你遇到的问题核心在于rescue块的作用域没有覆盖到抛出异常的代码，或者更准确地说，你的rescue写法可能存在缩进/结构错误，导致它无法捕获方法执行时抛出的ActiveRecord::RecordNotFound异常。

为什么rescue没生效？ #

在Ruby中，方法内部的rescue块需要包裹可能抛出异常的代码逻辑。看你的代码写法：

def blob
  @blob ||= ActiveStorage::Blob.find_signed(signed_id) unless signed_id.blank?
rescue StandardError => error
  # 处理错误
end

如果你的代码实际缩进是把rescue放在了def...end方法块的外部（比如不小心换行后缩进错误），那这个rescue只会捕获方法定义过程中的异常，而不会捕获方法调用时抛出的异常。这就解释了为什么find_signed抛出的RecordNotFound直接传到了控制器，根本没进入你的rescue处理逻辑。

退一步说，即使rescue在方法内部，你当前的写法还有个潜在问题：如果find_signed抛出异常，@blob不会被赋值，下次调用blob方法时会再次执行find_signed，重复抛出异常。

修复方案 #

你需要确保rescue块在方法内部，并且明确处理异常后给@blob赋值（或者返回一个安全的默认值），避免重复执行抛出异常的代码。

方案1：明确使用begin...end包裹逻辑（推荐，可读性更高）

def blob
  begin
    @blob ||= ActiveStorage::Blob.find_signed(signed_id) unless signed_id.blank?
  rescue StandardError => error
    # 添加错误处理逻辑，比如日志记录
    Rails.logger.error("Failed to retrieve ActiveStorage blob: #{error.message}")
    # 给@blob赋值为nil，避免下次调用重复执行find_signed
    @blob = nil
  end
end

方案2：简化写法（确保rescue在方法内部）

如果你喜欢简洁的写法，只要保证rescue和方法体代码在同一个def...end块内即可：

def blob
  @blob ||= ActiveStorage::Blob.find_signed(signed_id) unless signed_id.blank?
rescue StandardError => error
  Rails.logger.error("Blob retrieval failed: #{error.message}")
  @blob = nil
end

额外注意事项 #

1. 捕获更具体的异常：虽然StandardError能覆盖RecordNotFound和ActiveStorage::InvalidSignature（签名无效时抛出），但为了更精准，你可以直接捕获这两个异常：

   rescue ActiveRecord::RecordNotFound, ActiveStorage::InvalidSignature => error
     # 处理逻辑
   

2. 记忆化的正确性：在rescue块中给@blob赋值为nil，这样下次调用blob方法时，@blob ||= ...会直接返回nil，不会重复执行find_signed，避免重复抛出异常。

内容的提问来源于stack exchange，提问作者Austin Turner