Rails中类实例创建及Test类关联初始化异常问题求助

嘿，我帮你梳理下代码里的问题，这两个关键点导致了self.practice_questions始终为空：

1. 未给PracticeQuestion绑定当前Test实例 #

在练习测试的分支里，你把选中的练习题放进了array，但完全没把这些题和当前的Test关联起来。Rails的关联依赖外键绑定，你只调用了question.use_practice_question，却没设置question.test = self（如果是has_many through关联，就没创建中间表记录），这些题根本不属于当前Test，自然不会出现在practice_questions集合里。

2. 错误地用普通数组赋值关联集合 #

Rails里的self.practice_questions是ActiveRecord::Associations::CollectionProxy对象，不是普通的Ruby数组。你用self.practice_questions ||= array这种方式直接赋值，不会触发Rails的关联持久化逻辑，等于白忙活一场。得用关联集合自带的concat或者<<方法来添加元素。

修复后的完整代码 #

def get_questions
  if self.for_practice
    # 取前5道候选练习题，用limit更符合ActiveRecord规范
    selected_questions = PracticeQuestion.sort_for_selection.limit(5)
    selected_questions.each do |question|
      # 关键：把当前Test绑定到练习题上（如果是has_many through，要操作中间表）
      question.test = self
      question.use_practice_question
    end
    # 用concat把题添加到关联集合，自动处理持久化
    self.practice_questions.concat(selected_questions)
  else
    # 考试测试分支：如果还没关联考题，补充5道未使用的
    if self.exam_questions.empty?
      selected_questions = ExamQuestion.unused.limit(5)
      selected_questions.each do |question|
        puts "grabbing question #{question.title}"
        question.test = self
        question.use_question
      end
      self.exam_questions.concat(selected_questions)
    end
  end

  # 验证输出
  puts "Practice questions count: #{self.practice_questions.length}"
  self.practice_questions.each { |q| puts q.title }
  self.save # 其实concat和赋值外键已经会触发保存，这里可以根据情况去掉
end

针对has_many through关联的额外说明 #

如果你的Test和PracticeQuestion是通过中间表（比如TestPracticeQuestion）关联的（也就是has_many :practice_questions, through: :test_practice_questions），那上面的question.test = self就不适用了，得改成创建中间表记录：

selected_questions.each do |question|
  TestPracticeQuestion.create!(test: self, practice_question: question)
  question.use_practice_question
end

这样才能正确建立has_many through的关联，之后self.practice_questions就能正确加载到这些题了。

小优化建议 #

• 用limit(5)替代[0...5]，更符合ActiveRecord的查询规范，还能避免加载多余数据
• 把关联创建的逻辑包在事务里，防止部分失败导致数据不一致：

  ActiveRecord::Base.transaction do
    selected_questions.each do |question|
      question.test = self
      question.use_practice_question
    end
    self.practice_questions.concat(selected_questions)
  end
  

内容的提问来源于stack exchange，提问作者Louis Raymond