バージョン管理しているのに、バージョン管理を信用していないため、要らなくなったコードも残してしまうウンコード。
/* --- 2007.06.21 故障-ST-001 start --- */ // public void uncode (int status_code) { // int status = 0; // List unkodes = null; // if (status_code == 1) { // unkodes = fetch_unkode(); /* #### 2006.05.11 仕様追加-ST-012 k.sato start #### */ // // } else { // } else if (status_code == 2) { // unkodes = search_unkode(); // } else { // unkodes = create_unkode(); /* #### 2006.05.11 仕様追加-ST-012 k.sato start #### */ // } public int uncode (int status_code) { List unkodes = get_unkode(status_code); /* --- 2007.06.21 故障-ST-001 end --- */
使い方ヒント: 「これは臭う」という行を見付けたら、各行のをクリックしてマーキングしておきましょう(要Twitter OAuth認証)
前の会社がこれでした!PL「前からの慣習だから…」PM「変更した所わからなくなるじゃん」三( ;´Д`)逃げよう
前の会社がこれでした!PL「前からの慣習だから…」PM「変更した所わからなくなるじゃん」三( ;´Д`)逃げよう
よくある。ついでにいうと、こんな感じでぐっちゃぐっちゃになったコード数百行が実は #if 0 ~ #endif で囲まれてることにあとで気づいて殺意を覚える。
変更箇所が解らなくなるからって言う奴に限って、これ見ても変更箇所わからなかったりする
本当にこれはやめてほしい。何が書いてあるかわからないし、理解するのにも時間がかかるし・・・。 どういう経緯で、こうなったのかもわからないし。 今いる職場もこんな感じだけど、少しずつ直してる・・・。
よくある話。過去のいらないソースばっかで、本文は1/5程度だったものもアリ。。。
よくある話。過去のいらないソースばっかで、本文は1/5程度だったものもアリ。。。
やるならせめてローカルで覚書程度に使用して、公開しないで欲しい形
よくわかる。 何のためのバージョン管理ツールか・・・。
たまにイラっとして全部コメントアウトして、新しく書き直すとか暴挙に出てやる。 そして現場のコーダーに「やりすぎです」とか苦笑いされるw
バージョン管理ツールのヒストリーを見れば一発でわかるというのに・・・ 印刷したリストを見ながらレビューをやるという慣例がありまして…(--;
ここ2ヶ月やってきた作業環境がまさにコレ。実施する意義を問うたのだけど「そうだったから」的な回答。たいていそう。何カ所かは全コメントアウトして書き直した。だって「過去のStart/End囲いの中に、新しいStart/Endが入る」とか「既に入れ子じゃなくて複数修正が噛み合ってる」とか、仮にルールとして実施するにしても破綻/矛盾してるし。
たいていこういうのやらせる人は「VSSとかVisualStudioが無い状態を想定して……」とか抜かすけど、恐らく「ソースの9割が過去のコメント」となりはてたソースを読み解くのには、テキストエディタじゃ無理。Visual Studioとかの環境が持つエディタで色分けしてくれないと読めないという矛盾(笑)。ツール無し状況のためにやってる作業が、逆にツールが無いと読めないソースを作ってるというバカげた事態。
たいていこういうのやらせる人は「VSSとかVisualStudioが無い状態を想定して……」とか抜かすけど、恐らく「ソースの9割が過去のコメント」となりはてたソースを読み解くのには、テキストエディタじゃ無理。Visual Studioとかの環境が持つエディタで色分けしてくれないと読めないという矛盾(笑)。ツール無し状況のためにやってる作業が、逆にツールが無いと読めないソースを作ってるというバカげた事態。
あー懐かしいわ腐るほど見てきたなw VSS使ってるのにやめましょうは取り合ってくれなかったw まさに今までそうだし、こう書きましょうというルールがあるからで終了~~~みたいな
結果、条件式の追加したときインデント修正のために、ごっそりコメントアウトになって読みづらくなったコードや、逆にインデントを修正しないという暴挙にでて全くヨメなくなったコードを見たことある。
コメント投稿には、twitter認証が必要です。
Twitter認証
バージョン管理してるんだ・・・削除しよう!(`・ω・´)