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
AGENTS.mdとSkillsで始めるAIエージェント活用
Avatar for そのだ sonoda_mj
2
190
SIer20年！ 培ったスキルがスタートアップで輝く時
Avatar for しゅういちろ shucho0103
0
830
フロンティアAIのゲート化と地政学リスク
Avatar for 長津孝輔 nagatsu
0
120
AIソロプレナー時代に2ヶ月で20人増員した事業創造会社の開発組織の話
Avatar for Koji Miyata miyatakoji
0
590
Snowflakeと仲良くなる第一歩
Avatar for あべ coco_se
4
420
日本 Fintech 未来予測レポート 2027〜2028年（オリジナル版）
Avatar for 8maki 8maki
0
1.6k
失敗を資産に変えるClaude Code
Avatar for Shinya Saita shinyasaita
0
320
エンジニアリング戦略の作り方 / Crafting Engineering Strategy
Avatar for iwashi iwashi86
19
6.5k
200個のGitHubリポジトリを横断調査したかった
Avatar for Issei.Komori icck
0
110
FinOps × AIエージェントで実現する コストインシデントの自動調査
Avatar for T.REX oasis1994liveforever
0
120
2026TECHFRESH畢業分享會 - Lightning Talk - 資料也要 CI/CD? 用 Airbyte 自動化資料同步
Avatar for LINE Developers Taiwan line_developers_tw
PRO
0
760
ACE-Step-1.5で見る 音楽生成AIのしくみと“破綻だけ直す”Retake機能の開発【zennfes spring 2026 登壇資料】
Avatar for asap personabb
1
120

Featured

See All Featured
What’s in a name? Adding method to the madness
Avatar for The Alliance productmarketing
PRO
24
4.1k
WENDY [Excerpt]
Avatar for Tessa Abrams tessaabrams
11
38k
Odyssey Design
Avatar for Ryan Kendrick rkendrick25
PRO
2
690
Helping Users Find Their Own Way: Creating Modern Search Experiences
Avatar for Dan Newman danielanewman
31
3.2k
The AI Revolution Will Not Be Monopolized: How open-source beats economies of scale, even for LLMs
Avatar for Ines Montani inesmontani
PRO
3
3.5k
Agile Leadership in an Agile Organization
Avatar for Dr. Kim W Petersen kimpetersen
PRO
0
160
The browser strikes back
Avatar for Jono Alderson jonoalderson
0
1.2k
Breaking role norms: Why Content Design is so much more than writing copy - Taylor Woolridge
Avatar for UX Y'all uxyall
0
320
Making the Leap to Tech Lead
Avatar for Ryan Cromwell cromwellryan
135
9.9k
RailsConf 2023
Avatar for Aaron Patterson tenderlove
30
1.5k
実際に使うSQLの書き方 徹底解説 / pgcon21j-tutorial
Avatar for soudai sone soudai
PRO
201
75k
A Soul's Torment
Avatar for Nat Ma seathinner
6
2.9k

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