プログラム修正の際に古いソースをコメントアウトするのはダメ？

質問

SE歴8年目のものです。
現在の仕事では、PHPで開発しており、Subversionで構成管理しています。

私はこれまでソースを修正する場合には、古いソースをコメントアウトし、さらに修正日付、修正者名、修正内容をコメントで入れるようにしてきました。

(例)
-----------------------------------
//090401 update [氏名] #1234のバグ改修 start
/*
[古いソース]
*/
[修正後のソース]
//090401 update [氏名] #1234のバグ改修 end
-----------------------------------

しかし、今一緒に開発している同僚のSEは、「古いソースをコメントアウトしたり、コメントに日付を入れたりするのは、構成管理ツールが無かった頃の古いやり方だ。今はむしろやってはいけない風潮になっているから、すべきではない。」といい、そのやり方に反対しています。

確かに私も商用リリース前のソースの修正に関してはこんなことはしないのですが、商用リリース済みのものに関しては慎重にやりたいので、情報をたくさん残すようにしています。

この同僚のSEの言う通り、このやり方は古くやらない方が良いのでしょうか？
皆様の開発現場はどうされてますか？

回答No.6

mmsb · ベストアンサー

ＳＥやって１６年目になるものです。

同僚の方の主張、何かの本で読んだような気が…と思っていろいろ調べてみたら、
「達人プログラマー」という有名な書籍の中にありました。

『変更管理はツールの仕事。ヘッダに日付と名前と概要だけ書けば良い』
というようなことが記述されています。

改修を重ねるとコメントでソースが読みにくくなるのは気分が悪いですよね～
生産性にも影響しているはずですしね。

「達人プログラマー」を読んで以降、その通りだな～と考えているのですが…

わが社もVSSで管理しているのに修正前のロジックをコメントアウトしてます。
分かっちゃいるけど、やめられない…（＾＾；

そういえば昔、チーム内でソースを「作品」と呼んでいたことがありました。
「このソース、誰の作品？」とか。

みんなで「ソースはアートだ」ぐらいのプライドを持って仕事をすると、
３Ｋとか７Ｋとか言われているこの業界も、少しは人気が回復するかな？
ソースを汚す"修正履歴のコメントアウト文化"は捨てるべきかも知れませんよ（笑

参考URL：

http://www.amazon.co.jp/gp/product/4894712741/ref=sib_rdr_dp

お礼

回答ありがとうございます。
きっと同僚もその本読んだのでしょうね。

今は一切コメントアウトしない方針で開発しています。