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
2.8k
0

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

Avatar for ayumu mochizuki

ayumu mochizuki

November 28, 2024

Other Decks in Technology

See All in Technology
CIで使うClaude
Avatar for TomoyaIwata iwatatomoya
0
280
2年前に削除したPHPクラスが、 ある日突然決済をエラーにした
Avatar for ykagano ykagano
1
170
なぜ私たちのSREプラクティスはなかなか機能しないのか 〜システムより先に組織を見る〜 / Why our SRE practices aren't really working
Avatar for VTRyo vtryo
3
3.9k
AI時代の開発生産性は、個人技からチーム設計へ
Avatar for Atsushi Nakatsugawa moongift
PRO
4
2.2k
AI Coding Agent時代のcdk-nagガードレール 〜組織ルールを強制CIで守り抜く設計の挑戦〜
Avatar for みずほリサーチ&テクノロジーズ株式会社 先端技術研究部 mhrtech
3
320
Genie Ontologyは銀の弾丸かを考える / Is Genie Ontology a Silver Bullet?
Avatar for NTT docomo Business nttcom
0
350
Amplify Gen2でbackend.tsにCDKを定義する/しない事によるCDKの挙動の違いとユースケース
Avatar for Makky12 smt7174
1
310
Control Planeで育てるBtoB SaaSの認証基盤 - SRE NEXT 2026
Avatar for pokohide pokohide
1
2.5k
End-to-Endで考える信頼性 — LINEアプリにおける クライアント開発×SRE連携の実践
Avatar for maru maruloop
4
4.4k
SREとQA　二人三脚で進めるSLO運用/sre-qa-slo
Avatar for sugitak sugitak
0
700
タスクの複雑さでモデルを選ぶ ── Thompson Samplingで動かす“トークン/コスト最適化
Avatar for satoh-y-0323 satohy0323
0
510
はじめてのWDM
Avatar for miyukichi_ospf miyukichi_ospf
1
150

Featured

See All Featured
Conquering PDFs: document understanding beyond plain text
Avatar for Ines Montani inesmontani
PRO
4
2.9k
How to optimise 3,500 product descriptions for ecommerce in one day using ChatGPT
Avatar for Katarina Dahlin katarinadahlin
PRO
1
3.7k
Exploring anti-patterns in Rails
Avatar for Elle Meredith aemeredith
3
440
Mind Mapping
Avatar for Hélio Medeiros helmedeiros
PRO
1
280
DBのスキルで生き残る技術 - AI時代におけるテーブル設計の勘所
Avatar for soudai sone soudai
PRO
67
56k
Un-Boring Meetings
Avatar for Sebastian Deterding codingconduct
0
340
Documentation Writing (for coders)
Avatar for Carmen Chung carmenintech
77
5.4k
Design and Strategy: How to Deal with People Who Don’t "Get" Design
Avatar for Morgane Peng morganepeng
133
19k
Test your architecture with Archunit
Avatar for Yoan thirion
1
2.3k
Faster Mobile Websites
Avatar for Dean Hume deanohume
310
32k
Exploring the Power of Turbo Streams & Action Cable | RailsConf2023
Avatar for Kevin Liebholz kevinliebholz
37
6.5k
Mobile First: as difficult as doing things right
Avatar for Swwweet swwweet
225
10k

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