Rails使用不同模型的新操作

时间:2015-10-09 04:36:28

标签: ruby-on-rails ruby strong-parameters

我在使用创建操作的Rails中遇到了问题 - 我在控制器中有这些信息:

ComputerController 
def create
  @computer = Computer.new(computer_params)
  redirect_to computers_path
end

private  
 def computer_params
 require.params(:computer).permit(:computer_name,
 :cpu_tag,:serial,:location,:brand,:model,:ram,:cpu,:os,:warranty,:comments)  
end

然后在我的模型中我有一些验证:

class Computer < ActiveRecord::Base
 validates :computer_name,  uniqueness: true, presence: true,
 length:{maximum: 12} 
 validates :cpu_tag, length: {maximum: 4}, uniqueness: true, 
 :numericality =>   {:only_integer => true}
 validates :serial, presence: true
 validates :location, presence: true
 validates :brand, presence: true
 validates :model, presence: true
 validates :ram, presence: true
 validates :cpu, presence: true
 validates :os, presence: true
 validates :warranty, presence: true
 validates :comments, presence: true
end

视图new.html.erb是:

<div class="row text-center">
 <h2 class = "mimsinfoblackindex">Add A Computer To The Inventory </h2><hr/>

<div class="col-md-3 description_pc text-left">
   <%= form_for @computer do |f|%>  

    <h4 class = "mimsformgreen">
      <%= f.label :computer_name,'Computer Name:'%>
      <%= f.text_field :computer_name%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :cpu_tag, 'Computer Tag:'%>
      <%= f.text_field :cpu_tag%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :serial, 'Serial:'%>
      <%= f.text_field :serial%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :location, 'Location:'%> 
      <%= f.text_field :location%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :brand, 'Brand:'%>
      <%= f.text_field :brand%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :model, 'Model:'%>
      <%= f.text_field :model%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :ram, 'Ram:'%> 
      <%= f.text_field :ram%>
    </h4>

    <h4 class = "mimsformblack">  
      <%= f.label :cpu, 'Processor:'%>
      <%= f.text_field :cpu %>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :os, 'Operating System:'%> 
      <%= f.text_field :os%>
    </h4>

    <h4 class = "mimsformblack">
      <%= f.label :warranty, 'Warranty:'%>
      <%= f.text_field :warranty%>
    </h4>

     <h4 class = "mimsformblack">
      <%= f.label :comments, 'Comments:'%>
      <%= f.text_field :comments%>
    </h4>

        <%= f.submit 'Add The Computer'%>
  <% end %>

我已经为我的模型做过TDD而且我没有遇到任何问题,但是当我提交计算机表单时,我在屏幕上收到一条错误消息:

 wrong number of arguments (0 for 1)
 private  
 def computer_params
  require.params(:computer).permit(:computer_name,:cpu_tag,
  :serial,:location,:brand,:model,:ram,:cpu,:os,:warranty,:comments)  
 end

2 个答案:

答案 0 :(得分:3)

尝试将computer_params重写为:

private  

def computer_params
  params.require(:computer).permit(:computer_name, :cpu_tag, :serial, :location, :brand, :model, :ram, :cpu, :os, :warranty, :comments)  
end

您的原始代码似乎颠倒了paramsrequire

希望它有所帮助!

答案 1 :(得分:0)

要添加答案,您可以使用代码进行一些修复:

1个验证

定义相同的presence验证时,您可以pass multiple arguments(属性)方法:

#app/models/computer.rb
class Computer < ActiveRecord::Base
   validates :serial, :location, :brand, :model, :ram, :cpu, :os, :warranty, :comments, presence: true
end

2 Params

Rails'strong params功能非常具体,说明您需要“需要”顶级参数,然后“允许”其子参数:

def computer_params
  params.require(:computer).permit(:computer_name,:cpu_tag, :serial,:location,:brand,:model,:ram,:cpu,:os,:warranty,:comments)  
end

3循环

在编程中,最有效的代码获胜。

这意味着你不应该复制一堆代码时间&amp;再次(使用attributes方法):

#app/views/computers/new.html.erb
<%= form_for @computer do |f| %>

    <% @computer.attributes.each do |attr| %>
       <% xtra = "green" if attr == :computer_name %>
       <%= content_tag :h4, class: "misform #{xtra}" do %>
          <%= f.label attr.to_sym, attr.titleize + ":" %>
          <%= f.text_field attr.to_sym %>
       <% end %>
    <% end %>

    <%= f.submit 'Add The Computer'%>
<% end %>

看看有多清洁?

4个HTML类

您使用了这两个类名:

mimsformblack mimsformgreen

查看我的#3建议,您能看到效率非常低吗?它违反了一个名为DRY (Don't Repeat Yourself)的原则,在这个原则中,你应该使用尽可能少的代码来使用尽可能少的代码。

您可以对每个元素应用multiple CSS classes,这意味着您将能够执行以下操作:

<div class="mimsform">This will be black</div> 
<div class="mimsform green">This will be green</div>

5创建

在Rails中创建时,您必须对象保存到模型中:

def create
  @computer = Computer.new computer_params
  redirect_to computers_path if @computer.save
end

许多新手开发人员不保存他们的新对象,阻止他们实际将数据保存到数据库。