"在哪里写开始救援 Ruby（在 Rails 上）"

Question

当我写 ruby​​ 或 (rails) 时，我应该在哪里写关于 http 请求的begin和rescue （或者一般来说，在哪里写 begin rescue）？

我说的是控制器、模型或模块。

例如，我写了这样的代码。

module

module WhateverModule
  def get_list
    http, uri_path, headers = set_request("whatever_api_url")
    http.get(uri_path, headers)
  end

  #codes continue below

end

然后，我经常将begin和rescue代码放在控制器上。

controller

begin
  api_list = get_list
rescue => exception
  # do whatever I need to do to handle exceptions
  p "exception happened!"
end

但我的 PR 审阅者一直说“你应该写这样的代码”

module

module WhateverModule
  def get_list
    http, uri_path, headers = set_request("whatever_api_url")
    
    begin
      http.get(uri_path, headers)
    rescue => exception
      p "exception happend!"
    end
  end

  #codes continue below

end

我的问题是，哪一个是正确的（或干净或更好）？ 关于从哪里begin和rescue有什么普遍的共识吗？

谢谢

Answer 1

我会争辩说，在使用rescue块时，遵循两个准则是一种常见的模式：

1. 仅将rescue块放置在尽可能少的行数周围。 通常，这应该只有一行。 这当然取决于您的代码，但总的来说，特定方法会引发异常。 如果可能， rescue块应该只在该块周围。 这使得调试和重构变得更加容易。
2. 添加rescue块时，请准确命名您希望发生的那些异常。 否则，您可能会发现意外的错误（语法错误或nil上的方法调用）。 又是什么让调试变得更加困难。

在您的示例中，我同意 PR 审阅者的观点，但我会添加可能发生且我想要处理的特定异常。 例如像这样：

def get_list
  http, uri_path, headers = set_request("whatever_api_url")

  begin
    http.get(uri_path, headers)
  rescue Net::HTTPNotFound => e
    # handle page not found
  rescue Net::HTTPForbidden => e
    # handle invalid credentials
  rescue Net::HTTPServerError, Net::HTTPServerException
    # handle server error
  end
end

当然，如何处理异常以及是否有可能取决于您的应用程序。 因此，没有硬性规定，只有指导方针。