本地跳转错误find_each上没有给出(yield)错误的块

时间:2012-08-15 02:52:21

标签: ruby-on-rails-3

我正在尝试逐步查看使用find_each检索的记录列表。

我在这个stack overflow post的答案上构建了我的控制器代码,但我仍然得到了“No Block Given(Yield)”错误。

我刚刚开始使用Ruby和Rails,我还没有找到一个完整的解释(很多基本的例子)块和产量,它给了我我需要的东西。

我的代码如下所示:

def select_save
    @class = params[:class]
    @student_id = params[:id]
    @class.each do |id|
      old_subject = Subject.find(id)
      new_subject = old_subject.dup
      new_subject.student_id = @student_id
      new_subject.save
      Assignment.find_each.where(:subject_id => id) do |assignments|
        assignments.each do |a|
          new_assignment = a.dup
          new_assignment.subject_id = new_subject.id
          new_assignment.save
        end
      end
    end

    respond_to do |format|
      format.html { redirect_to @student, :notice => 'Subject and assignments created.' }
    end
  end

并且错误指向find_each的行。

我知道我需要一个块才能屈服,但在这个特殊情况下看起来究竟是什么让我失望。

感谢您的任何建议。

1 个答案:

答案 0 :(得分:5)

您正在将一个块传递给where,并且不会阻止find_each。你不能这样做。你需要在这一行上反转find_eachwhere,顺序很重要,因为块被传递给最后一个被调用的方法:

Assignment.find_each.where(:subject_id => id) do |assignments|

应该是:

Assignment.where(:subject_id => id).find_each do |assignments|

您的下一个问题是您正在尝试迭代assignments,这是一个分配。 find_each已经为你做了迭代,一次将一个赋值传递给块。该块应为:

Assignment.where(:subject_id => id).find_each do |assignment|
  new_assignment = assignment.dup
  new_assignment.subject_id = new_subject.id
  new_assignment.save
end

我将假设你的主题有很多作业,因为你的subject_id课程中有Assignment。如果是这种情况,那么编写循环的最终和最正确的方式将是:

old_subject.assignments.each do |assignment|
  new_assignment = assignment.dup
  new_assignment.subject_id = new_subject.id
  new_assignment.save
end