13

我正在尝试遵循使用向导 gem 对对象进行部分验证的指南，但我不断收到错误未定义方法“包括？” 对于 nil:NilClass，无法理解出了什么问题，已尝试按照分步说明进行操作。

日志中的错误显示。

NoMethodError - undefined method `include?' for nil:NilClass:
app/models/property.rb:22:in `active_or_tenants?'

这是我的步骤控制器。

class Properties::BuildController < ApplicationController
  include Wicked::Wizard

  steps :tenant, :confirmed 

  def show
    @property = Property.find(params[:property_id])
    @tenants = @property.tenants.new(params[:tenant_id])
    render_wizard
  end

  def update
    @property = Property.find(params[:property_id])
    params[:property][:status] = step.to_s
    params[:property][:status] = 'active' if step == steps.last
    @property.update_attributes(params[:property])
    render_wizard @property
  end

  def create
    @property = current_user.properties.build(params[:property])
      logger.info @property.attributes
    if @property.save
        flash[:success] = "Tenant Added"
        redirect_to wizard_path(steps.second, :property_id => @property.id)
    else
        render 'edit'
    end
  end
end

属性.rb

class Property < ActiveRecord::Base
  attr_accessible  :name, :address_attributes, :tenants_attributes, :property_id, :status
  belongs_to :user 

  has_one :address, :as => :addressable
  accepts_nested_attributes_for :address, :allow_destroy => true

  has_many :tenants 
  accepts_nested_attributes_for :tenants, :allow_destroy => true

  validates :name,        :presence => true
  validates :address,     :presence => true
  validates :tenants,     :presence => true, :if => :active_or_tenants?

  def active?
    status == 'active'
  end

  def active_or_tenants?
    status.include?('tenants') || active?
  end
end

让我知道您是否需要在问题中添加任何其他部分。提前致谢。

4

3 回答 3

8

从我的评论：

status是您的 Property 模型的一个属性。在某些情况下可能会nil引发错误：

undefined method include?' for nil:NilClass

它实际上是在尝试与nil( 'tenants'String) 进行比较。

要解决此问题，您可以使用空字符串进行比较，如果status是nil，

# an example (you can try in your IRB console):
nil || "No value"
# => returns "No value"

在你的情况下：

def active_or_tenants?
  status.to_s.include?('tenants') || active?
end

nil.to_s返回一个空字符串。这解决了你的问题;）

实际上，方法to_s,to_i等to_f经常用于删除可能的nil:

# in ruby console:
2.3.3 :018 > nil.to_i
# => 0 
2.3.3 :019 > nil.to_f
# => 0.0 
2.3.3 :020 > nil.to_s
# => "" 
2.3.3 :021 > nil.to_a
# => []
于 2013-08-05T17:06:23.023 回答
2

在 Ruby 2.3 中，您可以使用安全导航运算符，当您的对象为 nil 时，它将简单地返回 nil 并且不会抛出错误。

def active_or_tenants?
  status&.include?('tenants') || active?
end
于 2017-01-16T07:12:42.017 回答
1

对此还有另一种解决方案，您可能希望在创建对象时设置默认状态，这在迁移中更可取。

class AddStatusToProperties < ActiveRecord::Migration
  def change
    create_table :projects do |t|
      t.string :status, default: 'new'
    end
  end
end

在此之后，您将永远不会拥有nil您的州

于 2013-10-19T16:46:02.593 回答