7,600行のコードを安全にこの世から抹消した話

7,626行削減しているPull Request経緯「ランキング」のマイクロサービスを作った話 でも登場した、ランキングサービスでの話です。APIは一度リニューアルし最新はv2でした。 その後、新しい形のランキングを実装することになり、同時に、いままであった形式のランキングはなくなり、リプレイスされることになりました。そのため、新規の実装をするとともに、新しくバージョンを上げてv3のAPIを新設することにしました。 コアなロジックやデータストアにアクセスするコードについては、大部分を使い回すことができ、無事に本番リリースを行いました。 一方で、古いランキングはもう開催しないため、v2以下のAPIやそれに紐づくコードは実質的に不要になりました。そのコードを今回削除していくことにしました。 事前準備ビジネスサイドと話し、機能を切れる範囲やタイミングを確定FiNCアプリは様々なお客様に対して提供

[efcl]

コードの消し方

[h5y1m141]

手順もだけど、そこに至る前後の過程で何を考えていたのかをこうしてまとめてくれるのは 参考になる。言われると当たり前の作業かもしれないけど わかりやすくアウトプットしていくのはやっぱり大事だな。

[incep]

「当たり前」の手順が多いのだけど，体系的に把握して過不足なくやるのは難しいと思う．大事なことをきちんとまとめている

[okusa75]

rubyって不要コード消しにくいよね。機械的にチェックできないものか。

[golden_eggg]

良い

[daichirata]

deplicateメソッド使って一旦本番に上げてみるとかたまにやる

[okbm]

“消すコードが生まれるというのは、ビジネスの成長の証だと感じています。”

[ota42y]

メンテを考えるなら不要なコードは無い方がいいし、コード書くのと同じぐらい消していくのもだいじ

[qsona]

書きました。ぎんざRubyでの @mtsmfm さんの発表に触発されたので、追伸で紹介しました。