ruby​​ - 重構if else語句

Question

我試過閱讀一些關於重構的教程，我正在努力解決條件問題。我不想使用三元運算符，但也許應該用一種方法提取它？或者有一個聰明的方式來使用地圖？

detail.stated = if value[:stated].blank? 
        nil 
       elsif value[:stated] == "Incomplete" 
        nil 
       elsif value[:is_ratio] == "true" 
        value[:stated] == "true" 
       else 
        apply_currency_increment_for_save(value[:stated]) 
       end 

Answer 1

，它可以製成大量清潔器由於早期return（和關鍵字參數）：

def stated?(stated:, is_ratio: nil, **) 
    return if stated.blank? || stated == "Incomplete" 
    return stated == "true" if is_ratio == "true" 
    apply_currency_increment_for_save(stated) 
end 

然後...

detail.stated = stated?(value) 

Answer 2

stated = value[:stated] 
detail.stated = case 
    when stated.blank? || stated == "Incomplete" 
    nil 
    when value[:is_ratio] == "true" 
    value[:stated] == "true" 
    else 
    apply_currency_increment_for_save stated 
end 

發生了什麼事：  時case而不表達式中使用，它成爲一個if ... elsif ... else ... fi.

您可以使用它的結果的文明等同，太，就像if...end。

Answer 3

移動代碼到apply_currency_increment_for_save 和做：

def apply_currency_increment_for_save(value) 
    return if value.nil? || value == "Incomplete" 
    return "true" if value == "true" 
    # rest of the code. Or move into another function if its too complex 
end       

的邏輯被封裝並且如果移動此邏輯到一個方法需要2行僅

Answer 4

我喜歡@喬丹的建議。但是，看起來呼叫並不完整 - 「is_ratio」參數也是從值中選擇的，但未提供。

只是爲了爭論，我會建議你可以更進一步，並提供類，它非常狹隘地專注於評估「規定」的價值。這可能看起來很極端，但它符合單一責任的概念（責任在於評估所陳述的「價值」 - 而「細節」對象可能專注於其他事物，而僅僅利用評估）。

它會是這個樣子：

class StatedEvaluator 
    attr_reader :value, :is_ratio 

    def initialize(value = {}) 
    @value = ActiveSupport::StringInquirer.new(value.fetch(:stated, '')) 
    @is_ratio = ActiveSupport::StringInquirer.new(value.fetch(:is_ratio, '')) 
    end 

    def stated 
    return nil if value.blank? || value.Incomplete? 
    return value.true? if is_ratio.true? 
    apply_currency_increment_for_save(value) 
    end 
end 

detail.stated = StatedEvaluator.new(value).stated 

注意，這使得使用Rails的StringInquirer class。