Ruby - 即使抛出了 Stripe 异常，else 块中的代码也会执行

Question

我的代码的简化版：

1.upto(2) do |try|
  begin
    puts "Start"
  rescue => e
    if try == 2
      puts "Error - #{e}"
    end
  else
    puts "Success"
  end
end

我的问题是，当我遇到错误时，会发生这种情况：

Start
Start
Error - some message
Success

这里真的让我摸不着头脑，因为这不是应该发生的事情......有什么想法吗？

更新

好的，我认为我通过输入代码的简化版本对自己造成了伤害，因为这可行，但下面的完整版本却不行。即，当 Stripe 抛出 API 异常时，电子邮件仍在触发。

1.upto(2) do |try|
  Stripe.api_key = ENV['Stripe_Secret_Key']
  begin
    customer = Stripe::Customer.create(
      :source => params[:stripeToken],
      :email => @rentalrequest.email
    )
    @charge = Stripe::Charge.create(
      amount: (@rentalrequest.actual_amount * 100), # Stripe requires cents
      currency: "usd",
      customer: customer.id,
      description: "Payment from #{@rentalrequest.email} for gear rental"
    )
  rescue => e
    if try == 2
      logger.fatal "Stripe error: #{e.class}, #{e.message}"
      StripeErrorEmail.new.async.perform(e.class, e.message, @rentalrequest.id)
      render "rental_requests/new" and return
    end
  else
    RoadrunnerEmailAlert.new.async.perform(@rentalrequest.id)
  end
end

换句话说，StripeErrorEmail 和 RoadrunnerEmailAlert 都在开火

更新 KM 的答案

def error_handle(e, id)
  # Since it's a decline, Stripe::CardError will be caught
  body = e.json_body
  err  = body[:error]

  puts "Status is: #{e.http_status}"
  # => 400
  puts "Type is: #{err[:type]}"
  # => invalid_request_error
  puts "Code is: #{err[:code]}"
  # => nil
  puts "Param is: #{err[:param]}"
  # => nil
  puts "Message is: #{err[:message]}"
  # => You cannot use a Stripe token more than once

  logger.fatal "Stripe error: #{e.class}, #{e.message}"
  StripeErrorEmail.new.async.perform(e.class, e.message, id)
  render "rental_requests/new" and return
end

def charge
  @rentalrequest = RentalRequest.find(params[:rentalrequest_id])

  # Up to two tries
  1.upto(2) do |try|
    Stripe.api_key = ENV['Stripe_Secret_Key']
    begin
      customer = Stripe::Customer.create(
        :source => params[:stripeToken],
        :email => @rentalrequest.email
      )
      @charge = Stripe::Charge.create(
        amount: (@rentalrequest.actual_amount * 100), # Stripe requires cents
        currency: "usd",
        customer: customer.id,
        description: "Payment from #{@rentalrequest.email} for gear rental"
      )
    rescue Stripe::CardError => e
      if try == 2
        error_handle(e, @rentalrequest.id)
      end
    rescue Stripe::InvalidRequestError => e
      if try == 2
        error_handle(e, @rentalrequest.id)
      end
    rescue Stripe::AuthenticationError => e
      if try == 2
        error_handle(e, @rentalrequest.id)
      end
    rescue Stripe::APIConnectionError => e
      if try == 2
        error_handle(e, @rentalrequest.id)
      end
    rescue Stripe::StripeError => e
      if try == 2
        error_handle(e, @rentalrequest.id)
      end
    rescue => e
      if try == 2
        error_handle(e, @rentalrequest.id)
      end  
    else
      RoadrunnerEmailAlert.new.async.perform
    end
  end
end

Answer 1

不，如果出现异常，else 块不会被执行。

我已修改您的代码以引发异常，以便我们对其进行测试：

1.upto(2) do |try|  
  begin
    puts "Start"
    raise 'xxxx'
  rescue => e
    if try == 2
      puts "Error - #{e}"
    end
  else
    puts "Success"
  end
end

这是输出：

Start
Start
Error - xxxx

这清楚地描述了else块在出现异常时不会被执行。

else 块用于当块完成时没有抛出异常。另一种是ensure，无论块是否成功完成，都会执行。

更新

查看您的最新代码，似乎是条带问题。尝试像这样更优雅地处理条带错误：

begin
  customer = Stripe::Customer.create(
  :source => params[:stripeToken],
  :email => @rentalrequest.email
)

@charge = Stripe::Charge.create(
  amount: (@rentalrequest.actual_amount * 100), # Stripe requires cents
  currency: "usd",
  customer: customer.id,
  description: "Payment from #{@rentalrequest.email} for gear rental"
)
rescue Stripe::CardError => e
  # Since it's a decline, Stripe::CardError will be caught
  body = e.json_body
  err  = body[:error]

  puts "Status is: #{e.http_status}"
  puts "Type is: #{err[:type]}"
  puts "Code is: #{err[:code]}"
  # param is '' in this case
  puts "Param is: #{err[:param]}"
  puts "Message is: #{err[:message]}"
rescue Stripe::InvalidRequestError => e
  # Invalid parameters were supplied to Stripe's API
rescue Stripe::AuthenticationError => e
  # Authentication with Stripe's API failed
  # (maybe you changed API keys recently)
rescue Stripe::APIConnectionError => e
  # Network communication with Stripe failed
rescue Stripe::StripeError => e
  # Display a very generic error to the user, and maybe send
  # yourself an email
rescue => e
  # Something else happened, completely unrelated to Stripe
end

看看能不能解决你的问题。