ruby ​​- refactoring if else statement

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.