删除/重构，以避免N + 1

Question

我运行到哪里我在与下面的行大规模的N + 1个问题一个问题：

<% @reports.each do |report| %> 
    <% (@first_week..@current_week).each do |week_number| %> 
    <% if report.user.answers.where(week_number: @current_week).exists? %> 
     <%= image_tag report.user.avatar.url(:thumb), size: '30', class: 'img-circle media-object' %> 
     <%= link_to "#{report.user.first_name + ' ' + report.user.last_name}", weekly_report_path(report.user.id, week_number) %> 
     <%= report.user.due_date %> 
     <%= week_number %> 
    <% end %> 
    <% end %> 
<% end %> 

当你看到有一吨协会相互连接，我试图找到一种方法来简化这一点。

控制器代码：

def index 
    @reports  = current_user.active_managements 
    @current_week = Time.zone.now.strftime('%V').to_i 
    @first_week = current_user.created_at.strftime('%V').to_i 
    end 

如何避免这里的N + 1度的问题，是代码很好，只是需要有预先加载，或者我应该重新写 - 如果是这样，任何建议？

子弹错误，我得到的是：

Reviewer => [:user] 
    Add to your finder: :includes => [:user] 
    index:15 which is: 
    <% if report.user.answers.where(week_number: @current_week).exists? %> 



    User => [:company] 
    Add to your finder: :includes => [:company] 
    navigation_links:21 which is: 
<li><%= link_to "Settings", edit_company_path(current_user.company) %></li> 

为线

Answer 1

在有限的信息，我将它采取快速刺：

在你的控制器，你需要渴望加载用户及其答案：

@reports = current_user.active_managements.includes(user: [:answers, :avatar]) 

在视图中，您需要更改y我们的'where'通过渴望加载的对象进行检测和循环。注意：这没有经过测试，我只是在刺探它。我希望它能带给你正确的道路：

<% @reports.each do |report| %> 
    <% (@first_week..@current_week).each do |week_number| %> 
    <% if report.user.answers.detect { |a| a.week_number == @current_week.to_s}.present? %> 
     <%= image_tag report.user.avatar.url(:thumb), size: '30', class: 'img-circle media-object' %> 
     <%= link_to "#{report.user.first_name + ' ' + report.user.last_name}", weekly_report_path(report.user.id, week_number) %> 
     <%= report.user.due_date %> 
     <%= week_number %> 
    <% end %> 
    <% end %> 
<% end %>