如何清理它?它看起来很尴尬而且太长了。
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
答案 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) ...