Rails 7中如何避免重复编写API错误捕获代码？

针对你需要在50个API端点重复编写rescue Errors::NotFoundError的问题，有几种简洁的方式可以消除代码重复：

方案1：提取通用错误处理包装器 #

创建一个通用模块，封装捕获Errors::NotFoundError并返回nil的逻辑，然后在所有API方法中复用这个包装器：

# lib/api_error_handler.rb
module ApiErrorHandler
  def handle_not_found_error
    yield
  rescue Errors::NotFoundError
    nil
  end
end

# lib/custom_api.rb
module CustomApi
  extend self
  include ApiErrorHandler

  def fetch_transactions(user_id)
    handle_not_found_error do
      client.transactions.list(user_id:)
    end
  end

  def fetch_invoices(user_id)
    handle_not_found_error do
      client.invoices.list(user_id:)
    end
  end

  # 其他48个API方法同理，用handle_not_found_error包裹业务逻辑
end

这种方式的优势是直观，每个方法的错误处理逻辑一目了然，同时避免了重复编写rescue块。

方案2：用元编程批量注入错误处理 #

如果你的API方法命名有规律，或者想一次性给所有方法添加错误处理，可以用元编程自动修改方法定义：

# lib/custom_api.rb
module CustomApi
  extend self

  # 定义一个宏，批量给指定方法添加NotFound错误处理
  def self.rescue_not_found(*method_names)
    method_names.each do |name|
      original_method = instance_method(name)
      define_method(name) do |*args, **kwargs|
        original_method.bind(self).call(*args, **kwargs)
      rescue Errors::NotFoundError
        nil
      end
    end
  end

  # 原始API方法，无需写rescue
  def fetch_transactions(user_id)
    client.transactions.list(user_id:)
  end

  def fetch_invoices(user_id)
    client.invoices.list(user_id:)
  end

  def fetch_user_profile(user_id)
    client.user_profiles.get(user_id:)
  end

  # 批量指定需要处理的方法
  rescue_not_found :fetch_transactions, :fetch_invoices, :fetch_user_profile, 
                   # ... 剩下的47个方法名
end

这种方式适合方法数量多的场景，只需要在所有方法定义完成后，一次性注册需要处理的方法即可。

额外优化：简化控制器逻辑 #

顺便提一下，你控制器里的判断逻辑可以简化，无需重复赋值@transactions：

# transactions_controller.rb
class TransactionsController < ApplicationController
  def index
    @transactions = CustomApi.fetch_transactions(current_user.id)
    flash[:alert] = 'No transactions' unless @transactions
  end
end

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