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

除非语句使块更高效,否则如何重构这些语句?

ruby-on-rails

Shpigford · 技术社区 · 15 年前

我正在尝试根据URL中是否存在“order”参数向以下一组链接添加“current”类。(例如,example.com/photos?命令=视图。我需要一些帮助来重构这个,这样它就不会那么粗了。

<% unless params[:order] != 'created_at' %>
    <%= link_to "Recently Posted", photos_path, :class => 'current' %> 
<% else %>
    <%= link_to "Recently Posted", photos_path %> 
<% end %>
<span class="pipe">|</span> 
<% unless params[:order] != 'comments' %>
    <%= link_to "Most Commented", photos_path + "?order=comments", :class => 'current' %> 
<% else %>
    <%= link_to "Most Commented", photos_path + "?order=comments" %> 
<% end %>
<span class="pipe">|</span> 
<% unless params[:order] != 'views' %>
    <%= link_to "Most Viewed", photos_path + "?order=views", :class => 'current' %>
<% else %>
    <%= link_to "Most Viewed", photos_path + "?order=views" %>
<% end %>

2 回复 | 直到 15 年前

tig Charlie Martin 15 年前

例如,您可以使用助手:

<%= link_to_photos_with_order 'created_at', "Recently Posted" %>
<span class="pipe">|</span>
<%= link_to_photos_with_order 'comments', "Most Commented" %>
<span class="pipe">|</span>
<%= link_to_photos_with_order 'views', "Most Viewed" %>

def link_to_photos_with_order order_by, title
  url = photos_path(:order => (order_by != 'created_at' ? order_by : nil))
  unless params[:order] != order_by
    link_to title, url, :class => 'current'
  else
    link_to title, url
  end
end

另一种方法是将hash与order_by=>title一起使用,但它更丑

Avdi 15 年前

将逻辑提取到助手中:

def link_to_ordering(params, ordering, title)
  css_class = params[:order] == ordering ? 'current' : ''
  link_to(title, photos_path(:order => ordering), :class => css_class)
end

然后打电话,例如:

<%= link_to_ordering(params, "Most Commented", "comments") %>

在你看来。

推荐文章

aarona · Rails迁移批量更新change_column_default

4 月前

cluster1 · 采取独立的新行动的好处是什么?

6 月前

Robert · 使用JSON或哈希时,将NULL替换为NIL

6 月前

cluster1 · Rails form_with-helper:什么是:作用域?

11 月前

stevec Zxeenu · 如何在rails控制台中禁用寻呼机,以便将整个结果打印到控制台?

1 年前

Fred Willmore · Rails控制器不呈现任何模板

1 年前

Diogo Amaral · 实现API请求的正确方式

1 年前

Meknassih · 在控制器方法中分配给模型没有任何作用

1 年前

Michael Ding · Rails上的默认会话到期问题

1 年前

Flávio · 基于另外两个生成数组

1 年前