例外処理を共通化したい

Question

実現したいこと

以下の例外処理のrescueを何かの方法でまとめて、一行で呼び出す方法を知りたい。
service層に例外処理を置いて試してみたのですが私では出来ませんでした...
何回も、複数行に渡って同じ例外処理を書くのは生産性にかけると思うのでもし共通化する方法があればご教示いただきたいです。

  def create
    ActiveRecord::Base.transaction do
      Payjp::Customer.retrieve('cus_xxxx')
    end
  rescue Payjp::CardError => e
    error_message(e, 1)
  rescue Stripe::RateLimitError => e
    error_message(e, 2)
  rescue Payjp::InvalidRequestError => e
    error_message(e, 3)
  rescue Payjp::AuthenticationError => e
    error_message(e, 4)
  rescue Payjp::APIConnectionError => e
    error_message(e, 5)
  rescue Payjp::StripeError => e
    error_message(e, 6)
  rescue => e
    error_message(e, 7)
  end

  private

  def error_message(e, type)
    case type
    when 1 then puts "subject: card is incorrect"
    when 2 then puts "subject: Too many requests made to the API too quickly"
    end
    # puts e.error # NOTE only development
    Rails.logger.debug(e)
  end

Comments

[maisumakun]

error_messageはどのようなメソッドですか？

（番号をrescue側で付与せずに、error_messageの側に任せてしまうことはできないのでしょうか？）

[nanase21]

エラー内容をログに吐き出す処理が書かれています。

Answer 1

error_messageの側に振り分け処理を移設する、という形ではどうでしょうか？

  def create
    ActiveRecord::Base.transaction do
      Payjp::Customer.retrieve('cus_xxxx')
    end
  rescue => e
    error_message(e)
  end

  ERROR_MESSAGES = {
    Payjp::CardError => "subject: card is incorrect", 
    Stripe::RateLimitError => "subject: Too many requests made to the API too quickly"
  }.freeze

  def error_message(e)
    message = ERROR_MESSAGES[e.class]
    puts message if message
    # puts e.error # NOTE only development
    Rails.logger.debug(e)
  end

Comments

[nanase21]

ご教示頂きありがとうございます。
rescue => e
error_message(e, 7)
どの例外か分からなかったっ場合に、以下のような形にしました。
例外のclass（今回で言うPayJP）がなかった場合も考えれるのでその場合は、error_messageメソッド内にunkownというメッセージを作成し、三項演算子でclassがあるかどうかをチェックするようにしたのですが、このような形でも大丈夫でしょうか？
もしもっと良い方法がありましたら、二度手間をおかけして大変恐縮ですがご教示いただけると幸いです。
message = ERROR_MESSAGES[e.class]
unkown = "subject: Something else happened, completely unrelated to PayJP"
message.present? ? puts(message) : puts(unkown)

[maisumakun]

それでもいいですが、もっとシンプルに「puts(message || ""subject: Something else happened, completely unrelated to PayJP")」のような形にもできます。

[nanase21]

ありがとうございます。
ご教示頂いた方がシンプルですね。
教えていただいた方法にしたい思います。
ご教示いただきありがとうございました。