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.5k

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

Avatar for ayumu mochizuki

ayumu mochizuki

November 28, 2024
Tweet

Other Decks in Technology

See All in Technology
2025-07-25 NOT A HOTEL TECH TALK ━ スマートホーム開発の最前線 ━ SOFTWARE
Avatar for Yusuke Kuroiwa wakinchan
0
170
Power Automate のパフォーマンス改善レシピ / Power Automate Performance Improvement Recipes
Avatar for Takashi Shinohara karamem0
0
270
AIに全任せしないコーディングとマネジメント思考
Avatar for kichion kikuchikakeru
0
280
TypeScript 上達の道
Avatar for Kanon ysknsid25
23
4.8k
Wasmで社内ツールを作って配布しよう
Avatar for asuka askua
0
150
地域コミュニティへの「感謝」と「恩返し」 / 20250726jawsug-tochigi
Avatar for kasacchiful kasacchiful
0
100
生成AIによる情報システムへのインパクト
Avatar for Takaaki Yayoi taka_aki
1
200
Kiroから考える AIコーディングツールの潮流
Avatar for Oikon s4yuba
1
420
Kiro Hookを Terraformで検証
Avatar for ao-inoue ao_inoue
0
140
From Live Coding to Vibe Coding with Firebase Studio
Avatar for Firebase Thailand firebasethailand
1
320
Vision Language Modelと自動運転AIの最前線_20250730
Avatar for Yu Yamaguchi yuyamaguchi
2
720
MCPと認可まわりの話 / mcp_and_authorization
Avatar for convto convto
2
300

Featured

See All Featured
Building an army of robots
Avatar for Kyle Neath kneath
306
45k
4 Signs Your Business is Dying
Avatar for Josh Pigford shpigford
184
22k
Navigating Team Friction
Avatar for Lara Hogan lara
187
15k
The Invisible Side of Design
Avatar for Vitaly Friedman smashingmag
301
51k
Bootstrapping a Software Product
Avatar for Garrett Dimon garrettdimon
PRO
307
110k
Keith and Marios Guide to Fast Websites
Avatar for Keith Pitt keithpitt
411
22k
How to Ace a Technical Interview
Avatar for Jacob Kaplan-Moss jacobian
278
23k
Building Applications with DynamoDB
Avatar for Matt Wood mza
95
6.5k
What’s in a name? Adding method to the madness
Avatar for Product Marketing Alliance productmarketing
PRO
23
3.6k
Java REST API Framework Comparison - PWX 2021
Avatar for Matt Raible mraible
31
8.7k
GitHub's CSS Performance
Avatar for Jon Rohan jonrohan
1031
460k
GraphQLの誤解/rethinking-graphql
Avatar for sonatard sonatard
71
11k

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