良いコードを書くためのノウハウを蓄積するスレ
■ このスレッドは過去ログ倉庫に格納されています
0096デフォルトの名無しさん
NGNGループの場合はcontinue、ifやswitchの場合はその部分を関数に切り出して
returnさせるようにしてるんですが、こういうのってどうなんでしょ?自分では
読みやすい(コメントも付けてるし)良いコードだと思ってるんですが、読みにくかったり
するんでしょうか?
for (SmplCls hoge in arg) {
/* expr1がfalseの場合をふるい落とす */
if (!hoge.expr1)
continue;
/* expr2の文字列をチェック */
if (!(hoge.expr2 == "hogestr")
continue;
/* expr3が限界値を超えていればふるい落とす */
if (limit < hoge.expr3)
continue;
/* 実処理 */
/* ... */
}
どうせならループの場合も関数に切り出したほうがいいかとも思うんですが、結局
エラー・例外処理が絡んできて値を返し、呼び出し元で分岐させなければならなく
なるのでそこまではやってません。
■ このスレッドは過去ログ倉庫に格納されています