如何构造一个 try-catch-finally 块来处理 finally 内部的错误？

Question

我在调用第三方 C++ dll 时遇到问题，我使用 DllImport 将其封装在一个类中以访问其功能。

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：发生一个或多个错误。 ---> System.AccessViolationException：试图读或写保护 记忆。这通常表明其他内存已损坏。

我不确定我能做些什么来阻止这个错误，因为它似乎是由以线程方式使用 Dll 引起的——它应该是线程安全的。但是由于我的 CloseSession() 函数除了调用 Dll 的 close 函数之外什么都不做，所以我没有太多的回旋余地来“修复”任何东西。

然而，最终结果是会话没有正确关闭。因此，当该进程再次尝试时，它应该这样做，它会遇到一个打开的会话并不断抛出新的错误。该会话绝对必须关闭。

我不知道如何设计一个更强大的错误处理语句来确保会话始终关闭？

Answer 1

我会更改包装器以包括对外部资源的处理并同时包装句柄。 IE。您可以通过包装对象来表示会话，而不是通过句柄来表示会话。

此外，在lock-statements 中包装对DLL 的调用（正如@Serge 所建议的那样），可以完全防止多线程问题。请注意，锁对象是静态的，因此所有 DllWrapper 都使用相同的锁对象。

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 介于低级和高级之间。