可以急切加载与nested_attributes的关联吗？

Question

简而言之，我遇到了可怕的 2(n) 查询问题。 如果 n = 数据库中的技能数量，那么我的characters#edit 表单将需要 2(n) 次查询来加载页面。每个技能都会选择一次 PlayerSkill（连接表），并且每个技能都会查找一次技能。

这是一些我认为与情况相关的代码。本质上，这个过程中涉及的模型、视图和控制器，更少的模型验证和更少的我不关心的操作。

控制器：

  # GET /characters/1/edit
  def edit
    @character = Character.find(params[:id], :include => {:player_skills => :skill})
    stub_player_skills
  end

  private
    def stub_player_skills
      @skills = Skill.find(:all)
      @skills.each do |skill|
        if (skill.player_skills.empty?)
          ps = @character.player_skills.build(:skill_id => skill.id, :name => skill.name)
        end
      end
    end

模型：

class Character < ActiveRecord::Base
  belongs_to :user
  belongs_to :campaign
  has_many :sheets, :dependent => :destroy
  has_many :tokens, :dependent => :destroy

  has_many :player_skills, :dependent => :destroy
  has_many :skills, :through => :player_skills
  accepts_nested_attributes_for :player_skills, :allow_destroy => true
end

有问题的视图（HAML）：

%h1
  Editing Character

- form_for @character do |f|
  = f.error_messages
  %p
    = f.label :name
    %br
    = f.text_field :name
  %p
    = f.label :race
    %br
    = f.text_field :race
  %p
    = f.label :char_class
    %br
    = f.text_field :char_class
  %p
    -f.fields_for :player_skills do |ps|
      =ps.object.skill.name
      =ps.text_field :level
      =ps.hidden_field :skill_id
      -unless ps.object.new_record?
        =ps.check_box '_destroy'
        =ps.label '_destroy', 'Remove'
      %br
  %p
    = f.submit

我对这种情况的理解是，急切加载的存在是为了在（大致）单个额外查询中获取关联。

我需要在两个区域正确应用急切加载，但我只是不知道如何做到这一点。

在stub_player_skills方法中，它需要创建一个PlayerSkill对象假设角色还没有。 它可以从这里的预先加载中受益，因为它循环遍历数据库中的每项技能。这就是第一个“n 查询”的来源。

然后在视图上， fields_for 循环遍历我们已经积累的所有 PlayerSkills，因为这里无法立即加载，当我调用 =ps.object.skill.name 打印出技能名称时，它会进行技能查找，它引入了第二组“n 查询”。

我主要关心的是视图层，我找不到任何文档（Rails API 或其他）来说明如果您使用 fields_for 生成嵌套表单，如何立即加载关联。

感谢您的所有回复:) 〜罗比

Answer 1

你能试试这个并看看它是否有效吗？

您可以保持模型不变。

你的控制器看起来像这样

def edit
  # Get all the skill objects once only
  skills = Skill.find(:all)

  # Used to extract Skill#name
  skills_hash = {}
  skills.map { |s| skills_hash[s.id] = s.name }

  # Create an array of the skill-ids
  skill_ids = skills.map { |s| s.id }

  @character = Character.find(params[:id])

  # Determine the character's missing skills
  skill_ids -= @character.player_skill_ids

  # Build all of the missing skills
  skill_ids.each do |id|
    @character.player_skills.build(:skill_id => id, :name => skills_hash[id])
  end
end

Answer 2

如果有人对这个问题的“最终”解决方案感兴趣：

我已经诉诸于存储技能名称数组，并通过计数器在视图中引用它，如下所示：

  %p
    - index = 0
    -f.fields_for :player_skills do |ps|
      =@skill_arr[index]
      =ps.text_field :level
      =ps.hidden_field :skill_id
      -unless ps.object.new_record?
        =ps.check_box '_destroy'
        =ps.label '_destroy', 'Remove'
      - index += 1
      %br

在控制器中，我几乎移动了Stub_player_skills 方法所属的所有逻辑，并从 Coderama 的书中取出一页，我想出了这个：

  private
    def stub_player_skills
      @skills = Skill.find(:all)
      @skills.each do |skill|
        skill_exists = @character.player_skills.select do |i|
          i.skill_id == skill.id
        end
        if skill_exists.empty?
          ps = @character.player_skills.build(:skill_id => skill.id, :name => skill.name)
        end
      end

      @skill_arr = @character.player_skills.map do |el|
        el.name.nil? ? el.skill.name : el.name
      end
    end

在模型层中，我只需 :include =>; has_many :through 关系上的 :skill 可以消除更多查询。