1

我得到这个错误结束不知道如何解决它。奇怪的是,它以前工作过。我想在我运行注释之后,它坏了,但不确定。错误来自 confs.controller 索引和自己的方法。它也拒绝这样的事情: conf.machine_brand[0,1].upcase as NoMethodError [ ] bla bla 这是我的 conf 模型:

# == Schema Information
#
# Table name: confs
#
#  id                 :integer          not null, primary key
#  machine_brand      :string(255)
#  machine_model      :string(255)
#  control_unit_brand :string(255)
#  control_unit_model :string(255)
#  tool_axis_x        :decimal(, )
#  tool_axis_y        :decimal(, )
#  tool_axis_z        :decimal(, )
#  rotary_axis_number :integer
#  linear_axis_number :integer
#  turning_mode       :boolean
#  milling_mode       :boolean
#  description        :text
#  xml                :text
#  user_id            :integer
#  developer_id       :integer
#  created_at         :datetime         not null
#  updated_at         :datetime         not null
#

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

   belongs_to :developer, :class_name => 'User', :foreign_key => 'developer_id'
   belongs_to :receiver, :class_name => 'User', :foreign_key => 'user_id'

   validates :user_id, presence: true
   validates :developer_id, presence: true
end

这是 confs.controller:

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

def new
  @conf = Conf.new
end

def index
  @grouped = {}
  Conf.all.each do |conf|
    letter = conf.machine_brand.slice(0,1).upcase
    @grouped[letter] ||= []
    @grouped[letter] << conf
end
end

def show
@conf = Conf.find(params[:id])

respond_to do |format|
    format.html #index.html.erb
    format.json { render json: @conf }
    format.xml { render xml: @conf }  
  end
end

def own
  @grouped = {}
  Conf.where(:developer_id => current_user.id).each do |conf|
    letter = conf.machine_brand.slice(0,1).upcase
    @grouped[letter] ||= []
    @grouped[letter] << conf
  end
end

def create
@conf = Conf.new(conf_params)

  if @conf.save
    flash[:success] = "New Configuration uploaded!"
    redirect_to conf_show_path
  else
    flash[:error] = "There is a problem!"
    render 'new'
  end
end

def destroy
  @conf = Conf.find(params[:id]).destroy
  redirect_to conf_show_own_path
end

def update
  @conf.update_attributes(params[:conf])  
end

private

def signed_in_user
  unless signed_in?
    store_location
    redirect_to signin_url, notice: "Please sign in"    
  end
end

def admin_user
  redirect_to(root_path) unless current_user.admin?
end

def developer_user
  redirect_to(root_path) unless current_user.developer?
end

def conf_params
  params.require(:conf).permit(:xml, :user_id, :developer_id) if params[:conf]
end

end

如果您愿意,这是 conf.new:

<% provide(:title, 'New Configuration')%>
<h1> Upload new configuration </h1>

<div class="row">
  <div class="span6 offset3">

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

    <%= f.label :machine_brand %>
    <%= f.text_field :machine_brand %>

    <%= f.label :machine_model %>
    <%= f.text_field :machine_model %>

    <%= f.label :control_unit_brand %>
    <%= f.text_field :control_unit_brand %>

    <%= f.label :control_unit_model %>
    <%= f.text_field :control_unit_model %>

    <%= f.label :tool_axis_x %>
    <%= f.text_field :tool_axis_x %>

    <%= f.label :tool_axis_y %>
    <%= f.text_field :tool_axis_y %>

    <%= f.label :tool_axis_z %>
    <%= f.text_field :tool_axis_z %>

    <%= f.label :rotary_axis_number %>
    <%= f.text_field :rotary_axis_number %>

    <%= f.label :linear_axis_number %>
    <%= f.text_field :linear_axis_number %>

    <%= f.label :turning_mode %>
    <%= f.text_field :turning_mode %>

    <%= f.label :milling_mode %>
    <%= f.text_field :milling_mode %>

    <%= f.label :description %>
    <%= f.text_field :description %>

    <%= f.label :xml %>
    <%= f.text_field :xml %>

    <%= f.label :client %>
    <%= f.collection_select :user_id, User.where(:admin => false, :developer => false), :id, :name, options ={:prompt => "Select a client"}, :class =>"user" %>

    <%= f.label :me %>
    <%= f.collection_select :developer_id, User.where(:id => current_user.id), :id, :name, options ={:prompt => "Select me"}, :class =>"user" %>

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

2 回答 2

1

conf.machine_brand.slice(0,1)
我认为您在这里遇到了错误,machine_brand所以只需在您的控制器中执行
letter = params[:machine_brand].to_s.slice(0,1).upcase unless params[:machine_brand].blank?

letter = params[:conf][:machine_brand].to_s.slice(0,1).upcase unless params[:machine_brand].blank?

于 2013-08-26T07:59:59.477 回答
0

正如@Rajarshi 提到的,错误在以下代码中

conf.machine_brand.slice(0,1).upcase

该错误表明您正在调用slice一个nil对象,这意味着您的一个 conf 记录的 machine_brand 为零。我不确定你想如何解决这个问题,但是如果你添加一个需要 machine_brand 的验证,这个问题将会减少

class Conf < ActiveRecord::Base
  ...
  validates :machine_brand, presence: true

或者您只能获取machine_brand存在的记录

Conf.where('machine_brand IS NOT NULL').each do |conf|
  conf.machine_branch.slice(...)
于 2013-08-26T08:16:08.933 回答