Question

上下文：我正在通过媒体元素播放音乐，并使用滑块显示歌曲中的点。由于显而易见的原因，更新是在后台工作中完成的。

private void bgPlay_DoWork(object sender,DoWorkEventArgs e)
    {

        while (isMediaPlaying)
        {
            this.Dispatcher.Invoke((Action)(() =>
            {
                timelineSlider.Value = mediaElement1.Position.TotalMilliseconds;
            }));

        }
    }

    private void Library_MouseDoubleClick(object sender, MouseButtonEventArgs e)
    {
        try
        {
            mediaElement1.Stop();
            isMediaPlaying = false;
            mediaElement1.Source = new Uri(songData[Library.SelectedIndex].Location);
            mediaElement1.Volume = (double)volumeSlider.Value;

            mediaElement1.Play();

            isMediaPlaying = true;

            bgPlay.RunWorkerAsync();
        }
        catch(Exception ex) {
            F.MessageBox.Show(ex.ToString());
        }
    }

当我播放一首歌，然后双击另一首歌时，后台工作者仍在循环并抛出异常，因为它在前一个实例完成之前到达bgPlay.RunWorkerAsync();。我尝试使用isMediaPlaying bool告诉后台工作者何时退出循环，但主线程在完成之前到达bgPlay.RunWorkerAsync();。

Answer 1

当一个人几乎没有开始使用线程编程时，你会遇到一个常见的错误，race condition

我建议像这样重写代码：

private static String threadingLock = "";

private void bgPlay_DoWork(object sender,DoWorkEventArgs e)
    {

        while (true)
        {
            lock(threadingLock) {
                if(!isMediaPlaying)
                    break;
            }
            this.Dispatcher.Invoke((Action)(() =>
            {
                timelineSlider.Value = mediaElement1.Position.TotalMilliseconds;
            }));

        }
    }

    private void Library_MouseDoubleClick(object sender, MouseButtonEventArgs e)
    {
        try
        {
            lock(threadingLock) {
                isMediaPlaying = false;
            }
            mediaElement1.Stop();

            mediaElement1.Source = new Uri(songData[Library.SelectedIndex].Location);
            mediaElement1.Volume = (double)volumeSlider.Value;

            mediaElement1.Play();

            isMediaPlaying = true;

            bgPlay.RunWorkerAsync();
        }
        catch(Exception ex) {
            F.MessageBox.Show(ex.ToString());
        }
    }

作为一个友好的提示，在调用滑块上的更新之前添加Thread.sleep(200)。它将减少CPU使用量，而不会影响应用程序的功能。

确保后台工作人员在运行之前已经完成（再次）

1 个答案: