アカウント名:
パスワード:
まあ仕方ないと思わなくもなくもなくもないけど(VCS使っていない事自体がダメすぎてとっとと逃げる算段を計るけど)、VCS使ってまでソースをゴミで埋めたがるんだからマジで何やってるのか意味分からない。# いちおうソースコードの管理について意見を出せる立場だったので反対してみたけどダメだった
VCSありかつコードのコメントアウト禁止というルールで開発をやってたことがありますが、どうしてもやってしまう人がいました。
削除し忘れたと言うんですが、そもそもなんでコメントアウトしておくのかと聞いてみると、適当に修正してテストしてOKだったらcommitするというやり方をしているそうです。要するに原因を突き止めて対策を練ってから修正するのではなくて、行き当たりばったりなんですね。とりあえず、そういうやり方ならコードのコメントアウトは便利だと納得はしました。
コメントの意味も含めてコードレビューをすれば、最終的にはコードのコメントアウトを削除できるはずです。このコードには意味があってとってあると言われたら、検証済みか/検証手順は整備されているか聞けば良いでしょう。レビューしないような環境なら好きなようにやらせてあげようよ。
ウチである案件なら、コメントアウトされている理由の殆どは、「仕様が確定していないから」だったりする。だからコメントに「もし客が○○と言い出したら…」なんてコメントアウトされている予備のコードがバラバラと、下手すれば有効なコードより多かったり。コードレビューなんてさせてくれれば相当にマシ。検収試験中に思いつきで変更を指示し、即日対応を求める所も珍しい話じゃない。いや、それを断れなくする為にわざと仕様を曖昧なままにする会社すら有る。
それこそソース管理でやるべきでは複数のソースを一個においとくとか可読性が下がるだけですうまく使いましょう
つか、本当に仕様変更への対応予備コードとして準備しているのであれば、#ifdef~#endifなんかでコンパイルオプションにすればいいだけ。
「予備だから事前に充分検証しない」って危険性があるのであれば適当に#warn とか #error とか織り交ぜて防護壁張っとけばいいわけだし。
とかコードレビューで突っ込んだらコメントアウトする代わりに#if 0 ~ #endif使うバカがいた...
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
弘法筆を選ばず、アレゲはキーボードを選ぶ -- アレゲ研究家
VCS使ってないなら (スコア:2, 興味深い)
まあ仕方ないと思わなくもなくもなくもないけど(VCS使っていない事自体がダメすぎてとっとと逃げる算段を計るけど)、VCS使ってまでソースをゴミで埋めたがるんだからマジで何やってるのか意味分からない。
# いちおうソースコードの管理について意見を出せる立場だったので反対してみたけどダメだった
Re:VCS使ってないなら (スコア:1)
VCSありかつコードのコメントアウト禁止というルールで開発をやってたことがありますが、
どうしてもやってしまう人がいました。
削除し忘れたと言うんですが、そもそもなんでコメントアウトしておくのかと聞いてみると、
適当に修正してテストしてOKだったらcommitするというやり方をしているそうです。
要するに原因を突き止めて対策を練ってから修正するのではなくて、行き当たりばったりなんですね。
とりあえず、そういうやり方ならコードのコメントアウトは便利だと納得はしました。
コメントの意味も含めてコードレビューをすれば、最終的にはコードのコメントアウトを削除できるはずです。
このコードには意味があってとってあると言われたら、検証済みか/検証手順は整備されているか聞けば良いでしょう。
レビューしないような環境なら好きなようにやらせてあげようよ。
Re: (スコア:0)
ウチである案件なら、コメントアウトされている理由の殆どは、
「仕様が確定していないから」
だったりする。
だからコメントに
「もし客が○○と言い出したら…」
なんてコメントアウトされている予備のコードがバラバラと、下手すれば有効なコードより多かったり。
コードレビューなんてさせてくれれば相当にマシ。
検収試験中に思いつきで変更を指示し、即日対応を求める所も珍しい話じゃない。
いや、それを断れなくする為にわざと仕様を曖昧なままにする会社すら有る。
Re: (スコア:0)
それこそソース管理でやるべきでは
複数のソースを一個においとくとか可読性が下がるだけです
うまく使いましょう
Re: (スコア:0)
つか、本当に仕様変更への対応予備コードとして準備しているのであれば、
#ifdef~#endifなんかでコンパイルオプションにすればいいだけ。
「予備だから事前に充分検証しない」って危険性があるのであれば
適当に#warn とか #error とか織り交ぜて防護壁張っとけばいいわけだし。
とかコードレビューで突っ込んだらコメントアウトする代わりに
#if 0 ~ #endif
使うバカがいた...