アカウント名:
パスワード:
警告を無視しちゃいかんという教訓を得たと。
こんな感じに、if分を一行にまとめてしまえば、こんな不具合出なかった気がします。
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0) goto fail;if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0) goto fail;if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0) goto fail;
一行が長くなるのを嫌ったのかな?
一行にまとめるコーディングは嫌いですし、職業プログラマーはあまり使うべきではないとも思っています。なので対処はif分の処理が1行でも必ず括弧で括る事です。
四角四面なルールに沿って行動することで、プログラマは脳みそがルーチンワークに最適化されて、脳みそを使わず脊髄で仕事をするようになる。
それが最悪の結果をもたらす。
コーダーがルーチンワークに最適化されずにどうする?勘違いしてないか。
コーダーなんて役割を分業している開発体制に問題がある。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
犯人はmoriwaka -- Anonymous Coward
unreachable codeの警告は無視されたのか (スコア:0)
警告を無視しちゃいかんという教訓を得たと。
こんな感じに、if分を一行にまとめてしまえば、こんな不具合出なかった気がします。
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0) goto fail;
if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0) goto fail;
if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0) goto fail;
一行が長くなるのを嫌ったのかな?
Re: (スコア:3, すばらしい洞察)
一行にまとめるコーディングは嫌いですし、職業プログラマーはあまり使うべきではないとも思っています。
なので対処はif分の処理が1行でも必ず括弧で括る事です。
Re: (スコア:0)
四角四面なルールに沿って行動することで、
プログラマは脳みそがルーチンワークに最適化されて、
脳みそを使わず脊髄で仕事をするようになる。
それが最悪の結果をもたらす。
Re: (スコア:0)
コーダーがルーチンワークに最適化されずにどうする?
勘違いしてないか。
Re:unreachable codeの警告は無視されたのか (スコア:0)
コーダーなんて役割を分業している開発体制に問題がある。