bool / intを返し、実際のオブジェクトを出力パラメーターとして持つメソッドがあるのはなぜですか？

私の会社のコードベース（.NET 3.5アプリケーション）のいたるところに次のコードパターンがあります。

bool Foo(int barID, out Baz bazObject) { 
    try { 
            // do stuff
            bazObject = someResponseObject;

            return true;
    }
    catch (Exception ex) { 
        // log error
        return false;
    }
}

// calling code
BazObject baz = new BazObject();
fooObject.Foo(barID, out baz);

if (baz != null) { 
    // do stuff with baz
}

Fooメソッドが単にIDを取得してBazオブジェクトを返す代わりに、使用されていない値を返して実際のオブジェクトをrefまたは出力パラメーターにするのではなく、なぜこれを行うのかを頭に入れようとしています。

私が見逃しているこのコーディングスタイルの隠された利点はありますか？

通常、このパターンを使用して、次のようなコードを記述できます。

if (Foo(barId, out bazObject))
{
  //DoStuff with bazobject
}

たとえば、辞書クラスのTryGetValueのCLRで使用されます。それはいくつかの冗長性を回避しますが、outおよびrefパラメーターは常に少し厄介に思えました

これは、例外が発生する前の古いCスタイルのコードです。戻り値は、メソッドが成功したかどうかを示し、成功した場合は、パラメーターに結果が書き込まれます。

.netでは、そのための例外があります。このパターンに従う理由はないはずです。

[編集]例外処理には明らかにパフォーマンスへの影響があります。たぶんそれは何か関係があるのでしょう。ただし、そのコードスニペットには、既に例外がスローされています。より適切な場所に引っかかるまで、スタックを上に移動させる方が簡単です。

コードスニペットを考えると、完全に無意味に見えます。私にとって、最初のコードパターンは、BazObjectのnullが許容されるケースであり、boolの戻り値がフェイルケースを安全に判断する手段であることを示唆しています。次のコードが：

// calling code
BazObject baz = new BazObject();
bool result = fooObject.Foo(barID, out baz);

if (result) { 
    // do stuff with baz
    // where baz may be 
    // null without a 
    // thrown exception
}

これは、この方法で行う方が理にかなっています。おそらくこれは、オブジェクトパラメータがC＃で実際にどのように機能するかを理解せずに、bazが参照で渡されることを保証するために使用する前の誰かのメソッドです。

呼び出し側が、返された型が参照型であるか値型であるかを気にする必要がない場合に、このパターンが役立つことがあります。このようなメソッドを呼び出して値型を取得する場合、その値型には確立された無効な値（例：double.NaN）が必要か、成功を判断する他の方法が必要です。

このアイデアは、プロセスが成功したかどうかを示す値を返し、次のようなコードを許可することです。

Baz b;
if (fooObject.foo(id, out b)) {
   // do something with b
}
else {
   Error.screamAndRun("object lookup/whatever failed! AAaAAAH!");
}

オブジェクトをnullにできない場合（この例では正しいように見えます）、次のように実行することをお勧めします。

Baz b = fooObject.foo(id);
if (b != null) {
   // do something with b
}
else {
   Error.screamAndRun("object lookup/whatever failed! AAaAAAH!");
}

オブジェクトが nullになる可能性がある場合、例外を使用する方法があります。

try {
   Baz b = fooObject.foo(id);
}
catch (BazException e) {
   Error.screamAndRun("object lookup/whatever failed! AAaAAAH!");
}

これは、例外のないCで使用される一般的なパターンです。関数がエラー状態を返すことができます。例外は一般的にはるかにクリーンなソリューションです。