倒立的后台工作者

Question

我有许多类可以完成一些工作，通常是单步执行记录集并为每条记录调用一个或两个 Web 服务。

目前，这一切都在 GUI 线程中运行并挂起绘画。第一个想法是使用BackgroundWorker 并实现一个漂亮的进度条、处理错误、完成等。Background Worker 可以实现所有美好的事情。

代码一出现在屏幕上，它就开始散发出气味。我在每个类中编写了很多后台工作人员，在 bw_DoWork 方法中重复了大部分 ProcessRows 方法，并认为应该有更好的方法，而且可能已经完成了。

在我继续重新发明轮子之前，是否有一个类的模式或实现可以将后台工作人员分开？它需要实现 ibackgroundable 等接口的类，但这些类仍然可以独立运行，并且需要最少的更改来实现该接口。

编辑：@Henk 请求的简化示例：

我有：

    private void buttonUnlockCalls_Click(object sender, EventArgs e)
    {
        UnlockCalls unlockCalls = new UnlockCalls();
        unlockCalls.MaxRowsToProcess = 1000;
        int processedRows = unlockCalls.ProcessRows();
        this.textProcessedRows.text = processedRows.ToString();
    }

我想我想要：

    private void buttonUnlockCalls_Click(object sender, EventArgs e)
    {
        UnlockCalls unlockCalls = new UnlockCalls();
        unlockCalls.MaxRowsToProcess = 1000;

        PushToBackground pushToBackground = new PushToBackground(unlockCalls)
        pushToBackground.GetReturnValue = pushToBackground_GetReturnValue;
        pushToBackground.DoWork();
    }

    private void pushToBackground_GetReturnValue(object sender, EventArgs e)
    {
        int processedRows = e.processedRows;
        this.textProcessedRows.text = processedRows.ToString();
    }

我可以继续这样做，但不想重新发明。

我正在寻找的答案将类似于“是的，Joe 对此做了很好的实现（此处）”或“这是一个代理小部件模式，请阅读它（此处）”

Answer 1

每个操作都需要实现以下接口：

/// <summary>
/// Allows progress to be monitored on a multi step operation
/// </summary>
interface ISteppedOperation
{
    /// <summary>
    /// Move to the next item to be processed.
    /// </summary>
    /// <returns>False if no more items</returns>
    bool MoveNext();

    /// <summary>
    /// Processes the current item
    /// </summary>
    void ProcessCurrent();

    int StepCount { get; }
    int CurrentStep { get; }
}

这将步骤的枚举与处理分开。

这是一个示例操作：

class SampleOperation : ISteppedOperation
{
    private int maxSteps = 100;

    //// The basic way of doing work that I want to monitor
    //public void DoSteppedWork()
    //{
    //    for (int currentStep = 0; currentStep < maxSteps; currentStep++)
    //    {
    //        System.Threading.Thread.Sleep(100);
    //    }
    //}

    // The same thing broken down to implement ISteppedOperation
    private int currentStep = 0; // before the first step
    public bool MoveNext()
    {
        if (currentStep == maxSteps)
            return false;
        else
        {
            currentStep++;
            return true;
        }
    }

    public void ProcessCurrent()
    {
        System.Threading.Thread.Sleep(100);
    }

    public int StepCount
    {
        get { return maxSteps; }
    }

    public int CurrentStep
    {
        get { return currentStep; }
    }

    // Re-implement the original method so it can still be run synchronously
    public void DoSteppedWork()
    {
        while (MoveNext())
            ProcessCurrent();
    }
}

可以从如下形式调用：

private void BackgroundWorkerButton_Click(object sender, EventArgs eventArgs)
{
    var operation = new SampleOperation();

    BackgroundWorkerButton.Enabled = false;

    BackgroundOperation(operation, (s, e) =>
        {
            BackgroundWorkerButton.Enabled = true;
        });
}

private void BackgroundOperation(ISteppedOperation operation, RunWorkerCompletedEventHandler runWorkerCompleted)
{
    var backgroundWorker = new BackgroundWorker();

    backgroundWorker.RunWorkerCompleted += runWorkerCompleted;
    backgroundWorker.WorkerSupportsCancellation = true;
    backgroundWorker.WorkerReportsProgress = true;

    backgroundWorker.DoWork += new DoWorkEventHandler((s, e) =>
    {
        while (operation.MoveNext())
        {
            operation.ProcessCurrent();

            int percentProgress = (100 * operation.CurrentStep) / operation.StepCount;
            backgroundWorker.ReportProgress(percentProgress);

            if (backgroundWorker.CancellationPending) break;
        }
    });

    backgroundWorker.ProgressChanged += new ProgressChangedEventHandler((s, e) =>
    {
        var progressChangedEventArgs = e as ProgressChangedEventArgs;
        this.progressBar1.Value = progressChangedEventArgs.ProgressPercentage;
    });

    backgroundWorker.RunWorkerAsync();
}

我还没有这样做，但我将把 BackgroundOperation() 移动到它自己的类中，并实现取消操作的方法。

Answer 2

我会将非 UI 代码放入一个新类中并使用线程（而不是后台工作者）。要显示进度，请将新类触发事件返回 UI 并使用 Dispatcher.Invoke 更新 UI。

其中有一些编码，但它更干净并且有效。并且比使用后台工作人员更易于维护（这仅适用于小任务）。