Ruby on rails RubyonRails-从一个数组创建多个记录
我有一张我的型号Ruby on rails RubyonRails-从一个数组创建多个记录,ruby-on-rails,arrays,database,activerecord,Ruby On Rails,Arrays,Database,Activerecord,我有一张我的型号@gig的表格。我试图一次创建多个具有相同属性的gig,除了date之外,这些属性应该从数组中选取 到目前为止,只保存数组中的最后一个日期,并且只保存一条记录 _form.html.erb <%= simple_form_for @gig , url: gigs_path do |form| %> <div class="create-title"> <%= form.input :title, la
@gig
的表格。我试图一次创建多个具有相同属性的gig
,除了date
之外,这些属性应该从数组中选取
到目前为止,只保存数组中的最后一个日期,并且只保存一条记录
_form.html.erb
<%= simple_form_for @gig , url: gigs_path do |form| %>
<div class="create-title">
<%= form.input :title, label: t('gig.title'), placeholder: t('placeholder.title') %>
</div>
bla... bla... bla....
<p><%= t('gig.date') %> </p>
<% if !mobile_device? %>
<%= form.input :date, as: :string, label: false, placeholder: t('placeholder.date'), multiple: true %>
<% else %>
bla... bla... bla....
<% end %>
在控制器中使用放置日期
,我可以看到我的日期正确分开
服务器显示,只要数组中有日期,就会调用ROLLBACK
,然后正确保存最后的gig
更新1: 我已经更改了允许创建多个记录的控制器,我只是担心如果记录无法保存,会缺少重定向或消息
def create
@gigdates = params[:gig][:date].split(';')
@gigdates.each do |date|
@gig = Gig.new(gig_params)
@gig.date = date
@genres = Genre.where(:id => params[:choose_genres])
@gig.genres << @genres
@gig.save
end
redirect_to @gig
end
def创建
@gigdates=params[:gig][:date]。拆分(“;”)
@gigdates.each do| date|
@gig=gig.新(gig_参数)
@gig.date=日期
@流派=流派。其中(:id=>params[:choose_genres])
@gig.genres对于@gig_dates
数组中的每个日期,您正在使用gig_参数创建无效的gig。这是无效的,因为gig_params
的date
键将是最初在表单中输入的字符串,例如“2017-01-3;2017-05-09;2017-08-01”
。这是一个无效的日期,因此所有的回滚
s–您对Gig.create的调用都在数据库级别失败
让我们更深入地了解正在发生的事情:
# EACH LOOP BEGINS
# for each gig date:
@gigdates.each do |date|
# gig_params[:date] will be the whole string that was entered in the
# form. this is an invalid date, so Gig.create will fail to save the
# gig to the database (= ROLLBACK).
# so you create an gig, but it can't be saved because its date parameter
# is invalid. you assign this invalid gig to the @gig instance
# variable. this variable will be overwritten each time, so only the
# last created gig is stored in @gig
@gig = Gig.create(gig_params)
# you assign that invalid gig a (valid) date (but you don't save it!)
@gig.date = date
end
# EACH LOOP ENDS
# Save @gig, the last created gig, using the valid date you assigned
# it at the end of the each loop. so now the save will work!
if @gig.save
redirect_to @gig
else
Rails.logger.info(@gig.errors.inspect)
render 'new'
end
end
编辑:
处理验证的一种方法是在保存任何新GIG之前检查所有GIG的有效性。要做到这一点,您可以将每个替换为一个映射,该映射将创建一个新GIG数组,然后在保存它们之前检查它们是否都有效:
def create
gigdates = params[:gig][:date].split(';')
gigs = gigdates.map do |date|
gig = Gig.new(gig_params.merge(date: date))
genres = Genre.where(:id => params[:choose_genres])
gig.genres << genres
end
if gigs.all?(&:valid?) && gigs.all?(&:save)
redirect_to gigs.first
else
flash[:notice] = "something went wrong"
@gig = Gig.new(gig_params)
render :new
end
end
def创建
gigdates=params[:gig][:date]。拆分(“;”)
gigs=gigdates.map do| date|
gig=gig.new(gig_参数合并(日期:日期))
流派=流派。其中(:id=>params[:choose_genres])
我已经用更新版本更新了我的问题。我现在可以保存多条记录,但失败时没有验证或flash消息。@RobHughes查看我的编辑,了解如何处理验证的一个相当简单的示例。这并不理想(例如,如果所有的GIG都有效,则会出现不一致的情况,但其中一些GIG由于任何原因仍然无法保存)。处理这个问题的一个更好的方法是简化创建操作,一次只生成一个gig,并让浏览器为每个gig生成一个单独的请求(例如,使用一点javascript发送ajax请求)。但这超出了这个问题的范围。
def create
gigdates = params[:gig][:date].split(';')
gigs = gigdates.map do |date|
gig = Gig.new(gig_params.merge(date: date))
genres = Genre.where(:id => params[:choose_genres])
gig.genres << genres
end
if gigs.all?(&:valid?) && gigs.all?(&:save)
redirect_to gigs.first
else
flash[:notice] = "something went wrong"
@gig = Gig.new(gig_params)
render :new
end
end