C# 在“using”语句中调用“BackgroundWorker”是否安全？

Question

我开发了一个 Windows 服务。在服务中，我使用 BackgroundWorker 在我的数据库中发布数据。 我在我的数据库构造函数类中声明了一个 BackgroundWorker，并在需要时使用它。 在测试过程中，我遇到了一个错误：

此 BackgroundWorker 当前正忙，无法运行多个任务 同时

我试图找出解决方案，许多人建议为每个任务使用新实例。我改变了我的代码：

        ...
        using (BackgroundWorker bw = new BackgroundWorker())
        {
            bw.DoWork += new DoWorkEventHandler(bkDoPost);
            bw.RunWorkerAsync(dbobj);
        }
        ...

而我的“bkDoPost”是：

void bkDoPost(object sender, DoWorkEventArgs e)
{
    try
    {
        dbObject dbj = e.Argument as dbObject;
        this.db.Insert(dbj.tableName, dbj.data);
    }
    catch (Exception ex)
    {
        logs.logMessage("There was an error in data post. See the ErrorLog");
        logs.logError(ex);
    }
}

代码在测试过程中运行良好。

我的问题是我的方法正确吗？

或者这样做有什么问题吗？

谢谢

Answer 1

不要那样做。您的后台工作人员将在您的工作完成之前被处置。

工作完成后最好手动调用Dispose。

更好的是，考虑使用不同的方案来处理异步工作。后台工作者正在变得过时并且针对 UI 应用程序，而不是服务。对并行操作的限制突出了类的意图。

Answer 2

不要将 BackgroundWorker 放入 using 语句中。而是将 Dispose() 调用放入 RunWorkerCompleted 事件中。

尽管如此，BackgroundWorker 可能不是在您的情况下使用的最佳选择，因为它的主要用途是在 UI 保持响应时运行一些业务代码，并在 RunWorkerCompeleted 事件中自动更新 UI。

如果您在作业完成时不需要干预 UI，或者您有很多较小的作业要完成，那么切换到将您的作业封装在 Tasks 中会更有效。

Answer 3

如果您有许多更新，则为每个更新创建一个 BackgroundWorker 可能会非常耗时且耗费内存。
我会使用一个独立的线程，每次必须完成更新时我都会唤醒它：

Queue<DbWork> dbUpdates = new Queue<DbWork>();
EventWaitHandle waiter = new EventWaitHandle(false, EventResetMode.ManualReset);
...
// Init :
new Thread(new ThreadStart(DbUpdateWorker));
...

private void DbUpdateWorker()
{
    while (true)
    {
        DbWork currentWork = null;
        lock (dbUpdates)
        {
            if (dbUpdates.Count > 0)
                currentWork = dbUpdates.Dequeue();
        }

        if (currentWork != null)
        {
            currentWork.DoWork();
        }

        if (dbUpdates.Count == 0)
        {
            waiter.WaitOne();
        }
    }
}

public void AddWork(DbWork work)
{
    lock (dbUpdates)
    {
        dbUpdates.Enqueue(work);
    }
    waiter.Set();
}