Dictionaryにキーが含まれていない場合は例外をスローすることはできますか

Question

オブジェクトからプロパティを取得し、pdfに値を設定するコードを記述しています。

私は私の辞書からアイテムを取得するために使用するコードは以下の通りです：辞書がどのように見える

 public static int? GetYPrintCoord(string word) 
     { 
      // Try to get the result in the static Dictionary 
      PrintCoord result; 
      if (Dict.TryGetValue(word, out result)) 
      { 
       return result.Y; 
      } 
      else 
      { 
       return null; 
      } 
     } 

：

Dictionary<string, PrintCoord> Dict = new Dictionary<string, PrintCoord>(); 

事は、私はヌルをチェックするのはうんざりだときGetYPrintCoord(string word)が呼び出されます。実際には、辞書内の項目が存在しない場合、誤ったデータが含まれるため、pdf全体が生成する価値がありません。私は方法を変更する代わりに、考えていた：

public static int GetYPrintCoord(string word) 
    { 
     // Try to get the result in the static Dictionary 
     PrintCoord result; 
     if (Dict.TryGetValue(word, out result)) 
     { 
      return result.Y; 
     } 
     else 
     { 
      throw new KeyNotFoundException(); 
     } 
    } 

これは悪い習慣ですか？例外を投げてクラッシュさせると、実際にはユーザーの方がずっと良くなるでしょう。なぜなら、少なくともユーザーは無効なpdfを見ず、それが無効であることを知らないかもしれないからです。キーが見つからない

PrintCoord result = Dict[word]; // is equivalent to your code. 

場合は、例外がスローされます。あなたは、例外をスローする前に、いくつかのロギングを行う必要がない限り、あなたは車輪の再発明している

Answer 1

が、これは組み込みの動作です。

したがって、キーが辞書に見つからないと思われる場合、基本的にはTryGetValueを使用しますが、それでも問題はありません。 2番目のケースは、try...catchを使用せず、単純なif文を使用してキーが見つからない場合を処理する場合です。

Answer 2

例外は、名前が示すように、例外的な場合を表します。

辞書にキーが見つからなかったということは、何かが間違っていた場合、つまりがになるはずのものではない場合は、例外が当てはまる可能性があります。

つまり、必要なキーが見つからないことが予想される場合、例外が意味をなさない可能性があります。できることは、GetYPrintCoordメソッドの名前をTryGetYPrintCoordと変更することができます。次に、メソッドがブール値を返し、キーが見つかった場合はTrue、それ以外の場合はFalseを返し、outキーワードを使用して辞書の値を戻します。