C++/CLI優れたコーディング慣行、インデックス付きプロパティif-checkまたはtry..catch？

Question

かなり短く簡単な質問です。

のは、コードのこの部分を使用してみましょう：

public ref class Foo 
{ 
private: 
    System::Collections::Generic::Dictionary<System::String ^,System::String ^>^aDictionary; 

public: 

    property System::String^SomeIndexedProperty[System::String ^] 
    { 
    public: System::String^get(System::String^index) 
      { 
       return aDictionary[index]; 
      } 
    } 

public: 
    Foo(void) 
    { 
     aDictionary = gcnew System::Collections::Generic::Dictionary<System::String ^,System::String ^>(); 
    } 
}; 

if(aDictionary->ContainsKey(index))（if文で/事前チェックのリターンを囲むように良いだろうか、試してみるとreturn文を囲むように良いだろう。 .catchブロック？彼らは失敗したときに、両方のケースで

はnullptrを返す。

スピードは本当に問題のではありません。しかし、単に一般的な十分であろう「これがその理由のために優れている」。

Answer 1

合理的に許可されている法的条件がある場合、それを検出するために例外を検出すべきではないと確信しています。つまり、ifステートメントを使用します。これは、あなたがArrayIndexOfBoundsExceptionになるまで配列のすべてのループを使い果たし、それを忘却に捕まえようとするのと同じです。

関連するノートでは、nullを返すのではなく、プロパティがKeyNotFoundExceptionをスローすることが考えられます。呼び出し側はこれをnullとし、逆参照しようとすると、フォーカスを移動してバグを見つけにくくなります。

Answer 2

あなたの状況を説明すると、あなたの好みにのみ依存すると思います。 とにかく、失敗条件が確定的で、予期できる場合は、try..catchブロックを使用しない方が良い解決策であり、非確定的なエラーと予測できないエラーの場合にのみ残してください。

"if"ブロックの代わりに例外に対するすべての議論は、パフォーマンス（スピード、メモリ、スタック、eccなど）についてのみであり、accademicです。本当の意味では、完全に例外を除いたメソッドが必要で、戻り値がnullの理由を気にしない場合は、5行をtry..catchブロックにして忘れてください！ ;）