代码之家 › 专栏 › 技术社区 › JRL

如何使这个Ruby方法不那么难看(嵌套)

nested coding-style ruby ruby-on-rails

JRL · 技术社区 · 15 年前

我有一个助手方法为一些控制器创建导航链接。

  def gen_associations(controllers)
    content_for :leftnav do
      sorted_controllers = controllers.sort
      returning String.new do |content|
        content << content_tag(:h3, "Associations") <<
        content_tag(:ul, :class => "nav") do
          sorted_controllers.collect do |c|
            content_tag("li", :class => ("last" if c == sorted_controllers.last)) do
              link_to(c.humanize, eval("admin_#{c}_url"))
            end
          end
        end
      end
    end
  end

我不喜欢这种深嵌套结构,还有 << 其中一行的结尾。

如何重写它,使其不像那样嵌套(在较少的行中)并且没有长行(<80个字符)?

3 回复 | 直到 13 年前

Jerry Fernholz 15 年前

从内到外构建:

  def gen_associations(controllers)
    sorted_controllers = controllers.sort

    list_items = 
      sorted_controllers.collect do |c|
        content_tag("li", :class => ("last" if c == sorted_controllers.last)) do
          link_to(c.humanize, eval("admin_#{c}_url"))
        end
      end

    list = content_tag(:ul, list_items.join, :class => "nav")

    content_for :leftnav do
      content_tag(:h3, "Associations") << list
    end
  end

我可能会把 content_for 对视图或局部 gen_associations() 返回 list .

...
<% content_for :leftnav do %>
  <h3>Associations</h3>
  <%= gen_associations(@controllers) %>
<% end %>
...

tig Charlie Martin 15 年前

使用部分-把所有东西都放回关闭处,然后使用 render :partial => ...

giorgian 15 年前

考虑使用类似 markaby 而不是内容标签;它使您的助手更具可读性。也见此 railscast .

推荐文章

cng · 如何在嵌套数组中查找值,然后将值从布尔值更改为字符串?

2 年前

Tanu · 使用三元运算符时获得Lint无嵌套三元错误

2 年前

MagicMatt · 由于值(…)中的“SELECT*”,MySQL触发器的插入导致基数冲突

2 年前

SpaceBallz · 比较嵌套列表中的值

2 年前

Orsi · 选择嵌套列表的最大值和最小值。网

2 年前

Hendrik · 在尝试将数组分解为新变量以进行赋值时遇到问题

2 年前

SOS · 初学者对有关嵌套对象函数的JavaScript语法感到困惑

2 年前

Teodor Kostovski · OCaml中的嵌套函数及其参数

2 年前

ArushRaj · 在javascript中获取字符串数组中对象的所有叶节点属性的函数

2 年前

lingyau lee · 递归-对嵌套列表求和

2 年前