アカウント名:
パスワード:
警告を無視しちゃいかんという教訓を得たと。
こんな感じに、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行でも必ず括弧で括る事です。
スタイルの問題ではないでしょこれは。こういう重要な箇所くらいテストで100%のカバレッジを確保しろよと。どーでもいいとこに心血注ぐ必要はないけどさ。
gotoとか使うとか{}を書かないとかするなら、全パターン網羅しろって感じではあります。
だよね、カバレッジ取れば実行されないことが一発で分かる。どこまでのテストをしているのか分からないけど、品質管理は相当お粗末なのだろうな。
いいえ、コーディングスタイルとテストの両方が必要です。
テストもコードであるかぎり必ず漏れが発生するよ。常に完璧なテストコードを書けるようならそもそも最初のバグ自体入らん。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
ソースを見ろ -- ある4桁UID
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:unreachable codeの警告は無視されたのか (スコア:1)
スタイルの問題ではないでしょこれは。
こういう重要な箇所くらいテストで100%のカバレッジを確保しろよと。
どーでもいいとこに心血注ぐ必要はないけどさ。
Re: (スコア:0)
gotoとか使うとか{}を書かないとかするなら、全パターン網羅しろって感じではあります。
Re: (スコア:0)
だよね、カバレッジ取れば実行されないことが一発で分かる。
どこまでのテストをしているのか分からないけど、品質管理は相当お粗末なのだろうな。
Re: (スコア:0)
いいえ、コーディングスタイルとテストの両方が必要です。
Re: (スコア:0)
テストもコードであるかぎり必ず漏れが発生するよ。
常に完璧なテストコードを書けるようならそもそも最初のバグ自体入らん。