タグ

refactoringに関するluccafortのブックマーク (1)

  • Railsで効率的かつ安全に未使用のメソッドを削除した話 - てくすた

    ピクスタ開発部で毎日ヒィヒィ言いながらエンジニアをやっております @muramurasan です。 今回はPIXTAのとあるリポジトリにおいて、未使用のメソッドを削除しようとした際、gemを組み合わせることで、効率的かつ安全に削除することができたという話をしたいと思います。 よくやる方式 外部の勉強会などで、「未使用のメソッドを削除する際にどうしているか?」ということを聞いた際、よく聞くのが「未使用らしきコードを見つけ次第、ロギングを行うメソッド呼び出しを挟み込んでいく」というものでした。 この方式は、動的なメソッド呼び出しにも当然対応できますし、お手軽なので、一般的に好まれているようです。 問題点 ただし、この方式では以下の問題点があると私は考えています。 そもそも、未使用らしいメソッドを見つけるのが大変 プロダクションコードを汚してしまう これらの問題を解決するために、PIXTAでは

    Railsで効率的かつ安全に未使用のメソッドを削除した話 - てくすた
    luccafort
    luccafort 2017/05/26
    静的解析→動的監視の流れはいいのだけどおくりびとの問題が結構大きいように感じる。完全に分割出来るならかなり良さげだけど…。この部分だけを抽出したAPIとか用意してくれんだろうか…。
  • 1