社内勉強会用に作ったスライド アジャイル用語を連呼してても、目的にリーチしてないこと多数という風刺の導入がやたらウケました。

1. リファクタリング？

2. へぼい実装
ほころびの上に継ぎ足しの連続で拡張がむずい

3. 理想の実装
見やすい！綺麗！拡張もしたくなる！

4. リファクタリングとは？
まずここから

5. いつものことだが
横文字に気をつけよう
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
ア
ジ
ャ
イ
ル
リファクタリングをすればどうなるのか？
アジャイルとはなんなのか？
※意味わかって言ってるのかと
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ
リ
フ
ァ
ク
タ
リ
ン
グ

6. 知らないまま、ろくな改善もないこともざら
普通知らない（知ってたらこうならない）
ありがちなアジャイルメソッドの誤解
こ
の
コ
｜
ド
を
リ
フ
ァ
ク
タ
リ
ン
グ
し
た
の
は
誰
だ
ぁ
！

7. 本物のリファクタリング
どうせやるなら！
来
週
ま
た
こ
こ
に
来
て
く
だ
さ
い
本
物
の
リ
フ
ァ
ク
タ
リ
ン
グ
っ
て
奴
を
見
せ
て
や
り
ま
す
よ
わ
か
り
や
す
い
関
数
名
！
適
切
な
フ
ォ
ル
ダ
と
フ
ァ
イ
ル
こ
れ
な
ら
誰
で
も
バ
グ
が
直
せ
る
わ
コールフローも
追いやすいの！

8. リファクタリングのゴール
いつもフリが長いけど
誰でもわかる
↓
バグもわかりやすい
↓
誰でも直せる・拡張できる
↓
迅速な開発
わ
か
り
や
す
い
関
数
名
！
適
切
な
フ
ォ
ル
ダ
と
フ
ァ
イ
ル
こ
れ
な
ら
誰
で
も
バ
グ
が
直
せ
る
わ
コールフローも
追いやすいの！

9. これからやること
適切な構造
他人が読める関数名とフロー
上記を元にまともな話へ
こ
の
コ
｜
ド
を
書
い
た
の
は
誰
だ
ぁ
！
ロバート・マーチン著

10. まず、なにを
リファクタリングするのか？
それは、へぼいコードである

11. なにがへぼいコードなのか？
へぼいコードはこんな特徴である
硬い 変更・付け足ししにくい
もろい
変更したら無駄に関連して
支障が出るものがある
無駄に複雑
シンプルが100点複雑さは減点
理解にコストかかりまくる
無駄な繰り返し
バグる時も一緒
直すときは一つ
不透明 本人以外わからん
こ
の
コ
｜
ド
は
出
来
損
な
い
だ
迅
速
に
開
発
な
ん
か
出
来
ま
せ
ん
よ

12. マーチン曰く
「仕様は変わるものというか、
いろいろ変更してベストまで磨くのが良い製品
それをやりにくくさせるのが悪いコード」
硬い 変更・付け足ししにくい
もろい
変更したら無駄に関連して
支障が出るものがある
無駄に複雑
シンプルが100点複雑さは減点
理解にコストかかりまくる
無駄な繰り返し
バグる時も一緒
直すときは一つ
不透明 本人以外わからん
こ
の
コ
｜
ド
を
書
い
た
の
は ロバート・マーチン
誰
だ
ぁ
！

13. マーチンの教え
• プロのチームは前述のコードの腐敗を
許してはならない
• 可能な限り美しい設計を保つ
• 無菌の手術室を守る外科医の精神であ
れ
こ
の
コ
｜
ド
を
書
い
た
の
は ロバート・マーチン
誰
だ
ぁ
！

14. っというわけで
硬い 変更・付け足ししにくい
もろい
変更したら無駄に関連して
支障が出るものがある
無駄に複雑
シンプルが100点複雑さは減点
理解にコストかかりまくる
無駄な繰り返し
バグる時も一緒
直すときは一つ
不透明 本人以外わからん
この五箇条に当たるモノを直す

18. 本物のリファクタリング
わ
か
り
や
す
い
関
数
名
！
適
切
な
フ
ォ
ル
ダ
と
フ
ァ
イ
ル
こ
れ
な
ら
誰
で
も
バ
グ
が
直
せ
る
わ
コールフローも
追いやすいの！
来
週
ま
た
こ
こ
に
来
て
く
だ
さ
い
本
物
の
リ
フ
ァ
ク
タ
リ
ン
グ
っ
て
奴
を
見
せ
て
や
り
ま
す
よ
マーチン先生の教えを守り
をやりましょう！
ルールを頭にたたき込んで
ペアでお互いのコードをチェック！