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

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

[surume000]

スクリプト言語は大変だなｗ

[otakumesi]

おくりびと

[sho]

とても良いけど、Railsオンリーなのが惜しいなー

[easy-breezy]

しばらく見ておいて間違いなければ消すの良い

[ryu39]

とても良い、試してみたくなった。呼び出しの記録は、ログだけでも十分そう。

[diveintounlimit]

ネーミングセンス

[babydaemons]

これ、凄い

[yo-iida]

よさそう

[amagitakayosi]

おくりびと

[taketyan]

面白い

[tbpg]

"実際に削除したメソッド数は97、テストコードを含めた削除行数は約1000にのぼりました"

[daichirata]

prependでやればコード汚さずに済みそうな気がする

[masaru_b_cl]

基本は監視からなんだな、やっぱり

[rjge]

めっちゃ良さげ。あとokuribito_railsって名前が好き。

[masawada]

okuribito_rails ネーミングセンスがよすぎる

[tokzk]

“okuribito_rails”

[luccafort]

静的解析→動的監視の流れはいいのだけどおくりびとの問題が結構大きいように感じる。完全に分割出来るならかなり良さげだけど…。この部分だけを抽出したAPIとか用意してくれんだろうか…。

[uraway]

あとでやる

[t-wada]

"debrideで未使用と検出したメソッド群が、本当に未使用なのかを見定めることができる" "実際に削除したメソッド数は97、テストコードを含めた削除行数は約1000にのぼりました。一斉削除後の不具合報告は一切ありません"

[yatmsu]

いいなこれ

[Peranikov]

“okuribito_rails” www

[kamatama_41]

良さそう

[sonots]

okurubito

[civitaspo]

http://ift.tt/2rWZRjp

[Tomato-360]

これは便利そうだ

[joker1007]

なるほど、これは賢いし現実的だ。

[willnet]

よさそう