简单:需要帮助重构这种看似笨拙的方法

时间:2012-12-12 16:27:36

标签: ruby ruby-on-rails-3 refactoring

如何清理它?它看起来很尴尬而且太长了。

def report_total(feed_event, advisor)
  count = 0
  advisor.activity_feed_events.each do |lead|
    if lead == SignupFeedEvent
      count += 1
    else
      if lead.is_a?(feed_event) 
        if lead.event_date > (Time.now - 7.days)
          count += 1
        end
      end
    end
  end
  return count
end

2 个答案:

答案 0 :(得分:1)

也许你可以使用:

def report_total(feed_event, advisor)
  advisor.activity_feed_events.count do |lead|
     lead == SignupFeedEvent ||
         (lead.is_a?(feed_event) && lead.event_date > (Time.now - 7.days))
  end
end

相同,少用代码。

答案 1 :(得分:0)

阅读一本好书,例如Refactoring Ruby edition。

良好的OO实践建议不检查类相等或is_a?,而是检查对象的功能,例如使用respond_to?。

if lead.respond_to?(:event_date) ...