■http://anond.hatelabo.jp/20090809183815

提示したコードがアレ過ぎ。

背景も知らないし調べもせずにそんなコードをとりあえず書き換えるならこんな感じ。

適当に自分の好みを入れているのでツッコミ入れたくなる人もいるだろうけど。

int rc = RC_SUCCESS;
boost::shared_ptr<ObjA&gt; a = createA();
if (!a)
  rc = RC_ERROR1;
if (rc == RC_SUCCESS) {
  boost::shared_ptr<ObjB&gt; b = createB();
  if (!b)
    rc = RC_ERROR2;
}
return rc;

ちょっと言葉が汚いけど簡単に書くと

• そこでgotoは要らないだろ
• 型名がObjAとかObjBとかなにそれ？
• int型でRC_なんちゃらじゃなくて列挙型使えよ（intでしょうがないか、というケースはあるけど）
• createAとかcreateBってインスタンスを作っぽいけどそれで合ってる？その上でなんでそれを使ってないの？何をやりたいか分かり難いよ。設計おかしくない？

Permalink | 記事への反応(2) | 20:51

ツイートシェア