WebBrowser 控件 - 等待文档完成扩展方法

Question

我正在为WebBrowser 控件创建一个扩展方法，其目的是等待页面完成完全加载然后再返回。

完全加载，我的意思是在最后一个DocumentCompleted 事件之后必须经过一秒钟。这是为了考虑在页面加载期间触发多个 DocumentCompleted 事件的页面（这目前正在影响我的应用程序）。

我已经编写了当前的方法，它似乎可以工作，但我觉得可以改进：它在可能不需要时返回一个任务，并且它还存在如果应用程序关闭则会引发异常的问题在等待页面加载时。

    public static Task<bool> WaitLoad(this WebBrowser webBrowser, int wait)
    {
        var timerInternalWait = new Timer {Interval = 1000, Tag = "Internal"};
        var timerMaxWait = new Timer {Interval = wait};
        var tcs = new TaskCompletionSource<bool>();

        WebBrowserNavigatingEventHandler navigatingHandler = (sender, args) => timerInternalWait.Stop();
        webBrowser.Navigating += navigatingHandler;

        WebBrowserDocumentCompletedEventHandler documentCompletedHandler = (sender, args) => { timerInternalWait.Stop(); timerInternalWait.Start(); };
        webBrowser.DocumentCompleted += documentCompletedHandler;

        EventHandler timerHandler = null;
        timerHandler = (sender, args) =>
        {
            webBrowser.Navigating -= navigatingHandler;
            webBrowser.DocumentCompleted -= documentCompletedHandler;
            timerInternalWait.Tick -= timerHandler;
            timerMaxWait.Tick -= timerHandler;
            timerMaxWait.Stop();
            timerInternalWait.Stop();

            tcs.SetResult(((Timer) sender).Tag.ToString() == "Internal");
        };

        timerInternalWait.Tick += timerHandler;
        timerMaxWait.Tick += timerHandler;

        return tcs.Task;
    }

有什么可以改进的吗？

Answer 1

The design guidelines state that asynchronous methods should be suffixed.

那个计时器让你的代码更难阅读。试试这个：

public static Task WaitLoad(this WebBrowser webBrowser)
{
    var tcs = new TaskCompletionSource<object>();

    WebBrowserNavigatingEventHandler navigatingHandler = (sender, args) => timerInternalWait.Stop();
    webBrowser.Navigating += navigatingHandler;

    WebBrowserDocumentCompletedEventHandler documentCompletedHandler = (sender, args) => tcs.SetResult(null);

    try
    {
        webBrowser.DocumentCompleted += documentCompletedHandler;

        await tcs.Task;
    }
    finally
    {
        webBrowser.DocumentCompleted -= documentCompletedHandler;

        await Task.Delay(1000);
    }

    return tcs.Task;
}

public static Task<bool> WaitLoad(this WebBrowser webBrowser, int timeout)
{
    var webBrowserTask = webBrowser.WaitLoad();

    return await Task.WhenAny(webBrowserTask, Task.Delay(timeout)) == webBrowserTask;
}

但您应该查看取消令牌而不是双计时器。