-1

どうすればこれをクリーンアップできますか? 読みにくいし、長すぎる。

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
4

2 に答える 2

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

同じことを行いますが、コードははるかに少なくなります。

于 2012-12-12T16:32:03.313 に答える
0

たとえば、Rubyエディションのリファクタリングなどの優れた本を読んでください。

優れたOOプラクティスでは、クラスの同等性やis_a？をチェックするのではなく、たとえばrespond_to？を使用して、オブジェクトの機能をチェックすることをお勧めします。

if lead.respond_to?(:event_date) ...
于 2012-12-12T19:45:28.360 に答える