已处理表单上的 C# 异步回调

Question

快速提问：我的 winform 应用程序 (c#) 中的一个表单对 WCF 服务进行异步调用以获取一些数据。如果表单恰好在回调发生之前关闭，它会因访问已处置对象的错误而崩溃。检查/处理这种情况的正确方法是什么？该错误发生在对更新表单的方法的 Invoke 调用中，但我无法深入了解内部异常，因为它表示代码已被优化。

代码：

公共无效RequestUserPhoto（int userID） { WCF.Service.BeginGetUserPhoto（用户 ID， 新的 AsyncCallback（GetUserPhotoCB），用户 ID）； }

    public void GetUserPhotoCB(IAsyncResult result)
    {
        var photo = WCF.Service.EndGetUserPhoto(result);
        int userID = (int)result.AsyncState;
        UpdateUserPhoto(userID, photo);
    }

    public delegate void UpdateUserPhotoDelegate(int userID, Binary photo);
    public void UpdateUserPhoto(int userID, Binary photo)
    {
        if (InvokeRequired)
        {
            var d = new UpdateUserPhotoDelegate(UpdateUserPhoto);
            Invoke(d, new object[] { userID, photo });
        }
        else
        {
            if (photo != null)
            {
                var ms = new MemoryStream(photo.ToArray());
                var bmp = new System.Drawing.Bitmap(ms);
                if (userID == theForm.AuthUserID)
                {
                    pbMyPhoto.BackgroundImage = bmp;
                }
                else
                {
                    pbPhoto.BackgroundImage = bmp;
                }
            }
        }
    }

更新：

我仍然不知道该去哪里。我在这里真正需要的是一种设计模式，用于从 win 表单进行 WCF 异步服务调用，该模式在异步调用返回之前可以优雅地处理表单关闭。用户可以随时单击表单或任何表单上的 X。这个问题比我上面展示的单个例子要大得多。我的应用程序实际上进行了数百次 WCF 调用，我试图弄清楚如何优雅地处理这些，并且在整个应用程序中以一致的方式。例如，如果我必须为每个 WCF 调用添加 100 行带有 ManualResetEvents 或后台工作人员或互斥锁或其他任何东西的代码，这样它就不会轰炸我的应用程序，这将引入很大的错误空间。我需要的是一种干净的方式来异步调用服务，然后在表单关闭时将其转换为单向调用。也就是说，让服务跑完，但我不管结果是什么，也不要调用回调，因为它已经不存在了。

Answer 1

public void UpdateUserPhoto(int userID, Binary photo)
{
  if ( Disposed || !IsHandleCreated )
    return;
  if (InvokeRequired)
  ...

Answer 2

我相信 Form 对象上有一个 IsDisposed 属性。您可以在调用 Invoke 之前检查该属性。

Answer 3

创建一个标志，例如“IsWaiting”作为 ManualResetEvent（甚至是一个简单的布尔值），将其设置为 true，并且仅在异步结果返回时将其设置为 false。

在您的类 dispose 方法中检查标志，并且仅在标志清除后才释放对象。 （设置一个超时，以防万一出现错误）

Answer 4

当操作完成时，你给回调一个地址。关闭表单时地址无效，因此您收到此错误。在允许表单关闭之前，您需要确定是否有未完成的呼叫。

我会研究BackgroundWorker class。我相信你仍然可以通过在回调触发之前关闭你的表单来设置崩溃，但是使用BackgroundWorker，你应该能够询问它的状态并更优雅地处理你的情况。

您不应在 Async 调用仍处于活动状态时关闭表单。使用BackgroundWorker，您可以轻松确定 Async 调用是否处于活动状态。

Answer 5

我在这个问题上获得了很多优秀的 cmets，但没有可以在整个应用程序中使用的良好模式。我在这里发布了我最终做的答案，希望它可以帮助其他人。

自动生成的 WCF 代理创建同步调用方法、使用开始/结束模式的异步方法和使用 Completed 事件的基于事件的委托。我在上面的原始问题中发布的示例使用了 Begin/End 模式。问题是当回调发生时，你将不得不做一个 Invoke 来访问你的 UI 线程。如果该 UI 线程不再存在（即用户关闭了窗口），那么您就有问题了。新的基于事件的模式会自动找到返回 UI 线程的方式，并且根据我的测试，我无法通过在服务完成之前关闭来使其崩溃。我猜代理足够聪明，如果内存地址不存在，不会调用完成的处理程序？或者它可能挂在处理程序的内存地址上，所以它不会被垃圾收集？因此，您所要做的就是添加一个 Completed 事件处理程序，启动您的调用等 servicenameAsync()，然后等待它返回到您的处理程序（在 UI 线程上）。我还为了确保将完成的处理程序包装在 try/catch 块中以处理 ObjectDisposedExceptions（如果有）。

现在最重要的是：它不会崩溃。

一个问题（至少对我而言）...我使用单例模式来访问我的 WCF 服务。这样做的问题是它创建了对 WCF 代理的静态引用，在整个应用程序中使用。听起来很方便，但是当您在进行异步调用之前将事件处理程序附加到 Completed 事件时，这些事件可能会被重复、三次等。每次进行调用时，除了一个之外，您还会添加另一个已完成的事件处理程序已添加到您的静态 WCF 代理。为了解决这个问题，我开始为每个调用声明一个新的代理客户端，或者如果每个 winform 类没有进行多个调用，则每个 winform。如果有人对此有任何 cmets 或更好的方法，请告诉我！单例模式的最大问题是，如果您同时打开多个窗口（不同的类）调用相同的异步方法，但您希望它们返回不同的 Completed 处理程序，则不能这样做。