繁体   English   中英

使用Dispatcher.BeginInvoke()时如何避免出现竞争情况?

[英]How do I avoid a race condition when using Dispatcher.BeginInvoke()?

下面的代码仅记录a和b之间的质数。 C# async await适用于我的代码,但较旧的dispatcher方式给出奇怪的结果。当我单击按钮时,我得到以下结果:
70435介于2000000和2999999之间的素数
67883介于3000000至3999999之间的素数
66330介于4000000至4999999之间的素数
65367素数介于5000000和5999999之间
这是错误的,因为i should be <5 and begin with 1000000 有人在这里解释比赛情况吗?

private void _button_Click(object sender, RoutedEventArgs e)
{
    //Go();
    Task.Run(() => Go1());
}
void Go1()
{
    Dispatcher.BeginInvoke(new Action(() => _button.IsEnabled = false));
    for (int i = 1; i < 5; i++)
    {
        int result = GetPrimesCount(i * 1000000, 1000000);
        Dispatcher.BeginInvoke(new Action(() =>
        _results.Text += result + " primes between " + (i * 1000000) +
        " and " + ((i + 1) * 1000000 - 1) + Environment.NewLine));

    }
    Dispatcher.BeginInvoke(new Action(() => _button.IsEnabled = true));

}
int GetPrimesCount(int start, int count)
{
    return ParallelEnumerable.Range(start, count).Count(n =>
    Enumerable.Range(2, (int)Math.Sqrt(n) - 1).All(i => n % i > 0));
}

这是一个古老的陷阱:lambda封闭变量而不是 ,因此,多次更新访问的i实际上是相同的i

附带一提,我个人将重写代码,根本不使用Dispatcher 总有Dispatcher更好的解决方案:

private async void _button_Click(object sender, RoutedEventArgs e)
{
  _button.IsEnabled = false;
  var progress = new Progress<Tupe<int, int>>(update =>
  {
    _results.Text += update.Item1 + " primes between " + (update.Item2 * 1000000) +
        " and " + ((update.Item2 + 1) * 1000000 - 1) + Environment.NewLine));
  });
  await Task.Run(() => Go1(progress));
  _button.IsEnabled = true;
}
void Go1(IProgress<Tuple<int, int>> progress)
{
  for (int i = 1; i < 5; i++)
  {
    int result = GetPrimesCount(i * 1000000, 1000000);
    if (progress != null)
      progress.Report(Tuple.Create(result, i));
  }
}

暂无
暂无

声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.

 
粤ICP备18138465号  © 2020-2024 STACKOOM.COM