c# - C#で複数のエラーを処理するには?

Question

10個のレジストリキーを読み取るコードがいくつかあります。値が存在しない場合、キーが存在しない場合、値がブール値でない場合などがあります。これにエラー処理を追加するにはどうすればよいですか。現在、1つの大きな試行に配置されています{} catch{} ですが、2 番目に読み取った値が失敗した場合、プログラムが catch{} にジャンプするため、残りは読み取られません。それぞれに try{} catch{} を追加できますが、もっと良い方法があると思います. これをどのように処理しますか？似たような問題に定期的に遭遇するので質問しています。私自身の解決策は、try{} catch{} を追加することです。

助けてくれてありがとう。

score 2 · Accepted Answer

まず、例外を飲み込むことは一般的に悪い考えです。キーなどの存在をチェックし、存在する場合は値を返すメソッドを作成できませんか?

それが絶対に不可能な場合は、(それぞれに対して) try/catch (飲み込み) を行う単一のメソッドへの複数の呼び出しにコードをリファクタリングできます。

SomeReturnType foo = HackyMethod(first path);
SomeReturnType bar = HackyMethod(sedond path);

SomeReturnType HackyMethod(string path)
{
    try {} catch {} etc
}

score 1 · Accepted Answer

Dictionary<String,String> regKeys = new Dictionary<String,String>()
{
    { "Key1", String.Empty},
    { "Key2", String.Empty},
    { "Key3", String.Empty}
};

for (int i = 0; i < regKeys.Length; i++)
{
   try
   {
       regKeys[i].Value = ReadFromRegistry(regKeys[i].Key);
   }
   catch (Exception ex)
   {
      Console.WriteLine("Unable to read Key: " + regKeys[i].Key 
         + " Exception: " + ex.Message);
   } 
}

score 1 · Accepted Answer

レジストリ値をどのように読んでいますか? Registry クラス (Microsoft.Win32.Registry) を使用すると、次のように、レジストリ値を読み取って、値と名前のペアが存在しない場合に指定した既定値を返すことができます。

object o = Microsoft.Win32.Registry.GetValue(
    @"HKEY_CURRENT_USER\Software\Microsoft\Calc", "layout", "");

ここでの最後のパラメータは、値の名前が見つからない場合に返されるように指定されたデフォルトです。空白文字列にしましたが、これは好きなように変更できます。

score 0 · Accepted Answer

@Marcの答えが最善ですが、エラーが発生したレジストリキーのコレクションを含む単一の例外が絶対に必要な場合は、例外のDataプロパティを使用して確認する必要があります。このプロパティに関するMSDNドキュメントから、

Dataプロパティによって返されるSystem.Collections.IDictionaryオブジェクトを使用して、例外に関連する補足情報を格納および取得します。情報は、任意の数のユーザー定義のキー/値ペアの形式です。各キー/値ペアのキーコンポーネントは通常、識別文字列ですが、ペアの値コンポーネントは任意のタイプのオブジェクトにすることができます。

score 0 · Accepted Answer

値を読み取るコードを、必要な方法でエラーを処理する独自の関数にリファクタリングします。

score 0 · Accepted Answer

基本的に、はい、要素のセットでエラー処理を定義するのではなく、個々の要素でエラー処理を定義する必要があります。つまり、各エラーをキャッチしてプロセスを中止させたくない場合は、グループ全体ではなく、個々の要素に対してエラー処理を実行する必要があります。

score 0 · Accepted Answer

これは、エラーの重大度によって異なります。検索するキーの一部が欠落しているか、タイプが間違っている場合、プログラムが続行することは意味があり、有用ですか? 他のキーよりも重要なキーはありますか?

次のことをお勧めします。

必要なすべてのキーを見つけて、それらを単一の try{} に入れ、致命的なエラーを報告してクリーンアップを開始する catch{} を使用します。このブロックを最初に実行します。
オプションのキーをすべて見つけて、それぞれを独自の try{} ブロックに配置します。これにより、回復して他のキーに進むことができます。これを簡単にするために、必要な try/catch ブロックとエラーチェックを持ち、キー名をパラメーターとして受け取るラッパーメソッドを追加できます。

score 0 · Accepted Answer

編集：全体を変更しました。:PI は (以前は) 構造体またはクラスを提案していましたが、現在はそれを単純な文字列コレクションに変更しています。

頭のてっぺんからいくつかの疑似コード....

public IEnumerable<string> ReadRegistryKeys()
{
    IEnumerable<string> resultList = new List<string>();
    if (string.IsNullOrEmpty(read_in_key_#1())
    {

        resultList.Add("Failed to load key 'blah'..");
    }

    if (.... read in the next key .. etc.... ) ...

    return resultList == null || resultList.Count <= 0 ? null : resultList;
}

必要に応じて、StringCollection (System.Collections.Specialized?) も使用できます。

c# - C#で複数のエラーを処理するには?

8 に答える 8

Related

Reference