ruby-on-rails - 如何重构这个方法？

Question

我正在使用 Ruby on Rails 和 Stripe 构建一个小型支付处理模块，我想知道如何重构此方法(用于检查给定用户是否已经在 Stripe 中存有银行卡) :

class User < ActiveRecord::Base
...
def has_card?
    customer = Stripe::Customer.retrieve(self.stripe_customer_id)
    if customer.cards.count > 0
      true
    else
      false
    end
  end
end

我认为 if 语句看起来很愚蠢，但无法解释原因(我白天不是开发人员，我只是涉猎)

Answer 1

你的直觉是对的，但很愚蠢!

def has_card?
  Stripe::Customer.retrieve(stripe_customer_id).cards.count > 0
end