アカウント名:
パスワード:
処理が1行だからといって、if文で格好つけるのはやめましょう。
括弧の有無でパーサが限界に達したりするから簡単な問題じゃないんだよif(~!=0);else if(~!=0)~if(~!=0);else{/*SUCCESS*/}/*FAILURE*/;return err;みたいなの
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0) goto fail;
を
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0) { goto fail; }
にしちゃいけない理由なのか?
条件式の後に単文を実行する場合はブレースを付けないコーディング規約なのかも知れません。
私はかつて、最初にプログラマとして勤めた職場が「ブレース省略禁止」で叩き込まれたこともあり、ブレース省略の利点がさっぱりわからない(こういうバグを生む)のですが、なぜか「単文ではブレース省略」とする規約は見たことがあります。
不要なコードは書くな主義かな。ソースコードのサイズを制限したいとか#そういう時代もあったんです
あるいはK&Rスタイルに沿っているつもりとか。
単文にはそもそも中括弧は不要だからでしょう。つまり「省略」じゃない。Syntax Sugarの一種で水増しだといわれても仕方がない面があるから。
IFの実行部は必ずブロックにしろ、単文は使うなってのは30年以上前から言われてますけどね。
より多くのコメントがこの議論にあるかもしれませんが、JavaScriptが有効ではない環境を使用している場合、クラシックなコメントシステム(D1)に設定を変更する必要があります。
UNIXはただ死んだだけでなく、本当にひどい臭いを放ち始めている -- あるソフトウェアエンジニア
格好つけずに括弧つけろよ (スコア:5, おもしろおかしい)
処理が1行だからといって、if文で格好つけるのはやめましょう。
Re: (スコア:0)
括弧の有無でパーサが限界に達したりするから簡単な問題じゃないんだよ
if(~!=0);else if(~!=0)~if(~!=0);else{/*SUCCESS*/}/*FAILURE*/;return err;
みたいなの
Re: (スコア:0)
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0)
goto fail;
を
if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0)
{
goto fail;
}
にしちゃいけない理由なのか?
Re: (スコア:1)
条件式の後に単文を実行する場合はブレースを付けないコーディング規約なのかも知れません。
私はかつて、最初にプログラマとして勤めた職場が「ブレース省略禁止」で叩き込まれたこともあり、ブレース省略の利点がさっぱりわからない(こういうバグを生む)のですが、なぜか「単文ではブレース省略」とする規約は見たことがあります。
Re:格好つけずに括弧つけろよ (スコア:2)
不要なコードは書くな主義かな。
ソースコードのサイズを制限したいとか
#そういう時代もあったんです
あるいはK&Rスタイルに沿っているつもりとか。
単文にはそもそも中括弧は不要だからでしょう。
つまり「省略」じゃない。
Syntax Sugarの一種で水増しだといわれても仕方がない面があるから。
IFの実行部は必ずブロックにしろ、単文は使うなってのは30年以上前から言われてますけどね。