アカウント名:
パスワード:
コーディング標準が設定され、守られていればそんなことは起こらないはずだ。
あまり杓子定規なのも困り者。例えば静的解析ツールで特定のワーニングは0件にせよ、とか単体テストで全分岐の組み合わせ必ず通せ、だの。ツールや標準は決まりきった使い道や特定範囲の入力しかない(今後増える見込みもない)モジュール内の特定用途の下位関数と、他モジュールや人間とのI/F絡んでかなり異常値入力を警戒すべき関数の意味の違いを理解してくれない。やれelse節やswitchのdefaultがないだの引数NULLチェックしろだの金輪際使用されない予定の死んだコードを大量にソースに投入することでかえって見通しが悪くなったりバグ混入の機会が増えたり速度が遅くなったりバイナリサイズが膨れ上がったり...
×杓子定規○金科玉条
しかし、あんたが書いた内容ならレビュアーの言ってることの方が正しい。すくなくとも、職業プログラマーが企業システムを作るなら当然のことばかりだが。
個別にいちいち指摘はしなけど、switchにdefaultがない、と言われたら、throw new Exception('これが出たオレはプログラマーやめる’);とでも書いておけ。(すぐやめることになると思うし、とっとと辞めた方がいい)
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
私は悩みをリストアップし始めたが、そのあまりの長さにいやけがさし、何も考えないことにした。-- Robert C. Pike
みんな、勝手な書き方しやがって… (スコア:0)
コーディング標準が設定され、守られていればそんなことは起こらないはずだ。
Re: (スコア:0)
あまり杓子定規なのも困り者。
例えば静的解析ツールで特定のワーニングは0件にせよ、とか
単体テストで全分岐の組み合わせ必ず通せ、だの。
ツールや標準は決まりきった使い道や特定範囲の入力しかない
(今後増える見込みもない)モジュール内の特定用途の下位関数と、
他モジュールや人間とのI/F絡んでかなり異常値入力を警戒すべき
関数の意味の違いを理解してくれない。
やれelse節やswitchのdefaultがないだの引数NULLチェックしろだの
金輪際使用されない予定の死んだコードを大量にソースに投入することで
かえって見通しが悪くなったりバグ混入の機会が増えたり速度が遅くなったり
バイナリサイズが膨れ上がったり...
Re:みんな、勝手な書き方しやがって… (スコア:2, すばらしい洞察)
×杓子定規
○金科玉条
しかし、あんたが書いた内容ならレビュアーの言ってることの方が正しい。
すくなくとも、職業プログラマーが企業システムを作るなら当然のことばかりだが。
個別にいちいち指摘はしなけど、
switchにdefaultがない、と言われたら、
throw new Exception('これが出たオレはプログラマーやめる’);
とでも書いておけ。
(すぐやめることになると思うし、とっとと辞めた方がいい)