Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ajax'@'localhost' (using password: NO) in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ajax'@'localhost' (using password: NO) in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ajax'@'localhost' (using password: NO) in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ajax'@'localhost' (using password: NO) in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ajax'@'localhost' (using password: NO) in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'ajax'@'localhost' (using password: NO) in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/ajax/sakano.com/public_html/wp/wp-content/plugins/statpress-reloaded/statpress.php on line 1786
ソースコード変更コメントという馬鹿げた習慣、そろそろやめにしませんか? - サカノシステム設計事務所

ソースコード変更コメントという馬鹿げた習慣、そろそろやめにしませんか?

2016-12-21

これほどバージョン管理システムが浸透しているにもかかわらず、なお多くの企業でソースコード変更コメントなるものを付ける習慣から抜けきれないようで笑ってしまう。

まともに開発をしていたら、ソースコード変更コメントがすぐに破綻することを知っているはずなので、それを推奨するのは現場を離れて、利益とコストのことしか頭にない管理者ぐらいなものだろう。

それでは、きわめて単純なケースで、これがどれだけ馬鹿らしい代物か見てみよう。
ここでは、物事をシンプルにするために、ソースコードも抽象化している。

(1) 元のソース

aaaaaaaaaaaaaaa
bbbbbbbbbbbbbbb

(2) 1行追加する

aaaaaaaaaaaaaaa
bbbbbbbbbbbbbbb
ccccccccccccccc

(3) 1行削除する

bbbbbbbbbbbbbbb
ccccccccccccccc

(4) 1行変更する

ddddddddddddddd
ccccccccccccccc

さて、これにVB形式のコメントを付けてみよう。次の3つの変更区分を使用する。
Add:追加
Del:削除
Upd:変更

なお、startは追加・削除・変更の開始、endは終了を表す

 

(2) 1行追加する

 

(3) 1行削除する

 

(4) 1行変更する

どうだろう?極めて単純なケースにおいても、たった3回の変更を行うだけでこの有様である。

現場では通常はこんなものでは済まされない。

こんなソースを見たらメンテナンスする勇気も失せてしまうだろう。

ソース管理はバージョン管理ソフトで適切なコメントを付けてコミットするのが正解だと早く気がつくことを切に願う。

 

Copyright© 2012-2016 サカノシステム設計事務所 All Rights Reserved.