Noob问题:Ruby on Rails中的类和实例方法

时间:2011-08-30 04:06:05

标签: ruby-on-rails ruby ruby-on-rails-3

我在comments_controller.rb中有一个名为update_status的函数:

def update_status
  @comment.relative_value = @comment.points_up - @comment.points_down
  randomize!
end

@comment = Comment.find(params[:id])

由于我设置网站的方式,我希望能够为任何评论c调用c.update_status。例如,在我的posts_controller中,我希望能够这样做:

def show
  @posts = Post.order('trending_value DESC').page(params[:page]).per(5)
  @post = Post.find(params[:id])
  @comments = @post.comments

  @comments.each do |c| #TODO: FIX
    c.update_status
    c.save
  end
end

我如何让它工作?我一直在为# < Comment >获取未定义的方法错误。我必须做def self.update_status吗?这似乎也没有用。

1 个答案:

答案 0 :(得分:6)

您在控制器中使用注释实例上的成员函数混淆了辅助函数update_status()。您可以通过将注释作为参数传递给辅助函数来使控制器代码工作:

def update_status(comment)
  comment.relative_value = comment.points_up - comment.points_down
  comment.save
  randomize!
end

def show
  @posts = Post.order('trending_value DESC').page(params[:page]).per(5)
  @post = Post.find(params[:id])
  @comments = @post.comments

  @comments.each do {|c| update_status(c) }
end

您还可以在Comment类本身上将其添加为成员函数,如下所示:

class Comment < ActiveRecord::Base
  def update_status
    update_attributes(:relative_value => points_up - points_down)
    randomize!
  end
end

def show
  @posts = Post.order('trending_value DESC').page(params[:page]).per(5)
  @post = Post.find(params[:id])
  @comments = @post.comments

  @comments.each do {|c| c.update_status }
end