Rails中如何迭代Location类并生成页脚带链接导航？

我来帮你解决这个问题，其实你遇到的冗余输出是ERB语法的小坑，咱们一步步捋清楚：

首先说你最初代码的问题根源：
你用<%= Location.all.each do |location| %>会产生冗余输出，是因为ERB里的<%= %>会把表达式的返回值输出到页面上。而each方法执行完后会返回原集合（也就是Location.all的整个查询结果），所以除了生成的<li>元素，页面上还会额外输出类似#<ActiveRecord::Relation [...]>的字符串，这就是你看到的冗余内容。

最简单的修复方案 #

把<%= %>改成<% %>——因为我们只需要执行循环生成HTML，不需要输出each的返回值：

<ul class="footer-nav" id="foot">
  <% Location.all.each do |location| %>
    <li class="footer-list">
      <%= link_to location.name, location_path(location.name), class: "footer-location"%>
    </li>
  <% end %>
</ul>

这样就不会有冗余输出，同时能正确生成每个地点的链接。

关于辅助方法的优化思路 #

你想把逻辑封装到辅助方法里的想法完全没问题，只是原来的辅助方法只返回了名字字符串，没法生成链接。这里有两种常用的实现方式：

方式1：辅助方法返回数据集合，视图处理HTML结构

在app/helpers/application_helper.rb里定义辅助方法：

def footer_locations
  Location.all
end

然后在页脚视图里调用并迭代：

<ul class="footer-nav" id="foot">
  <% footer_locations.each do |location| %>
    <li class="footer-list">
      <%= link_to location.name, location_path(location.name), class: "footer-location"%>
    </li>
  <% end %>
</ul>

这种方式把数据逻辑和视图结构分开，更符合Rails的最佳实践。

方式2：辅助方法直接生成完整的HTML导航

如果想让辅助方法直接输出整个导航的HTML，可以这样写：

# app/helpers/application_helper.rb
def footer_location_nav
  content_tag(:ul, class: "footer-nav", id: "foot") do
    Location.all.map do |location|
      content_tag(:li, class: "footer-list") do
        link_to location.name, location_path(location.name), class: "footer-location"
      end
    end.join.html_safe
  end
end

然后在视图里直接调用：

<%= footer_location_nav %>

这里要注意join.html_safe，因为map返回的是HTML元素数组，join后需要标记为安全HTML，否则ERB会把它转义成纯文本。

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