如何在 Rails 3 中制作自定义助手？

Question

我对 Rails 还很陌生，我对如何在 if...else 语句之外传递局部变量感到困惑。看起来在帮助文件中创建一个方法是执行此操作的常规方法，但我不确定如何执行此操作。

所以我想找到任务的作者。如果任务的作者不存在，我想使用其父教学大纲的作者（任务属于教学大纲）。然后我想打印出那个作者的用户名。当我只处理一个任务时，我能够做到这一点，例如：

//controller
@mission = Mission.first
if !@mission.author.blank?
  @author = @mission.author
else
  @author = @mission.syllabus.author
end

//view
<%= @author.username %> 

但我在处理 foreach 循环时不确定如何执行此操作：

//controller
@mission = Mission.all

//view 
<% @mission.each do |mission| %>
..(where do I put the logic of finding author? I can't put it in my controller anymore and it won't pass the @author variable outside the if else statement if I put the logic here in the view)..
<%= @author.username %>
<% end %>

我徒劳的尝试是创建一个助手：

def author_getter(mission_id)
  @mission = Mission.find(params[:mission_id])
  if !@mission.author.blank? 
    @author = @mission.author 
    return @author
  else
    @author = @mission.syllabus.author  
    return @author
  end 
end

并将以下内容放入循环中

<%= author_getter(mission) %>

但是，这不起作用。在 if...else 语句之外传递变量的最佳方法是什么？

Answer 1

除非我误解了您的问题，否则您可以在循环中执行此操作而无需使用 find。

//view 
<% @mission.each do |mission| %>
   <%= mission.author.username %>
<% end %>

请记住，您正在遍历所有任务对象。一旦你有了一个任务对象，你就可以像往常一样访问作者。

Answer 2

您的辅助方法有点混乱。助手不应该在params 中闲逛，他们应该只是用他们被调用的参数来做事。您传入了 mission_id 但未使用它，当参数名称指示而不是要求 ID 时，您还使用（显然）mission 对象调用它。此外，您无需在帮助程序中处理实例变量，只需简单的旧变量即可。

调整界面以请求任务对象，然后使用该对象：

def author_of(mission)
  mission.author.present?? mission.author : mission.syllabus.author
end

或者，既然mission.author应该是nil或者那里，你可以利用nil的错误：

def author_of(mission)
  mission.author || mission.syllabus.author
end

然后在你的 ERB 中：

<!-- Note that you should use the plural @missions for a collection, you'll have to fix your controller as well. -->
<% @missions.each do |mission| %>
    <%= author_of(mission).username %>
<% end %>

当然，一旦我们简化并更正了您的助手，您可能会认为它太小而不值得打扰；如果是这样，那么您可以放弃助手并在 ERB 中完成所有操作：

<% @mission.each do |mission| %>
    <%= (mission.author || mission.syllabus.author).username %>
<% end %>

但是，我认为您的这种逻辑是错误的：这应该在 Mission 本身内部，以便所有东西（其他模型、JSON 构建器......）都可以利用它。所以，这样的方法是有意义的：

class Mission
  def real_author
    author || syllabus.author
  end
end

那么你可以在你的 ERB 中这样说：

<% @missions.each do |mission| %>
    <%= mission.real_author.username %>
<% end %>