Upgrade to Pro — share decks privately, control downloads, hide ads and more …

リファクタリング

81fe622dd65db0bd338dfccecdbf49f3?s=47 takakuda
June 26, 2019
17

 リファクタリング

81fe622dd65db0bd338dfccecdbf49f3?s=128

takakuda

June 26, 2019
Tweet

Transcript

  1. ZEALS Co.,Ltd デザインパターンでリファクタ リファクタリング テクノロジー開発部 takakuda

  2. 前職:雛人形売り - Ruby - Rails ZEALS Rails Engineer takakuda :takakdua

    : @kutaike1504
  3. fanpの紹介

  4. リファクタリング

  5. コンピュータプログラミングにおいて、プログラムの外部 から見た動作を変えずにソースコードの内部構造を整理 すること 引用: ウィキペディア リファクタリングとは?

  6. リファクタやってます?

  7. どうやってリファクタする?

  8. ・まずやったこと ・オブジェクト指向でリファクタ ・デザインパターンでリファクタ 今日話すこと

  9. まずやったこと

  10. 使われてないコードを消す

  11. debride2okuribito(https://github.com/shakemurasan/debride2okuribito) • コードを静的解析するgem • 未使用の可能性が高いメソッドを列挙してくれる • 静的解析なので、列挙に時間がかからない

  12. okuribito_rails(https://github.com/shakemurasan/okuribito_rails) • コードを動的解析するgem • APIによる外部呼び出しも含めて分析可能 • ymlファイルにメソッドを記述するだけで使用可能

  13. Image

  14. コードを追加することは簡単 コードを削除することは難しい

  15. オブジェクト指向でリファクタ

  16. 読み倒す

  17. 第7章 モジュールでロールの振る舞いを共有する

  18. 同じロジックが複数fileに記述されている

  19. ActiveSupport::Concern

  20. 使い回すことができる ロジックを concernに切り出して、使い たいところでincludeするだ け

  21. 変更が簡単

  22. 楽に行数は減らせる …でも委譲したい

  23. デザインパターンでリファクタ

  24. デザインパターン…?

  25. GoF

  26. 書籍『オブジェクト指向における再利用のためのデザインパター ン』の著者4人のことを指す プログラミングのベストプラクティスを体系化したもの

  27. ・アブストラクトファクトリ  ・インタプリタ ・ビルダ           ・イーテレータ ・ファクトリメソッド     ・オブサーバ ・シングルトンパターン    ・ストラテジ ・アダプタ         ・コンポジット ・デコレータ ・プロキシ ・コマンド

  28. Builder Pattern

  29. オブジェクトの作成が複雑 複数modelをまたぐ処理

  30. None
  31. Builder Patternは、 作成過程を決定する「Director」 作業インタフェースをもつ「Builder」 を組み合わせることで、 柔軟にオブジェクトを生成をできるデザインパターン Builder Pattern とは…?

  32. None
  33. Fat model を避け 単機能として委譲

  34. Active::Record == model?

  35. Active::Recordを継承しない ふつうのオブジェクトに委譲

  36. まとめ

  37. - 既存のプログラミングベストプラクティス調べる - Active::Record == model ではない - 単機能として切り出す -

    なによりリファクタは辛いことではなく、ひたすらコード と向き合い新たな知見が広がるすごく楽しいこ と!!!!
  38. None
  39. Thank you!!