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

我如何重构这些除非语句以提高块的效率?

ruby-on-rails

Shpigford · 技术社区 · 16 年前

我正试图根据URL中是否有“订单”参数(即example.com/photos?order=views)向以下链接集添加一个“当前”类。我需要一些帮助来重构它,这样它就不会那么笨重了。

<% 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 回复 | 直到 16 年前

tig Charlie Martin 16 年前

例如,您可以使用助手:

<%= 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

另一种方法是使用order_by=>标题,但更丑

Avdi 16 年前

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

1 年前

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

1 年前

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

1 年前

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

1 年前

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

2 年前

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

2 年前

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

2 年前

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

2 年前

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

2 年前

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

2 年前