rails 3 has_and_belongs_to_many NoMethodError未定义的方法`each'代表nil:NilClass

时间:2013-09-10 09:06:16

标签: ruby-on-rails ruby each nomethoderror has-and-belongs-to-many

我有两个模型UserConf。他们有has_and_belongs_to_many的关系。在创建新的conf对象时,我想将其分配给多个现有用户,但是我收到了这个错误:

NoMethodError in Confs#create
undefined method `each' for nil:NilClass

这些是我的模特:

class User < ActiveRecord::Base
  attr_accessible :email, :name, :password, :password_confirmation, :developer, :admin, :company_id, :boss_id
  belongs_to :company
  has_and_belongs_to_many :confs

  validates :company_id, presence: true
  validates :boss_id, presence: true
  validates :name,  presence: true, length: { maximum:50 }
end

class Conf < ActiveRecord::Base
  attr_accessible :id, :linear_axis_number, :control_unit_brand, :control_unit_model, :description, 
              :machine_brand, :machine_model, :milling_mode, :rotary_axis_number, :tool_axis_x, :tool_axis_y, 
              :tool_axis_z, :turning_mode, :machine_name, :developer_id, :xml, :users

  validates_presence_of :users
  has_and_belongs_to_many :users
  belongs_to :developer, :class_name => 'User', :foreign_key => 'developer_id'

  has_attached_file :xml, :url => "downloads/:attachment/:id/:basename.:extension", :path => ":rails_root/downloads/:attachment/:id/:basename.:extension"
  attr_protected :xml_file_name, :xml_content_type, :xml_file_size
end

这是confs_controller.rb:

class ConfsController < ApplicationController
  before_filter  :signed_in_user, only:[:index, :edit, :update, :destroy]
  before_filter  :developer_user, only: :destroy

  def new
    @users = User.where(:boss_id => current_user.id)
    @conf = Conf.new
  end

  def create
    @conf = Conf.new(conf_params)    
    if @conf.save
      flash[:success] = "New Configuration uploaded!"
      redirect_to conf_show_own_path
    else
      flash[:error] = "There is a problem!"
      render 'new'
    end
  end

  private
  def conf_params
    params.require(:conf).permit( :id, :linear_axis_number, :control_unit_brand, :control_unit_model, :xml,
                              :description, :machine_brand, :machine_model, :milling_mode, :developer_id,
                              :rotary_axis_number, :tool_axis_x, :tool_axis_y, :tool_axis_z, :turning_mode, 
                              :machine_name, :users) if params[:conf]
  end
end

这里是new.html.erb:

<%= form_for @conf, :html => {:multipart => true} do |f| %>

    <%= f.label :machine_name %>
    <%= f.text_field :machine_name %>
    ....
    <% @users.each do |g| %>
      <%= check_box_tag 'conf[user_ids][]', g.id, false, :id => g.name %>
      <%= label_tag g.name %>
    <% end %>

    <%= f.submit "Upload", class: "btn btn-large btn-primary" %>
<% end %>

1 个答案:

答案 0 :(得分:1)

您未在@users操作中设置create实例变量,因此如果您尝试在其上调用nil,则评估为each并引发错误。仅当您的记录未保存时才会发生这种情况,因为您的控制器会将new部分呈现。

这应该有效:

  def create
    @conf = Conf.new(conf_params)    
    if @conf.save
      flash[:success] = "New Configuration uploaded!"
      redirect_to conf_show_own_path
    else
      @users = User.where(:boss_id => current_user.id)
      flash[:error] = "There is a problem!"
      render 'new'
    end
  end