try-catch-finallyブロックを構成して、最終的にエラーを処理するにはどうすればよいですか？

Question

私は、その関数にアクセスするために、DllImportを使用してクラスにラップしたサードパーティのC++ dllへの呼び出しに問題があります。

dllは、使用前にセッションが開かれることを要求します。これは、操作を実行するときにそのセッションを参照するために使用される整数ハンドルを返します。終了したら、同じハンドルを使用してセッションを終了する必要があります。だから私はこのような何かをした：

public void DoWork(string input) 
{ 
    int apiHandle = DllWrapper.StartSession(); 

    try 
    { 
     // do work using the apiHandle 
    } 
    catch(ApplicationException ex) 
    { 
     // log the error 
    } 
    finally 
    { 
     DllWrapper.CloseSession(apiHandle); 
    } 
} 

私が持っている問題はCloseSession（）は時々、ねじを実行しているときにエラーをスローする問題のDLLを引き起こすことがある：

System.AggregateException：1つ以上のエラーが発生しました。 ---> System.AccessViolationException：保護された メモリを読み書きしようとしました。これはしばしば、他のメモリが壊れていることを示します。

Dllをスレッド方式で使用しているように見えるので、私はこのエラーを停止することができません。スレッドセーフであると思われます。しかし、私のCloseSession（）関数は、Dllのクローズ関数を呼び出す以外は何もしないので、何かを "修正"する余裕があまりありません。

しかし、最終的にはセッションが正常に終了しないことになります。だから、プロセスがやり直さなければならないプロセスは、オープンセッションに遭遇し、新しいエラーを投げつけ続けます。そのセッションは絶対にに閉鎖されるを持っています。

エラー処理ステートメントを設計する方法について、私は紛失しています。これは、セッションが常に閉じることを保証するものです。

Answer 1

私は、外部リソースの廃棄を含むようにラッパーを変更し、ハンドルもラップします。私。ハンドルでセッションを表現するのではなく、ラッパーオブジェクトで表現します。

さらに、DLLへの呼び出しをlock-statsements（@ Sergeの示唆）にラップすると、マルチスレッドの問題を完全に防ぐことができます。すべてのDllWrappersが同じロックオブジェクトを使用するように、ロックオブジェクトは静的であることに注意してください。

public class DllWrapper : IDisposable 
{ 
    private static object _lockObject = new object(); 

    private int _apiHandle; 
    private bool _isOpen; 

    public void StartSession() 
    { 
     lock (_lockObject) { 
      _apiHandle = ...; // TODO: open the session 
     } 
     _isOpen = true; 
    } 

    public void CloseSession() 
    { 
     const int MaxTries = 10; 

     for (int i = 0; _isOpen && i < MaxTries; i++) { 
      try { 
       lock (_lockObject) { 
        // TODO: close the session 
       } 
       _isOpen = false; 
      } catch { 
      } 
     } 
    } 

    public void Dispose() 
    { 
     CloseSession(); 
    } 
} 

ここで、メソッドはインスタンスメソッドであることに注意してください。

今、あなたはusingステートメントとのセッションの閉鎖を確保することができます。

using (var session = new DllWrapper()) { 
    try { 
     session.StartSession(); 
     // TODO: work with the session 
    } catch(ApplicationException ex) { 
     // TODO: log the error 
     // This is for exceptions not related to closing the session. If such exceptions 
     // cannot occur, you can drop the try-catch completely. 
    }  
} // Closes the session automatically by calling `Dispose()`. 

あなたはこのクラスSessionとメソッドOpenとCloseを呼び出すことによって命名向上させることができます。このクラスのユーザーは、ラッパーであることを知る必要はありません。これは単なる実装の詳細です。また、メソッドの命名は対称的であり、名前Sessionを繰り返す必要はありません。

エラー処理、エラー状況からの回復、リソースの廃棄など、すべてのセッション関連の要素をカプセル化することで、コード内の混乱を大幅に減らすことができます。 Sessionクラスは現在、高度な抽象クラスです。古いDllWrapperは、低レベルと高レベルの中間のどこかにありました。