2016-06-19 31 views
1

我试图挽救ActiveRecords数组在我的控制器,但这个错误是显示出来:保存ActiveRecords对象的数组

未定义的方法“保存”为#<Array:...>

我有模型的这种方法:

def self.import(file) 
    reservations = [] 
    CSV.foreach(file, headers: true) do |row| 
    room_id = Classroom.where(code: row[0]).pluck(:id).first 
    row[0] = room_id 
    reservations << Reservation.new(number_of_guests: row[1], check_in_date: row[2], check_out_date: row[3], room_id: row[0]) 
    end 
    reservations 
end 

而且我有这个以下控制器:

def create_import 
    @reservations = Reservation.import(params[:file].path) 
    respond_to do |format| 
    if @reservations.save 
     format.html { redirect_to @reservation, notice: 'Reservations was successfully created.' } 
     format.json { render :show, status: :created, location: @reservation } 
    else 
     format.html { render :import } 
     format.json { render json: @reservations.errors, status: :unprocessable_entity } 
    end 
    end 
end 

我该如何做这个保存方法?我想在我看来显示错误的报告。

+1

如果'Classroom.where(code:row [0])。pluck(:id)'是'Classroom.where(code:row [0])。pluck(:id).first' ???第一个会给你包装一个数组的'room_id'。 – br3nt

+1

@ br3nt是的,修复。谢谢 – gumaro

回答

2

那么,你应该叫save方法对他们每个人:

@reservations.each(&:save) 

或者安装一些第三方的宝石,它提供了多种insert功能。您还应该重写您的if声明,因为上面的表达式将始终返回集合本身,因此评估为true

+0

因此,如果我使用这种方法,我怎样才能将所有错误返回到我的视图? – gumaro

+0

@gumaro:您可以使用'reject'或'select'而不是'each'来收集失败或成功保存的实例。 – potashin

+0

你能告诉我代码是怎么样的吗? – gumaro