Как я могу реорганизовать этот метод?

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 

Как я могу сделать небольшой рефакторинг на этом?

EDIT: окончательный код после предлагаемого рефакторинга.

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

Используйте join вместо отводом вокруг с обработкой последнего элемента специально (что также не будет работать так, как вы делаете это, если эти элементы не являются уникальными), а затем конкатенацию после. Вызов each также является избыточным.

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

Попробуйте (с link_to как предложил хпМ):

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

Answer 4

Для кода рефакторинга http://refactormycode.com является хорошим вариантом

Это раздел Ruby:

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