ruby-on-rails - 如何进行更好的零检查？

Question

我收到以下错误：'未定义的方法'活动？' 对于零：NilClass'

当我尝试打电话时

@user.subscription.active?

因为有时订阅尚未创建。然后我写了一个方法来检查用户是否有一个现有的活动订阅：

  def active_subscription?
    if !self.subscription.nil?
      if self.subscription.active?
        return true
      else
        return false
      end
    else
      return false
    end
  end

有没有更好的方法来编写这个 active_subscription？没有所有 if else 检查和或返回语句的方法？

这是订阅模型中检查订阅状态是否处于活动状态的方法

def active?
 status == 'active'
end

Answer 1

这应该没问题：

def active_subscription?
  subscription && subscription.active?
end

甚至更短：

def active_subscription?
  subscription.try(:active?)
end

为确保此方法可以返回的唯一值是trueand false，您可以：

def active_subscription?
  !!subscription.try(:active?)
end

Answer 2

您可以使用delegate：

class User < ActiveRecord::Base
  has_one :subscription
  delegate :active?, to: :subscription, prefix: :subscription, allow_nil: true
  # ...
end

现在你可以打电话

user.subscription_active?

如果订阅可用且关联丢失，则返回subscription.active?（即true或）。falsenil

Answer 3

def active_subscription?
  subscription && subscription.active?
end

Answer 4

通过安全导航，您可以使用@user.subscription&.active

它在调用该方法之前检查 nil。

Answer 5

你可以这样做

@user.subscription.active? if @user.subscription.present?