コードを共通化するな

プログラミングできる気になった自称中級者は、ソースコードに共通のパターンが現れると決まって、その処理を関数などに共通化したがる。 たしかに、そうすることでソースコードは短くなるし、一見して保守性が上がったような気になるのだが、それは間違った作法だから止めろ。 細かいこと言っても伝わらない自称プログラマが読んでることを想定して、先に結論を簡単に書いておく。 お前は絶対にコードを共通化するな。 共通化してはいけない理由なぜコードを共通化するのがいけないのか。理由は簡単だ。要するに、コードが似ているのは単なる偶然であって、それらは別の処理だからだ。 別の処理だから共通化するのはおかしいし、もし共通化した処理の一方のみ仕様が変わった場合、その修正は他方にも影響してしまう。つまり、保守性が下がっている。 たとえば、同じプロジェクトの中に、10%の消費税を加える処理と、10%の金利を加える処理があった

[moxtaka]

これは正しい。自分がプログラミングで大事にしているのは名前、スコープ、アルゴリズムで、共通化は優先度が低い。

[by-king]

ジャズができる気になった自称中級者は、コード進行に共通のパターンが現れると決まって、アドリブを共通化したがる。たしかに、そうすることで覚える量は少なくなるし、一見して上達した気になるのだが以下略

[Yoshiya]

まともな事務系のプログラマならば、こんなマヌケな処理はしない「10%の消費税を加える処理と、10%の金利を加える処理があったとする。この2つの処理はともに元の金額を1.1倍する処理であり、全く同じ処理であるが…」

[madooka]

“名前をつけろ” が本文で、適切な名前がつけばその共通処理は意味も共通なのでは。名前が処理名になったらそれは偶然一致した処理で。名付け重要

[Lagenaria]

要件上は全く別の機能なのに実装が似てるからと言って共通化してしまうと、結合度が不必要に高い処理（極端になると神クラス化する）が生まれてしまう