ruby-on-rails - 我怎么能重构这个方法？

Question

给定以下辅助方法。

def link_tags(collection)
  tags = collection.split(',')

  tags.map.each do |tag|
    if tag == tags.last
      content_tag(:a, tag, href: tags_filter_post_path(tag) )
    else
      content_tag(:a, tag, href: tags_filter_post_path(tag) ) + ', '
    end        
  end.reduce(:<<)
end

我怎么能对此做一点重构呢？

编辑：建议重构后的最终代码。

def link_tags(collection)  
  collection.split(',').collect do |tag| 
    link = ""
    link += link_to tag, tags_filter_post_path(tag)
  end.join(', ').html_safe
end

Answer 1

def link_tags(collection)
  collection.split(',').map do |tag| 
    link_to tag, tag_filter_post_path(tag)
  end.join(', ')
end

Answer 2

使用 ajoin而不是专门处理最后一个元素（除非元素是唯一的，否则这也不会像你这样做的那样工作），然后在之后连接。调用 toeach也是多余的。

def link_tags(collection)
  tags = collection.split(',')
  tags.map do |tag|
    content_tag(:a, tag, href: tags_filter_post_path(tag))
  end.join(', ')
end

Answer 3

对于代码重构http://refactormycode.com是一个不错的选择

这是 Ruby 部分：

http://refactormycode.com/codes/recent/ruby

Answer 4

试试这个（link_to按照xnm的建议）：

def link_tags(collection)
  collection.split(',').each {|tag| link_to tag, tag_filter_post_path(tag)}.join(', ')
end