Каков наилучший способ запроса?

Question

У меня две модели: employee и in_outs. связь между ними: employeehas manyin_outs и in_outbelongs toemployee .i хочу показать attendance всего employees. Для этого мой current logic - это.

логика в действии контроллера:

def view_all_employee_attendance 
    employees = Employee.all 
    @all_employess_punch_in_outs = [] 
    employees.each do |employee| 
    @all_employess_punch_in_outs << employee.in_outs.where('date >= ? and date <= ?', Date.today.at_beginning_of_month, Date.today) 
    end 
end 

и ввиду:

 <tbody> 
     <% @all_employess_punch_in_outs.each do |punch_record| %> 
      <tr> 
      <td><%= punch_record.employee.name %></td> 
      <td><%= punch_record.check_in %></td> 
      <td><%= punch_record.check_out %></td> 
      </tr> 
     <% end %> 
     </tbody> 

в этом случае в моем viewqueries снова выполняются. как сделать этот запрос optimise в view и в action с помощью eagerloading?

Answer 1

измените строку запроса на это. Это будет загружать все ваши in_outs с предложением where, чтобы вы просто запускали один запрос. Это сделает так, чтобы ваше представление было оптимизировано, а также не нужно запускать N+1 запросов каждый раз при каждом запросе in_out

employees = Employee.joins(:in_outs).all.where('in_outs.date >= ? and in_outs.date <= ?', Date.today.at_beginning_of_month, Date.today).preload(:in_outs) 

Answer 2

Ваш запрос снова вызывается по следующей строке: punch_record.employee.name.

Для нетерпеливого нагрузки employee вам нужно изменить ваш запрос будет выглядеть примерно так:

def view_all_employee_attendence 
    @employee_in_outs = Employee.all.includes(:in_outs).where('in_outs.date >= ? and in_outs.date <= ?', Date.today.at_beginning_of_month, Date.today) 
end 

includesdocumentation.