【问题标题】:Wait until my BackgroundWorker finish in order to open new BackgroundWorker等到我的 BackgroundWorker 完成以打开新的 BackgroundWorker
【发布时间】:2013-05-28 10:35:17
【问题描述】:

在我的应用程序中,我通过打开 Wireshark 进程检查我的文件,然后再添加到我的列表框。 这是添加目录单击事件,它获取根文件夹并检查此文件夹和子文件夹中的所有文件:

private void btnAddDir_Click(object sender, EventArgs e)
{
    try
    {
        if (folderBrowserDialog1.ShowDialog() == DialogResult.OK)
        {
            ThreadStart threadStart = delegate
            {
                foreach (string file in SafeFileEnumerator.EnumerateFiles(folderBrowserDialog1.SelectedPath, "*.*", SearchOption.AllDirectories))
                {
                    Interlocked.Increment(ref numWorkers);
                    StartBackgroundFileChecker(file);
                }
            };

            Thread thread = new Thread(threadStart);
            thread.IsBackground = true;
            thread.Start();
        }
    }
    catch (Exception)
    { }
}

private void StartBackgroundFileChecker(string file)
{
    ListboxFile listboxFile = new ListboxFile();
    listboxFile.OnFileAddEvent += listboxFile_OnFileAddEvent;
    BackgroundWorker backgroundWorker = new BackgroundWorker();
    backgroundWorker.WorkerReportsProgress = true;
    backgroundWorker.DoWork +=
    (s3, e3) =>
    {
        //check my file
    };

    backgroundWorker.RunWorkerCompleted += backgroundWorker_RunWorkerCompleted;
    backgroundWorker.RunWorkerAsync();
}

void backgroundWorker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    if (Interlocked.Decrement(ref numWorkers) == 0)
    {
        //update my UI
    }
}

当我检查这个文件时,我打开了Wireshark 进程,所以如果我选择包含许多文件的文件夹,则会打开许多​​Wireshark 进程,这会占用大量内存, 我怎么能等到我的 BackgroundWorker 完成,然后才打开新的?

【问题讨论】:

  • 很确定我已经评论了这段代码。完全相同的建议,将 foreach 循环移到 worker 内部,这样你就只需要一个 BGW。等待线程总是一个坏主意,你的代码会死锁。

标签: c# winforms


【解决方案1】:

据我了解,您只希望一次启动单个后台工作人员。如果是这样,那么试试这个(基于System.Threading.AutoResetEvent

//introduce additional field
private AutoResetEvent _workerCompleted = new AutoResetEvent(false);
//modify StartBackgroundFileChecker
private void StartBackgroundFileChecker(string file)
{
    ListboxFile listboxFile = new ListboxFile();
    listboxFile.OnFileAddEvent += listboxFile_OnFileAddEvent;
    BackgroundWorker backgroundWorker = new BackgroundWorker();
    backgroundWorker.WorkerReportsProgress = true;
    backgroundWorker.DoWork +=
    (s3, e3) =>
    {
        //check my file
    };

    backgroundWorker.RunWorkerCompleted += backgroundWorker_RunWorkerCompleted;
    backgroundWorker.RunWorkerAsync();
   //new code - wait for completion
   _workerCompleted.WaitOne();
}
//add completion notification to backgroundWorker_RunWorkerCompleted
void backgroundWorker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    if (Interlocked.Decrement(ref numWorkers) == 0)
    {
        //update my UI
    }

    //new code - notify about completion
    _workerCompleted.Set();
}

在该解决方案中,您的后台线程将一一启动新的 BackgroundWorker - 这可能不是最佳的(您可以完全避免 BackgroundWorker,只需通过 threadStart 委托中的 Dispatch 更新 UI)

我认为最好控制并行线程的数量,并且仍然在多个但数量有限的线程中处理文件。

这是替代解决方案(基于 System.Threading.Tasks 命名空间):

 private void btnAddDir_Click(object sender, EventArgs e)
 {
   var selectedPath = folderBrowserDialog1.SelectedPath;
   Task.Factory.StartNew(() =>
        {
           
            var files = Directory.EnumerateFiles(selectedPath, "*.*", SearchOption.AllDirectories);

            Parallel.ForEach(files,
                             new ParallelOptions
                             {
                                     MaxDegreeOfParallelism = 10 // limit number of parallel threads here 
                             },
                             file =>
                             {
                                 //process file here - launch your process
                             });
        }).ContinueWith(
            t => { /* when all files processed. Update your UI here */ }
            ,TaskScheduler.FromCurrentSynchronizationContext() // to ContinueWith (update UI) from UI thread
        );
}

您可以根据自己的特定需求调整此解决方案。

使用的类/方法(参考 MSDN):

  • Task
  • TaskScheduler.FromCurrentSynchronizationContext
  • Parallel.ForEach 方法(IEnumerable、ParallelOptions、Action)

【讨论】:

  • 我觉得控制并行数会好很多
【解决方案2】:

也许是这样,而不是 foreach 保留文件列表,完成后只需获取第一个元素并更新您的列表

private List<string> _files;

private void btnAddDir_Click(object sender, EventArgs e)
{
    try
    {
        if (folderBrowserDialog1.ShowDialog() == DialogResult.OK)
        {

            _files = new List<string>(SafeFileEnumerator.EnumerateFiles(folderBrowserDialog1.SelectedPath, "*.*", SearchOption.AllDirectories));

            Interlocked.Increment(ref numWorkers);
            var file = _files.FirstOrDefault();
            if(file != null)
                StartBackgroundFileChecker(file);
        }
    }
    catch (Exception)
    { }
}

private void StartBackgroundFileChecker(string file)
{
    ListboxFile listboxFile = new ListboxFile();
    listboxFile.OnFileAddEvent += listboxFile_OnFileAddEvent;
    BackgroundWorker backgroundWorker = new BackgroundWorker();
    backgroundWorker.WorkerReportsProgress = true;
    backgroundWorker.DoWork +=
    (s3, e3) =>
    {
        //check my file
    };

    backgroundWorker.RunWorkerCompleted += backgroundWorker_RunWorkerCompleted;
    backgroundWorker.RunWorkerAsync();
}

void backgroundWorker_RunWorkerCompleted(object sender, RunWorkerCompletedEventArgs e)
{
    if (Interlocked.Decrement(ref numWorkers) == 0)
    {
        //update my UI
        _files = _files.Skip(1);
        var file = _files.FirstOrDefault();
        if(file != null)
            StartBackgroundFileChecker(file);
    }
}

【讨论】:

  • 问题是我正在使用 EnumerateFiles 来获取所有“正在处理”的文件,我不想等到我得到所有文件
  • 你可以将它转换成一个字符串列表,你会得到进程中的所有文件,但是每个文件都会在Backgroundworker完成后处理。 backgroundworker在哪里打开wireshark?
  • wireshark 通过另一个类打开进程 //check my file
猜你喜欢
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
相关资源
最近更新 更多