如何关闭在 wpf 中的线程中打开的窗口？

Question

按照规范，我构建了一个监视某些资源的应用程序。现在我有一个主窗口，当事件发生时会发出警报，说明规范必须是一个颜色鲜艳的大窗口。 我对问题的解决方法如下

 private void StatusChanges(Alarm m, EventArgs e)
        {

            //Compares two 32-bit signed integers for equality and, if they are equal, replaces one of the values.
                if (Interlocked.CompareExchange(ref running, 1, 0) == 0)
                {
                    newWindowThread = new Thread(new ThreadStart(() =>
                                                                     {

                        try
                        {
                            // Create and show the Window
                            Alert tempWindow = new Alert();

                            tempWindow.Show();
                            //if (cancelRun)
                            //    tempWindow.Close();

                            // Start the Dispatcher Processing
                            System.Windows.Threading.Dispatcher.Run();



                        }
                        catch (Exception)
                        {

                            throw;
                        }
                        finally
                        {
                            running = 0;

                        }

                    }));

                    // Set the apartment state
                    newWindowThread.SetApartmentState(ApartmentState.STA);

                    // Make the thread a background thread
                    newWindowThread.IsBackground = true;

                    newWindowThread.Start();

                }


            }
            else
            {


                if (Interlocked.CompareExchange(ref running, 0, 1) == 1)
                {
                    try
                    {
                        System.Windows.Threading.Dispatcher.FromThread(newWindowThread).InvokeShutdown();
                        Alert tempWindow = new Alert();
                        tempWindow.Close();
                    }
                    catch (Exception)
                    {

                        //throw;
                        MessageBox.Show("Todo esta ok");
                    }

                }

                //running = 0;
            }
        }

此方法由 Timer 每隔一段时间执行一次

我的目标是当警报结束时，移除警报窗口的过程，我使用了Thread.Abort();没用，有时程序抛出异常并显示没有Visual Studio in aplition发生在外面，尝试使用threa.Join ();但它没有用，直到

System.Windows.Threading.Dispatcher.FromThread use (newWindowThread) InvokeShutdown ().; 

这至少停止了窗口的显示，但是当我创建时

Alert tempWindow = new Alert(); 

在初始化构造函数中

SoundPlayer player = new Sound ("../../Sounds/BOMB_SIREN-BOMB_SIREN-247265934.wav");
  player.PlayLooping();

当我关闭窗口时

player.Stop();

现在如果没有，我会做下一个

 Alert tempWindow = new Alert();
                        tempWindow.Close();

声音继续播放

我知道这不是最好的解决方案，但它确实有效。 我想给我您的意见，并就如何改进代码给我一些建议。

Answer 1

快速查看您的代码，您正在做的是关闭临时代码。窗口错误。当您想要关闭现有窗口时，您正在创建一个全新的窗口。 这应该更新，以便保存现有的临时窗口，以便以后可以关闭相同的实例。

我还更新了临时窗口的关闭，以便它在关闭后关闭线程。

我更新代码如下：

Alert _tempWindow;

private void StatusChanges(Alarm m, EventArgs e)
{
    //Compares two 32-bit signed integers for equality and, if they are equal, replaces one of the values.
        if (Interlocked.CompareExchange(ref running, 1, 0) == 0)
        {
            newWindowThread = new Thread(new ThreadStart(() =>
            {
                try
                {   
                    // Create and show the Window
                    _tempWindow = new Alert();
                    _tempWindow.Close += OnTempClosed;
                    _tempWindow.Show();

                    System.Windows.Threading.Dispatcher.Run();
                }
                catch (Exception)
                {
                    throw;
                }
                finally
                {
                    running = 0;
                }

            }));

            // Set the apartment state
            newWindowThread.SetApartmentState(ApartmentState.STA);
            // Make the thread a background thread
            newWindowThread.IsBackground = true;
            newWindowThread.Start();
        }
    }
    else
    {
        if (Interlocked.CompareExchange(ref running, 0, 1) == 1)
        {
            try
            {
                _tempWindow.Dispatcher.BeginInvoke((Action)_tempWindow.Close);
            }
            catch (Exception)
            {
                //throw;
                MessageBox.Show("Todo esta ok");
            }
        }
        //running = 0;
    }
}

private void OnTempClosed(object sender, EventArgs e)
{
    System.Windows.Threading.Dispatcher.FromThread(newWindowThread).InvokeShutdown();
}

Answer 2

这个想法是只在一个线程中使用 UI。因此，与其创建一个全新的调度程序，不如简单地将新的窗口逻辑调度到现有的 UI 线程（毕竟，定时器不是已经完全由 UI 线程结束了吗？）。

为什么要在新线程中启动新窗口？你希望通过它实现什么？

您的关闭代码没有意义：

Dispatcher.FromThread(newWindowThread).InvokeShutdown();
Alert tempWindow = new Alert();
tempWindow.Close();

让我们来看看这个。首先，关闭第二个 UI 线程的调度程序。这意味着所有将流向窗口的消息 - 不要。

然后，您在原始 UI 线程上创建一个 new 窗口，然后关闭它。那应该完成什么？您要关闭之前打开的窗口，而不是新窗口！

实际上，您需要保留对您打开的原始窗口的引用 - 对其线程的引用并不重要。现在，如果您真的想继续使用双 UI 线程方法，您可以这样做：

alertWindow.Dispatcher.Invoke(alertWindow.Close);

这将告诉（已经打开的）警报窗口上的消息循环尽快关闭窗口。

除此之外，您的代码有太多问题了。一方面，您使用的操作级别太低（Interlocked.CompareExchange 无法保持警报/没有警报逻辑？）。二，你的 cmets 完全没用 - 你只是在说下一行的作用，字面意思 - 这没有帮助。而不是

// Compares two 32-bit signed integers for equality and, 
// if they are equal, replaces one of the values.

您想使用描述您想要达到的目标的内容，而不是如何您正在做的事情：

// If we're the first ones to attempt to change the running flag,
// we'll create a new alert window.

请注意，该评论也很好地表明您正在做一些非常奇怪的事情。

三，您不必要地创建线程（更重要的是，UI 线程 - 除了非常特殊的情况外，实际上应该只有一个）。没有任何理由在新线程中创建新窗口。你为什么要这么做？

第四，您的异常处理也不是很有帮助。一方面，如果您所做的只是重新抛出异常，则可以省略 catch 子句 - 您可以单独使用 try ... finally。更重要的是，异常将传播到新线程，而不是旧线程 - 这很可能会导致您的应用程序完全崩溃。