Rubocop: метод имеет слишком много строк

Question

Здравствуйте, я новичок в программировании ruby. Ran rubocop в моем проекте, и он говорит:

Метод имеет слишком много линий. [13/10] Защита refresh_status

Вот мои методы:

def refresh_status 
    lost = false 
    in_progress = false 
    won = false 
    @bets.each do |bet| 
     lost = true if bet.result == :lost 
     if bet.result == :canceled 
     @to_return /= bet.odd 
     won = true 
     end 
     in_progress = true if bet.result == :in_progress 
     won = true if bet.result == :won 
    end 
    def_result_after_refresh(lost, in_progress, won) 
    end 

    def def_result_after_refresh(lost, in_progress, won) 
    if lost 
     @result = :lost 
    elsif in_progress 
     @result = :in_progress 
    elsif won 
     @result = :won 
    end 
    end 

Не можешь найти способ, чтобы сделать этот метод короче, может быть, вы могли бы помочь?

Answer 1

Вы можете использовать некоторые методы Enumerable.

def refresh_status 
    @to_return /= @bets.select { |bet| bet.result == :canceled }.map(&:odd).reduce(1, :*) 

    results = @bets.map { |bet| bet.result == :cancelled ? :won : bet.result }.uniq 
    @result = case 
      when results.include?(:lost) then :lost 
      when results.include?(:in_progress) then :in_progress 
      when results.include?(:won) then :won 
      end 
end