嵌套枚举是不好的做法吗？

Question

在我看来，我正在生成许多餐厅菜单，这些菜单分为菜单类别，并且在这些类别中是菜肴。现在我的模板设置如下：

<% @menus.each do |menu| %>
  <h2><%= menu.name %></h2>
  ...    
  <% menu.categories.each do |category| %>
    <h3><%= category.name %></h3>
    ...
    <% category.dishes.each do |dish| %>
     <p><%= dish.name %></p>
      ...
    <% end %>
  <% end %>
<% end %>

我想知道以这种方式嵌套多个可枚举对象的最佳做法是什么？除了将此代码插入部分代码之外，还有没有更好的方法来完成此操作而不会弄乱视图。照原样好吗？谢谢。

Answer 1

显然，您的模型设置方式（Menu has_many categories、Category has_many dishes 等），如果您要展示所有这些信息都在同一个视图中，您别无他法，只能遍历它们并在视图中显示所需的数据，这很好，恕我直言。我看不出有什么问题。

只有一个提示，在循环遍历关联模型的数据时，您只需要注意始终存在关联模型属性的数据，否则您可能会收到 undefined method 'method_name' for nil:NilClass 错误。为避免该问题，您还可以使用try。例如

menu.try(:name)
category.try(:name)
dish.try(:name)

等等

这样，如果某些或任何一个关联模型没有数据存在（即nil），您的视图将不会爆炸。