使用信号量而不是 while 循环。这是好事还是坏事？

Question

我有一个进程在它自己的线程中运行，并且可以在没有阻塞的情况下启动/停止。这最终将进入 Windows 服务，但我现在在控制台应用程序中设置它，直到它完全充实。

在调用 Start() 之后，我希望主程序线程阻塞，直到按下 Ctrl-C。我知道这会奏效：

public static void Main(string[] args)
{
    bool keepGoing = true;

    var service = new Service();

    System.Console.TreatControlCAsInput = false;

    System.Console.CancelKeyPress += delegate(object sender, ConsoleCancelEventArgs e)
    {
        e.Cancel = true;
        service.Stop();
        keepGoing = false; // Break the while loop below
    };

    service.Start();

    while( keepGoing )
    {
        Thread.Sleep(100); // 100 is arbitrary
    }

}

但是，我发现标志和任意睡眠值很麻烦。我知道在 while 循环中 CPU 成本实际上是 0，但我宁愿有一个“硬”块，在 Ctrl-C 处理程序完成后立即释放。我设计了以下内容，使用信号量阻止匿名 Ctrl-C 处理程序完成：

public static void Main(string[] args)
{
    var service = new Service();

    var s = new Semaphore(1, 1);

    System.Console.TreatControlCAsInput = false;

    System.Console.CancelKeyPress += delegate(object sender, ConsoleCancelEventArgs e)
    {
        e.Cancel = true;
        service.Stop();
        s.Release(); // This will allow the program to conclude below
    };

    service.Start();

    s.WaitOne(); // This will not block
    s.WaitOne(); // This will block w/o CPU usage until the sempahore is released

}

这是一个糟糕的设计吗？是不是矫枉过正？危险吗？

编辑：

我还连接了 AppDomain.CurrentDomain.UnhandledException 如下：

AppDomain.CurrentDomain.UnhandledException += delegate {
  service.Stop();
  s.Release();
};

编辑第二个：

我应该注意到，在退出时调用 Stop() 方法至关重要。 @Adam Ralph 有一个非常好的混合控制台/服务模式，但在回答问题时没有此信息。

Answer 1

我们在一些应用中有类似的要求。它们是 Windows 服务，但为了调试，我们通常希望将它们作为控制台应用程序运行。此外，我们通常会在较早的时候将新应用程序编码为 Windows 服务，但通常不希望等到后来，一旦我们证明了这个概念等，就必须将它们作为服务实际运行。

这是我们使用的模式：-

using (var service = new Service())
{
    if (Environment.UserInterActive)
    {
        service.Start();
        Thread.Sleep(Timeout.Infinite);
    }
    else
    {
        ServiceBase.Run(service);
    }
}

让线程无限休眠可能看起来效率低下，但这只是用于调试场景，冗余线程不消耗 CPU 时间，只是一些内存（约 1MB），主要由分配给线程的堆栈空间组成。该过程仍然可以通过 Ctrl+C 或关闭命令窗口来退出。

-- 编辑--

如果您发现在按下 Ctrl+C 时没有调用 service.Dispose()（即发生粗鲁的中止）并且对 Dispose() 的调用至关重要，那么我想您可以像这样明确地这样做：-

using (var service = new Service())
{
    if (Environment.UserInterActive)
    {
        Console.CancelKeyPress += (sender, e) => service.Dispose();
        service.Start();
        Thread.Sleep(Timeout.Infinite);
    }
    else
    {
        ServiceBase.Run(service);
    }
}

注意Stop()应该封装在Dispose()中。