Upgrade to Pro
— share decks privately, control downloads, hide ads and more …
Speaker Deck
Features
Speaker Deck
PRO
Sign in
Sign up for free
Search
Search
リファクタリング
Search
takakuda
June 26, 2019
0
39
リファクタリング
takakuda
June 26, 2019
Tweet
Share
More Decks by takakuda
See All by takakuda
パフォーマンス改善
takakuda
0
150
gem version up
takakuda
0
2.3k
ぼくらのかんがえたさいきょうのfactory_bot
takakuda
0
2.4k
LINE, Messenger比べてみました。
takakuda
0
1.4k
Featured
See All Featured
BBQ
matthewcrist
85
9.3k
Embracing the Ebb and Flow
colly
84
4.4k
XXLCSS - How to scale CSS and keep your sanity
sugarenia
246
1.3M
GitHub's CSS Performance
jonrohan
1030
460k
Cheating the UX When There Is Nothing More to Optimize - PixelPioneers
stephaniewalter
280
13k
Creating an realtime collaboration tool: Agile Flush - .NET Oxford
marcduiker
25
1.8k
Gamification - CAS2011
davidbonilla
80
5k
Save Time (by Creating Custom Rails Generators)
garrettdimon
PRO
27
770
Learning to Love Humans: Emotional Interface Design
aarron
272
40k
It's Worth the Effort
3n
183
27k
Designing for humans not robots
tammielis
249
25k
We Have a Design System, Now What?
morganepeng
50
7.2k
Transcript
ZEALS Co.,Ltd デザインパターンでリファクタ リファクタリング テクノロジー開発部 takakuda
前職:雛人形売り - Ruby - Rails ZEALS Rails Engineer takakuda :takakdua
: @kutaike1504
fanpの紹介
リファクタリング
コンピュータプログラミングにおいて、プログラムの外部 から見た動作を変えずにソースコードの内部構造を整理 すること 引用: ウィキペディア リファクタリングとは?
リファクタやってます?
どうやってリファクタする?
・まずやったこと ・オブジェクト指向でリファクタ ・デザインパターンでリファクタ 今日話すこと
まずやったこと
使われてないコードを消す
debride2okuribito(https://github.com/shakemurasan/debride2okuribito) • コードを静的解析するgem • 未使用の可能性が高いメソッドを列挙してくれる • 静的解析なので、列挙に時間がかからない
okuribito_rails(https://github.com/shakemurasan/okuribito_rails) • コードを動的解析するgem • APIによる外部呼び出しも含めて分析可能 • ymlファイルにメソッドを記述するだけで使用可能
Image
コードを追加することは簡単 コードを削除することは難しい
オブジェクト指向でリファクタ
読み倒す
第7章 モジュールでロールの振る舞いを共有する
同じロジックが複数fileに記述されている
ActiveSupport::Concern
使い回すことができる ロジックを concernに切り出して、使い たいところでincludeするだ け
変更が簡単
楽に行数は減らせる …でも委譲したい
デザインパターンでリファクタ
デザインパターン…?
GoF
書籍『オブジェクト指向における再利用のためのデザインパター ン』の著者4人のことを指す プログラミングのベストプラクティスを体系化したもの
・アブストラクトファクトリ ・インタプリタ ・ビルダ ・イーテレータ ・ファクトリメソッド ・オブサーバ ・シングルトンパターン ・ストラテジ ・アダプタ ・コンポジット ・デコレータ ・プロキシ ・コマンド
Builder Pattern
オブジェクトの作成が複雑 複数modelをまたぐ処理
None
Builder Patternは、 作成過程を決定する「Director」 作業インタフェースをもつ「Builder」 を組み合わせることで、 柔軟にオブジェクトを生成をできるデザインパターン Builder Pattern とは…?
None
Fat model を避け 単機能として委譲
Active::Record == model?
Active::Recordを継承しない ふつうのオブジェクトに委譲
まとめ
- 既存のプログラミングベストプラクティス調べる - Active::Record == model ではない - 単機能として切り出す -
なによりリファクタは辛いことではなく、ひたすらコード と向き合い新たな知見が広がるすごく楽しいこ と!!!!
None
Thank you!!