エクストリームエンジニア５
- 2. • リファクタリング • 設計改善 内容
- 3. リファクタリングとは • 外部から見たときの振る舞いを保ちつつ、 理解や修正が簡単になるように、ソフトウェアの 内部構造を変化させること （出典：リファクタリング） Aを実現する コード Bを実現する コード Cを実現する コード Aを実現する コード Bを実現する コード Cを実現する コード 共通コード リファクタリングのイメージ 振る舞いA 振る舞いB 振る舞いC 振る舞いA 振る舞いB 振る舞いC リファクタリング前 リファクタリング後
- 4. • リファクタリングのきっかけとなる、設計上 の問題 コードの臭い（code smell） 臭い 状態 重複したコード 同じ処理が複数個所に存在する 長すぎるメソッド 一つのメソッドに多くの処理が記述されている 巨大なクラス あるクラスが、責務を多く持ちすぎている 多すぎる引数 メソッドに渡す引数の数が多すぎる 変更の発散 １つのクラスがさまざまな変更要求の影響を被る 変更の分散 1つの変更要求によって、複数のクラスが影響を被る 属性、操作の横恋慕 自クラスより他クラスの属性、操作と関連が深い データの群れ 数個のデータがグループとなって各所に現れる 基本データ型への執着 基本データ型を使用し、複雑なロジックが記述されている スイッチ文 スイッチ文により設計が複雑化、硬直化している パラレル継承 あるクラスの継承ツリーと、他のクラスの継承ツリーに関連がある。 表. コードの臭いの例 出典：リファクタリング
- 7. 進化的設計とリファクタリング • 進化的設計 – 仕様・設計の変更にともない、コードの修正が多発する • コストとリスクを抑える必要がある – TDD – リファクタリング • これらの手法を学び、適用することで、「コストとリスクを抑えて」 進化的設計(アジャイル開発)を実現する
- 8. オブジェクト指向とカプセル化① • オブジェクト指向、非オブジェクト指向のソフトウェアの構成の差異 データ 処理 非オブジェクト指向 オブジェクト指向 メッセージ カプセル化 データ 処理
- 11. オブジェクト指向とカプセル化④ • クラスは属性と振る舞い（操作）を持っている • クラスは実体を持たない • オブジェクトはクラスをインスタンス化することで生成される実体 クラス 鯛焼きの型 鯛焼き 作る インスタンス化 オブジェクト (インスタンス)
- 12. オブジェクト指向とカプセル化⑤ • インターフェースと 実装の分離 処理１ 処理2 処理3 処理4 データA クラスA 処理１ 処理3 インターフェース1 処理2 インターフェース2 クラスB クラスC 処理１、処理３が できるということ しか知らない 処理２ができると いうことしか 知らない クライアント
- 13. クライアント オブジェクト指向とカプセル化⑥ • クライアントは、扱うオブジェクトが求める インターフェースにしたがっている限り、 そのオブジェクトの型を知らなくてすむ。 • クライアントは、これらのオブジェクトを実装 しているクラスを知らなくて済む。クライアン トは インターフェースについてのみ知っていれば よい。 • 当該インターフェースの振る舞いさえ保たれ れば、クライアントは、オブジェクトの内部実 装が変化しても影響は無い。 クラスB クラスC 処理１、処理３が できるということしか 知らない 処理２ができるというこ としか知らない リファクタリングのポイント ！
- 14. 実装 実装 オブジェクト指向とカプセル化⑦ • コンポーネントの形成 クラスA 処理１ 処理3 インターフェース1 処理2 インターフェース2 一つのクラスでもよい クラスの集合体でもよい または コンポーネント
- 15. カプセル化とリファクタリング • リファクタリングとは – 外部から見たときの振る舞いを保ちつつ、理解や修正が簡単になるように、 ソフトウェアの内部構造を変化させること （出典：リファクタリング） コンポーネント単位で、これを適用する 振る舞い：コンポーネントとして提供するインターフェース 内部構造：クラス構造
- 16. 臭 い きれい 臭 い きれい パターン適⽤用の意義① • リファクタリングとパターンの関係 同じようなクラス構造が よく出てくる… ・分析・抽出 ・一般化 ・カタログ化 リファクタリング コード 臭い コード きれい リファクタリング コード 臭い コード きれい 過去 現在 デザインパターン 指針
- 17. • リファクタリングとパターンの関係 パターン適⽤用の意義② リファクタリング規模 小 大 リファクタリング・ カタログ （出典：リファクタリング） デザインパターン （GoF） ・Factory Method ・State など 領 域 カタログやデザインパターンはリフ ァクタリング全体の中では一部である ！
- 19. • 第２回のコード 例例題
- 20. 例例題 社⻑⾧長のコード shain = Tanto.new(100) puts shain.standup puts "私の給料は#{ shain.calculate_salary}円です。" shain = Shunin.new(100) shain.standup puts "私の給料は#{ shain.calculate_salary}円です。" shain = Bucho.new(100) shain.standup puts "私の給料は#{ shain.calculate_salary}円です。"
- 22. パターンを適⽤用
- 23. 社⻑⾧長コード ファクトリーを作成 factory = ShainFactory.new 社員の役職を引数でファクトリークラスに渡す shain = factory.create(“Tanto”,100) 部長の生成 shain = factory.create(“Bucho”,100)
- 24. 社員コード module Salary def calculate_salary kihon + teate end def kihon @kihonkyu end def teate 0 end end class Shain include Salary def initialize(kihonkyu) @kihonkyu = kihonkyu end end class Tanto < Shain def standup "新入社員は慌てて起立しました" end end 第２回のコード
- 25. • まずはテストの作成 • Rspecを使用 テスト # -*- coding: utf-8 -*- require_relative './shain' describe ShainFactory do let(:shain_factory) {ShainFactory.new} it ' Tantoオブジェクトを生成しているかどうかをチェックする' do expect(shain_factory.create('Tanto',100)).to be_an_instance_of Tanto end end
- 26. • ShainFactoryクラスを作っていないのでエラー テスト はじめはエラー class ShainFactory def create(type,kihonkyu) shain = nil if type == 'Tanto' shain = Tanto.new(kihonkyu) end shain end end
- 27. Rspecでテスト yoshi$ rspec -‐-‐format=d test_spec.rb ShainFactory Tantoオブジェクトを生成しているかどうかをチェックする
- 29. • 車とトラックのクラスがあり、前進させるメソ ッドを持っています。さらに、タイヤを回転さ せます。 • このプログラムをリファクタリングして、重複 をなくしてください。 演習２
- 30. 演習２ ⾞車車、トラック、タイヤクラス class Tire def spin_car "spinning car tire" end def spin_truck "spinning truck tire" end end class Vehicle def initialize @tire = Tire.new end end class Car < Vehicle def move_forward puts "前進する" return @tire.spin_car end end class Truck < Vehicle def move_forward puts "前進する" return @tire.spin_truck end end
- 31. 演習２ RSpec # -*- coding: utf-8 -*- require_relative './vehicle' describe Car do let(:car) {Car.new} it ' spining car tireを返すかどうかをチェックする' do expect(car.move_forward).to eq 'spinning car tire' end end describe Truck do let(:truck) {Truck.new} it ' spining truck tireを返すかどうかをチェックする' do expect(truck.move_forward).to eq 'spinning truck tire' end