这是定期异步下载的好代码吗？

Question

我正在使用 Windows 服务定期检查更新。就像一两​​分钟。为此，我使用 .net 4.6.1 和

1. 系统定时器，&
2. 任务

下面是我的代码

    /// <summary>
    /// Download updates from server
    /// </summary>
    /// <returns></returns>
    private void DownloadUpdate()
    {
        try
        {
            var package = string.Empty;
            if (_server != null)
            {
                timer.Stop();
                package = _server.Download();
                if (package.Length > 0)
                    _server.Install(package);
            }
        }

        finally
        {

            timer.Start();

        }

    }

    /// <summary>
    /// To carry out download asynchronous way
    /// </summary>
    private async void DowloadUpdateAsync()
    {
       await Task.Run(()=>DownloadUpdate());
    }

只是想知道，如果它是管理间隔的正确方法。

Answer 1

1. 对于async 方法，如果您不希望它返回结果，请返回Task。见this link。

2. Task.Run(()=>DownloadUpdate());可以简化为Task.Run(DownloadUpdate);

3. 我真的看不出有什么理由需要将任务包装在 async 方法中。由于DownloadUpdate 不是异步的，它无论如何都会阻塞一个线程，你需要做的就是将它作为Task 运行或者直接将它排队到线程池中。

Answer 2

补充 Lifu 的回答：

我建议，您应该在 ContinueWith() 中启动计时器。

TPL 使用垃圾收集器的终结机制来发现失败的任务何时被放弃，导致异常未观察到。发生这种情况时，TaskScheduler 类会引发其 UnobservedTaskException 事件。最好处理您的异常。这是使用 ContinueWith 时的优势。

试试这样的：

    Timer timer = new Timer();
    public Program()
    {
        timer.Interval = 5000;
        timer.Elapsed += DownloadUpdate;
        timer.Enabled = true;
    }

    private void DownloadUpdate(object sender, ElapsedEventArgs e)
    {
        Task.Factory.StartNew((Action)DownloadUpdate).ContinueWith(t =>
        {
            bool shouldTimerStartAgain = true;
            if (t.IsFaulted)
            {
                // handle t.Exception
                // Compute if this task need to stop?
                // If so set shouldTimerStartAgain = false
            }
            if (shouldTimerStartAgain)
            {
                timer.Start();
            }
        });
    }

    private void DownloadUpdate()
    {
        var package = string.Empty;
        if (_server != null)
        {
            timer.Stop();
            package = _server.Download();
            if (package.Length > 0)
                _server.Install(package);
        }
    }