추가 클래스를 위임하거나 인스턴스화 하시겠습니까?

Question

계정 클래스와 AccountReport 클래스가 있다고 가정 해 보겠습니다. accounts # show 계정에 대한 보고서를 보여주고 싶습니다. Account와 AccountReport는 여러 가지 public 메소드를 가지고 있습니다. 뒤에 오는 기술의 어느 기술이 더 낫습니까?

1) 계정 및 AccountReport를 인스턴스화하고 계정 데이터로 AccountReport를 초기화합니다.

class AccountsController < ActionController::Base 
    def show 
    @account = current_user.account 
    @account_report = AccountReport.new(@account.orders) 

    respond_with(@account) 
    end 

    # ... 
end 

2) 계정의 인스턴스 AccountReport 위임 방법은 옵션이 나에게 깨끗하고 접근하는 것 같지만 방법이 많은 계정을로드하는 것은이 같은 느낌

class Account < ActiveRecord::Base 
    attr_reader :account_report 

    delegate :method_a, :method_b, :method_c, :method_d, :to => :account_report 

    after_initialize :setup_account_report 

    def setup_account_report 
    @account_report = AccountReport.new(orders) 
    end 

    # ... 
end 

호출 인스턴스화 할 수 있도록 허용 하나님 계급.

Answer 1

음, 두 가지 옵션을 혼합해야한다고 생각합니다.

첫 번째 보고서는 보고서 만 사용하는 경우 유용합니다. 두 번째 계정은 모든 시간 보고서를 계정에 사용하는 경우 유용합니다.

언제든지 보고서가 인스턴스화되고 성능이 저하 될 수 있습니다.

당신은 아마 이런 식으로 뭔가를 시도해야합니다 :

class Account < ActiveRecord::Base 
    # ... 

    @report = nil 
    def report 
    if @report.nil? 
     @report = AccountReport.new(self.orders) 
    end 
    end 

    # ... 
end 

이 솔루션의 좋은 점은 필요한 경우 해당 보고서는로드,하지만 때마다로드되지 않습니다. 이 솔루션의 나쁜 점은 일부 주문을 추가하면 보고서가 최신 상태가 아닐 수 있다는 것입니다.

UPDATE : 이를 개선하기 위해, 당신이 낮은 커플 링을 유지하기 때문에 나는 첫 번째 옵션을 좋아하는이 하나

if @report.nil || self.created_at_changed? 

Answer 2

과 조건을 대체 할 수 있습니다. 두 번째 옵션은 Account와 AccountReport를 아마도 불필요한 방식으로 연결합니다. 다른 유형의 보고서를받을 때마다 어떤 일이 발생합니까? 계정에서 많은 것들을 변경해야 할 것입니다. 계정은 겉으로보기에는 관계가 없기 때문에 슬픈 일입니다.

서비스 개체에서이 두 가지를 결합하여 컨트롤러에서 논리/자세한 정보를 낮게 유지하고 뷰에 전달할 수 있습니다.

AccountReporting.new(current_user.account) 

이 메이크업 감각을합니까 :는 AccountReporting 서비스는이 두 가지 클래스를 결합 뒤에 논리, 예컨대이 :

class AccountReporting 
    def initialize(account) 
     @account = account 
    end 
    def report 
     AccountReport.new(account.orders) 
    end 
end 

그런 다음 컨트롤러에서 사용하는 처리 할 수 ​​있습니까?