如何重构case..when红宝石

-1

def readable 
    uptime = (Time.now - self).to_i 
    case uptime 
    when 0 then 'just now' 
    when 1 then 'uptime second ago' 
    when 2..59 then uptime.to_s + ' seconds ago' 
    when 60..119 then 'uptime minute ago' # 120 = 2 minutes 
    when 120..3540 then (uptime/60).to_i.to_s + ' minutes ago' 
    when 3541..7100 then 'an hour ago' # 3600 = 1 hour 
    when 7101..82_800 then ((uptime + 99)/3600).to_i.to_s + ' hours ago' 
    when 82_801..172_000 then 'uptime day ago' # 86400 = 1 day 
    else ((uptime + 800)/86_400).to_i.to_s + ' days ago' 
    end 
    end

Linter谈到了以下错误，这怎么能解决？如何重构case..when红宝石

FeatureEnvy：时间＃可读指 '正常运行时间' 比自己更[https://github.com/troessner/reek/blob/master/docs/Feature-Envy.md]
TooManyStatements（也许它移动到另一个类？）：时间＃可读具有约10语句[https://github.com/troessner/reek/blob/master/docs/Too-Many-Statements.md]
分配分支条件可读性的大小太高。 [20.64/15]
可读性的圈复杂度太高。 [9/6]
方法有太多的行。 [12/10]

来源

2017-09-18 Igor Polyakov

https://codereview.stackexchange.com/是这类问题的适当位置。 – mudasobwa

我投票结束这个问题作为题外话，因为它是工作代码，而应该发布在[代码审查]（http://codereview.stackexchange.com） –

这是一种情况下，我暂时禁用rubocop规则b/c我同意这是一个非常简洁的实现。 – Joe

看看time_ago_in_words并随意使用它或它的代码。

关于代码指标 - 您的代码非常简单，您应该只提取uptime方法。

来源

2017-09-18 10:17:45

[实施]（https://github.com/rails/rails/blob/v5.1.4/actionview/lib/action_view/helpers/date_helper.rb#L93-L156）并不那么简单，是吗？ – Stefan

当然。没有机会写它更简单。我提到它只是想着不要重复Rails :) –

如何重构case..when红宝石

回答

相关问题