横流しブローカーの除去(Remove Middle Man)

リファクタリングすべき時

クラスが単純な委譲をやり過ぎている。

リファクタリングした方がいい理由

委譲オブジェクトに追加するメンバが増えてきた際、サーバーに委譲メソッドを追加する負担を抑えるため。

◯ テストコード

# rspec

describe Person do
  let(:department){ Department.new('manager-name') }
  let(:person){ Person.new(department) }

  describe 'manager' do
    subject{ person.manager }
    it{ should == 'manager-name' }
  end
end

※参考資料
 リファクタリング:Rubyエディション