這個Rails代碼如何被重構？

Question

只是出於好奇：

這個（相當醜陋的）Rails代碼怎麼能美化/重構：

def section_link(name, path)  
    link = link_to(name, path) 
    if name != controller.controller_name.titlecase 
    link 
    else 
    link_to(name, path, :class => 'current') 
    end 
end 

Answer 1

def section_link(name, path) 
    options = {} 
    options[:class] = 'current' if name == controller_name.titlecase 
    link_to name, path, options 
end 

Answer 2

def section_link(name, path)  
    if name != controller_name.titlecase 
    link_to(name, path) 
    else 
    link_to(name, path, :class => 'current') 
    end 
end 

或者這樣

def section_link(name, path) 
    link_to(name, path, :class => "#{"current" if name == controller_name.titlecase }") 
end 

不要的東西覺得它真的需要重構壽，如果它的工作...

Answer 3

我會寫：

def section_link(name, path) 
    is_current = (name == controller.controller_name.titlecase) 
    link_to(name, path, :class => ('current' if is_current)) 
end 

理由：1）的可變is_current使得代碼稍微更聲明性的。 2）link_to假設nil表示空的類（我們在這裏想要的）。

Answer 4

你可以做這樣的事情：

def section_link(name, path) 
    link_to(name, path, class: name == controller.controller_name.titlecase ? "current" : nil) 
end 

但這種情況正在變得有點難以閱讀。我會分裂類判定爲另一種方法：

def section_link(name, path) 
    link_to(name, path, class: class_for(name)) 
end 

def class_for(name) 
    name == controller.controller_name.titlecase ? "current" : nil 
end 

Answer 5

def section_link(name, path)  
    link_to(name, path, 
    *({class: "current"} if name == controller.controller_name.titlecase)) 
end