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

歴史あるRuby on Railsでデッドコードを見つけ、 消す方法@yabaibuki.de...

Avatar for ayumu mochizuki ayumu mochizuki
November 28, 2024
Technology
0
2.6k

歴史あるRuby on Railsでデッドコードを見つけ、 消す方法@yabaibuki.dev #3

Avatar for ayumu mochizuki

ayumu mochizuki

November 28, 2024
Tweet

Other Decks in Technology

See All in Technology
20251014_Pythonを実務で徹底的に使いこなした話
Avatar for 森田　一平 ippei0923
0
210
AWS Control Tower に学ぶ！ IAM Identity Center 権限設計の第一歩 / IAM Identity Center with Control Tower
Avatar for Yuta Otani y___u
1
200
Claude Code Subagents 再入門 ~cc-sddの実装で学んだこと~
Avatar for Gota gotalab555
10
16k
能登半島災害現場エンジニアクロストーク 【JAWS FESTA 2025 in 金沢】
Avatar for Masakatsu Sugii ditccsugii
0
900
20251010_HCCJP_AdaptiveCloudUpdates
Avatar for osamut sdosamut
0
140
カンファレンスに託児サポートがあるということ / Having Childcare Support at Conferences
Avatar for nobu09 nobu09
1
620
WEBサービスを成り立たせるAWSサービス
Avatar for Takano takano0131
1
180
それでも私が品質保証プロセスを作り続ける理由 #テストラジオ / Why I still continue to create QA process
Avatar for ぱいん pineapplecandy
0
130
コンテキストエンジニアリング入門〜AI Coding Agent作りで学ぶ文脈設計〜
Avatar for KNOWLEDGE WORK / 株式会社ナレッジワーク kworkdev
PRO
3
1.8k
Introduction to Sansan for Engineers / エンジニア向け会社紹介
Avatar for Sansan, Inc. sansan33
PRO
5
43k
Introduction to Sansan, inc / Sansan Global Development Center, Inc.
Avatar for Sansan, Inc. sansan33
PRO
0
2.8k
Click A, Buy B: Rethinking Conversion Attribution in ECommerce Recommendations
Avatar for LINEヤフーTech (LY Corporation Tech) lycorptech_jp
PRO
0
100

Featured

See All Featured
Bootstrapping a Software Product
Avatar for Garrett Dimon garrettdimon
PRO
307
110k
What’s in a name? Adding method to the madness
Avatar for Product Marketing Alliance productmarketing
PRO
24
3.7k
YesSQL, Process and Tooling at Scale
Avatar for Rocio Delgado rocio
173
14k
ピンチをチャンスに：未来をつくるプロダクトロードマップ #pmconf2020
Avatar for Aki / @LoveIdahoBurger aki_iinuma
127
54k
Understanding Cognitive Biases in Performance Measurement
Avatar for Philip Tellis bluesmoon
31
2.7k
CSS Pre-Processors: Stylus, Less & Sass
Avatar for Bermon Painter bermonpainter
359
30k
10 Git Anti Patterns You Should be Aware of
Avatar for Lemi Orhan Ergin lemiorhan
PRO
657
61k
Easily Structure & Communicate Ideas using Wireframe
Avatar for Afnizar Nur Ghifari afnizarnur
194
16k
The Cult of Friendly URLs
Avatar for Andy Hume andyhume
79
6.6k
Being A Developer After 40
Avatar for Adrian Kosmaczewski akosma
91
590k
Visualizing Your Data: Incorporating Mongo into Loggly Infrastructure
Avatar for mongodb mongodb
48
9.7k
Creating an realtime collaboration tool: Agile Flush - .NET Oxford
Avatar for Marc Duiker marcduiker
34
2.3k

Transcript

  1. 歴史あるRuby on Railsで デッドコードを⾒つけ、 消す⽅法 2024.11.28 リブセンス アルバイト事業部 望⽉歩 Copyright

    © Livesense Inc.

  2. 株式会社リブセンス  アルバイト事業部開発グループ 2018年新卒で⼊社 マッハバイトの開発をしています 望⽉ 歩(ayumu838) Copyright © Livesense Inc.

  3. • 2006年にジョブセンスとしてリリース • 2017年にマッハバイトと改名 • リリース当時はPHPだったが • 2014年からRuby on Railsにリプレイスを開始

    • 2022年に完了 について Copyright © Livesense Inc.

  4. • 2006年にジョブセンスとしてリリース • 2017年にマッハバイトと改名 • リリース当時はPHPだったが • 2014年からRuby on Railsにリプレイスを開始

    • 2022年に完了 について Copyright © Livesense Inc.

  5. • 2006年にジョブセンスとしてリリース • 2017年にマッハバイトと改名 • リリース当時はPHPだったが • 2014年からRuby on Railsにリプレイスを開始

    • 2022年に完了 リプレイス開始から10年 について Copyright © Livesense Inc.

  6. 不要なコードを削除したい Copyright © Livesense Inc.

  7. • 不要なコードを検知する • 検知したコードを継続的に消していけるようにする 何をしたらいいのか Copyright © Livesense Inc.

  8. • 不要なコードを検知する • 検知したコードを継続的に消していけるようにする←こっちが本題 何をしたらいいのか Copyright © Livesense Inc.

  9. 不要なコードを検知する Copyright © Livesense Inc. • oneshot coverageで動的に検知 • coverbandを導⼊することにしました

    • いい感じに保存や可視化をしてくれるツール https://github.com/danmayer/coverband/tree/main

  10. 不要なコードを検知する Copyright © Livesense Inc. • 実⾏されていない⾏

  11. 不要なコードを検知する Copyright © Livesense Inc. • 描画されていないView

  12. 不要なコードを検知する Copyright © Livesense Inc. • 使われていないRoute

  13. 検知はできるようになった  あとは関連したコードをどう継続して消せるようにするか Copyright © Livesense Inc.

  14. 仕組み作って、CI化しよう Copyright © Livesense Inc.

  15. 消したい対象 • routesにあるが、存在しないcontroller • routesにあるが、存在しないaction • controllerはあるが、存在しないroutes 検知したコードに関連するものも消すことを継続させる Copyright ©

    Livesense Inc.

  16. 消したい対象 • routesにあるが、存在しないcontroller • routesにあるが、存在しないaction • controllerはあるが、存在しないroutes 検知したコードに関連するものも消すことを継続させる Copyright ©

    Livesense Inc.

  17. 理想はRouting ErrorのController#Actionを取得したい routesを取得したい Copyright © Livesense Inc.

  18. Rails.application.routes.routesとすると ActionDispatch::Journey::Routesとなって Rails.application.routes.routes.frist.defaultsでcontrollerとactionが取れる routesを取得したい Copyright © Livesense Inc.

  19. 消したい対象 • routesにあるが、存在しないcontroller • routesにあるが、存在しないaction • controllerはあるが、存在しないroutes 検知したコードに関連するものも消すことを継続させる Copyright ©

    Livesense Inc.

  20. 先ほど取得した結果のcontrollerを app/controllers/{controller}_controller.rbにしてファイル検索ができる controllerやactionを取得したい Copyright © Livesense Inc.

  21. 先ほど取得した結果のcontrollerを app/controllers/{controller}_controller.rbにしてファイル検索ができる actionも def {action}がファイルにあるかどうかで判断できる controllerやactionを取得したい Copyright © Livesense Inc.

  22. 消したい対象 • routesにあるが、存在しないcontroller • routesにあるが、存在しないaction • controllerはあるが、存在しないroutes 検知したコードに関連するものも消すことを継続させる Copyright ©

    Livesense Inc.

  23. 先ほど取得した結果のcontrollerを app/controllers/{controller}_controller.rbにしてファイル検索ができる ↓ app/controllers/**/*.rb から app/controllers/ _controller.rb を消したものがcontrollerになる controllerだけあるものを取得したい Copyright ©

    Livesense Inc.

  24. 仕組み作って、CI化しよう Copyright © Livesense Inc.

  25. • GitHub Actions上で動かす • GitHub Actionsの形式に沿ったエラーメッセージを出す 検知したコードに関連するものも消すことを継続させる Copyright © Livesense

    Inc.

  26. • GitHub Actions上で動かす • GitHub Actionsの形式に沿ったエラーメッセージを出す 検知したコードに関連するものも消すことを継続させる Copyright © Livesense

    Inc.

  27. • 不要なコードを検知し、消す • controllerやaction,routesを消せばCIが落ちるように仕組み化 まとめ Copyright © Livesense Inc.

  28. None

  29. 詳しくはアドベントカレンダーに書きます！ 12/1予定です Copyright © Livesense Inc.

  30. None
  31. None