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
31
リファクタリング
takakuda
June 26, 2019
Tweet
Share
More Decks by takakuda
See All by takakuda
パフォーマンス改善
takakuda
0
140
gem version up
takakuda
0
2.3k
ぼくらのかんがえたさいきょうのfactory_bot
takakuda
0
2.3k
LINE, Messenger比べてみました。
takakuda
0
1.4k
Featured
See All Featured
jQuery: Nuts, Bolts and Bling
dougneiner
59
7.2k
Optimising Largest Contentful Paint
csswizardry
12
2.4k
A designer walks into a library…
pauljervisheath
201
23k
Teambox: Starting and Learning
jrom
128
8.4k
Web Components: a chance to create the future
zenorocha
306
41k
Thoughts on Productivity
jonyablonski
59
3.8k
Typedesign – Prime Four
hannesfritz
36
2.1k
A Philosophy of Restraint
colly
197
16k
I Don’t Have Time: Getting Over the Fear to Launch Your Podcast
jcasabona
22
1.6k
Designing the Hi-DPI Web
ddemaree
276
33k
Ruby is Unlike a Banana
tanoku
96
10k
Principles of Awesome APIs and How to Build Them.
keavy
121
16k
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!!