一部のビュー コードをヘルパー メソッドにリファクタリングしていますが、さらに単純化できるかどうか疑問に思っています。
# In the view...
<%= navigation_link params[:controller], 'accounts', account_path %>
# ApplicationHelper
def navigation_link(controller_name, page_name, path_name)
if controller_name == page_name
content_tag :li, "#{page_name.capitalize.singularize}", :class => 'current'
else
content_tag :li, link_to("#{page_name.capitalize.singularize}", path_name)
end
end
ルート ヘルパー メソッド名 account_path と共に文字列 'accounts' を渡すのは冗長に思えます。
本当に 3 番目の引数 path_name を渡す必要がありますか?それとも、2 番目の引数として渡す文字列を使用して、navigation_link ヘルパーでそれを生成できますか?