【问题标题】:Need method create and not update when instance actually does exist当实例确实存在时需要方法创建而不是更新
【发布时间】:2014-09-17 00:02:43
【问题描述】:

Ruby 2.0.0、Rails 4.0.3

我有一个 _new 部分。但是,我用一个实际存在的实例来渲染它。这是必要的,以便我可以通过 JavaScript 在视图和控制器之间传递实例 ID,因为我实际上无法传递实例本身。我只是在新方法中调用 Class.first,这样我就有一个现有的实例可以在整个过程中使用。

我的问题是 _new 部分提交按钮识别出该实例已经存在。这会导致它更新而不是创建。该按钮字面意思是更新。按下时,它会路由到更新方法。那不是我想要的。我想要 create 方法,我将在其中创建一个填充了收集的参数的新实例。

怎么办?我只是携带一个虚拟实例来启动该过程是错误的吗?如果是这样,正确的解决方案是什么?如果这是可以接受的,我如何强制按钮创建而不是更新?

感谢所有帮助和 cmets。

编辑:我尝试了按钮的变体,试图强制它触发新方法。它继续触发更新。我最后一次失败的努力是:

  <button type="submit" formaction="new_admin_car_path" class="btn btn-default btn btn-primary">Create Car</button>

...结束编辑...

形式为:

<div class="span8">
  <% car_id = @car.id %>
  <%= simple_form_for [:admin, @car],
                      defaults: {label: false},
                      html: {id: 'new_admin_car', class: 'form-vertical', method: post},
                      wrapper: :vertical_form,
                      wrapper_mappings: {
                              check_boxes: :vertical_radio_and_checkboxes,
                              radio_buttons: :vertical_radio_and_checkboxes,
                              file: :vertical_file_input,
                              boolean: :vertical_boolean
                      } do |f| %>
      <%= f.input(:stock_number, {input_html: {form: 'new_admin_car', car: @car, value: nil}, autocomplete: :off, placeholder: 'Stock number?'}) %>
      <%= f.input(:ymm_year_id, {input_html: {form: 'new_admin_car', car_id: car_id, value: nil}, collection: YmmYear.all.order("year desc").collect{|c| [c.year, c.id]}, prompt: "Year?"}) %>
      <%= render partial: "makes", locals: {form: 'new_admin_car', car_id: car_id} %>
      <%= render partial: "models", locals: {form: 'new_admin_car', car_id: car_id} %>
      <%= f.association(:color, {input_html: {form: 'new_admin_car', value: nil}, autocomplete: :off, prompt: 'Color?'}) %>
      <div class="col-xs-6 col-sm-3">
        <br/>
      <input type="submit" form="new_admin_car" value="Create Car" class="btn btn-default btn btn-primary">
  <% end %>
  </div>
</div>

局部化:

<% unless car_id.blank? %>
    <% car = Car.find(car_id) %>
    <%# car.ymm_make_id = nil %>
    <%= simple_form_for [:admin, car],
                        defaults: {label: false},
                        remote: true do |f| %>
        <% makes ||= "" %>
        <% make = "" %>
        <% make = car.make_id if car.class == Car and Car.exists?(car.id) %>
        <% if !makes.blank? %>
            <%= f.input :ymm_make_id, {input_html: {form: form, car: car, car_id: car.id, value: make}, collection: makes.collect { |s| [s.make, s.id] }, prompt: "Make?"} %>
        <% else %>
            <%= f.input :ymm_make_id, {input_html: {form: form, car: car, car_id: car.id, value: make}, collection: [], prompt: "Make?"} %>
        <% end %>
    <% end %>
<% end %>

Controller Car Method New where clear 只是清除所有字段:

  def new
    @car = Car.first
    @car.clear
  end

呈现形式:

表单的 JavaScript 是:

// when the #year field changes
$("#car_ymm_year_id").change(function () {
    // make a GET call and replace the content
    // First select identifies what has been selected, or fired
    var year = $('select#car_ymm_year_id :selected').val();
    // Pull the variables from the input_html tag
    var form = $('select#car_ymm_year_id').attr("form");
    var car_id  = $('select#car_ymm_year_id').attr("car_id");
    // Routes to the controller action
    $.post('/admin/cars/make_list/',
        {
            form: form,
            year: year,
            car_id: car_id
        },
        function (data) {
            $("#car_ymm_make_id").html(data);
        });
    return false;
});

控制器方法:

  def make_list
    makes = YmmMake.makes(params[:year])
    #@car = Car.find(params[:car_id])
    render partial: "makes", locals: {car_id: params[:car_id], form: params[:form], makes: makes}
  end

【问题讨论】:

  • 您能否详细说明通过 JavaScript 在视图和控制器之间传递实例 ID 的含义?我们也许可以在那里找到更好的解决方案,并从一开始就避免整个事情。使用现有实例,然后强制表单助手将其视为带有撬棒的新实例确实听起来是个坏主意。
  • @janfoeh 我试图保持简单,但这并不奏效……我已经更新了问题以包括所有相关的部分、表单和方法。问题是我有一系列依赖选择。当我为汽车选择一年时,它会返回当年制造汽车的所有“制造商”或制造商。等等......为此,我使用了一个实例。请参考代码,如果您还有其他问题,请告诉我。谢谢。
  • 我认为您的代码存在一些误解和完全错误。我将创建一个空白的、干净的版本,说明您正在尝试实现的目标,并尝试在此过程中清除它们。

标签: javascript ruby-on-rails ruby


【解决方案1】:

如果我理解正确,您希望实现以下目标:

显示用于创建新汽车模型的表单。在这种形式中,用户输入年份;然后系统会从服务器加载该年份的所有品牌,并提供一个选择下拉菜单供用户从中选择品牌。

汽车模型属于_to ymm_make,所以我们在提交表单时包含了选择的ymm_make_id。

这是我将如何解决这个问题。我将使用标准的 Rails 表单助手,这样我们就少了一个需要担心的抽象层。

表格(Car.new):

<%= form_for [:admin, Car.new] do |f| %>
  <%= f.text_field :stock_number, autocomplete: "off", placeholder: "Stock number?" %>
  <%= text_field_tag :year_search, nil, placeholder: "Year" %>
  <%= f.select :ymm_make_id %>

  <!-- skipping models and colors here for the sake of brevity -->

  <%= f.submit "Create" %>
<% end %>

对于年份搜索字段,我使用text_field_tag 而不是f.text_field,因为我不希望在提交整个表单时将搜索字段值作为汽车的一部分提交。我暂时将下拉字段留空 - 我们将通过 Javascript 和 JSON 填充它。

对于品牌列表,我将创建一个返回 JSON 的资源控制器:

class YmmMakesController < ApplicationController
  respond_to :json

  def index
    @makes = YmmMake.makes(params[:year])

    respond_with @makes
  end
end

不要忘记此控制器的 route.rb 条目,例如

namespace :admin do
  resources :ymm_makes, only: :index
end

我们将在 JavaScript 中从 JSON 中创建 &lt;select&gt; 选项:

$("input[name=year_search]").change(function () {

    // send a GET request to /admin/ymm_makes with the 'year' parameter
    // set to the value of the year_search text field
    $.getJSON( "/admin/ymm_makes", {year: $(this).val()}, function(data) {
      var options_html = [];

      // iterate over the JSON that we received back; each entry is one 'ymm_make'
      // in JSON form
      $.each( data, function( index, make ) {
        // make a new <option> tag for each make and push it into the options_html array
        // I assume here that YmmMake has an attribute called 'name' you want to display
        options_html.push( "<option value='" + make.id + "'>" + make.name + "</option>" );
      });

      // put all our generated <options> tags into the <select> tag
      $('select#car_ymm_make_id').html( options_html.join('') );
    });

});

所有这些都准备就绪后,您应该有一个工作表单来创建与 YmmMake 模型对象关联的新车。

对您现有代码的一些观察:

嵌套表单

<%= simple_form_for [:admin, @car] do |f| %>
  <%= render partial: "makes", locals: {form: 'new_admin_car', car_id: car_id} %>
<% end %>

如果我没看错,您的“makes”部分还包含一个表单,因此您正在创建一个嵌套在&lt;form&gt; 中的&lt;form&gt;。 HTMLdoes not allow that.

错误的结束标签顺序

      <div class="col-xs-6 col-sm-3">
        <br/>
      <input type="submit" form="new_admin_car" value="Create Car" class="btn btn-default btn btn-primary">
  <% end %>
  </div>

结束符&lt;/div&gt; 必须在&lt;% end %&gt; 之前。如果您构建无效的 HTML,您将面临奇怪的视觉行为和 Javascript 错误的风险。

冗余参数

<%= simple_form_for [:admin, @car],
                      html: {id: 'new_admin_car', class: 'form-vertical', method: post},
                      do |f| %>

“id”和“method”应该是默认值。省略它们使代码更易于阅读。

通过 input_html 的 &lt;input&gt; 属性无效

<%= f.input(:stock_number, {input_html: {form: 'new_admin_car', car: @car, value: nil}, autocomplete: :off, placeholder: 'Stock number?'}) %>

我对simple_form不熟悉,但是据我所见,input_html是用来给输入元素添加属性的。因此,像上面这样的行会产生一个带有无效car 属性的文本输入。删除嵌套表单后,form 属性就不再是必需的了。

HTTP 方法错误

$.post('/admin/cars/make_list/'

您通过 POST AJAX 请求加载您的品牌。对于只返回数据但不改变任何内容的请求,GET 通常更合适。

【讨论】:

  • 好的,看起来很有趣,我正在尝试实现它。但是,我的应用程序基于 Simple Form。关于在其中实施的建议?
  • 我建议从上面概述的内置表单助手开始。一旦你有了一个工作基线,你就可以重构它以使用简单表单,直到它再次工作。这两种方法并不相互排斥,可以并行使用。
  • 在实施此建议时存在几个问题,但最终,它比旧代码有了巨大的改进。谢谢。
  • @R_G 感谢您的反馈!如果您有一两分钟的空闲时间,我很想听听您遇到的问题。
  • 这个概念和设计成功了。实施过程中涉及很多细节。年份必须是一个系列。馆藏需要提示。像这样的东西。仍在努力使默认值起作用。请查看此网址并告诉我您是否可以回答?谢谢:stackoverflow.com/questions/25951405/…
【解决方案2】:

在您的表单中,您可以指定提交按钮的路径:

<%= simple_form_for [:admin, @car], url: create_action_path

...

【讨论】:

  • 没有改变任何东西。谢谢。
【解决方案3】:

我觉得你的答案很简单,用create而不是update,只需清除模型的id即可。
示例:@car.id = nil

【讨论】:

    猜你喜欢
    • 2012-05-27
    • 1970-01-01
    • 1970-01-01
    • 2013-03-23
    • 1970-01-01
    • 2018-02-19
    • 1970-01-01
    • 2012-02-22
    • 2023-03-10
    相关资源
    最近更新 更多