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。