try-catch-finally 引发的奇怪问题

今天，发现我们的一个Windows Service无法正常停止，无奈之下只能杀了进程。

为了找到原因，我在本地进行调试，发现程序里用到了多线程，而代码正是卡在了workThread.Abort()语句而无法停止。

为什么不能Abort？ 继续看线程调用的方法的代码，发现没有什么特殊的代码，只是在其中用了Thread.Sleep进行长时间等待。

难道是这个引起的？ 写了一个测试程序验证，

class Program    {        private readonly Thread workThread;        public Program()        {            workThread = new Thread(DoWork);        }        static void Main(string[] args)        {            new Program().Work();            Console.ReadLine();        }        private void Work()        {            workThread.Start();            Thread.Sleep(1 * 1000);            Console.WriteLine("aborting");            workThread.Abort();            Console.WriteLine("aborted");        }        private void DoWork()        {                Console.WriteLine("started");               Thread.Sleep(300 * 1000);        }    }

发现可以正常终止。

startedabortingaborted

 

再仔细检查，发现其中一处Thread.Sleep放在了finally块中，修改测试代码

class Program    {        private readonly Thread workThread;        public Program()        {            workThread = new Thread(DoWork);        }        static void Main(string[] args)        {            new Program().Work();            Console.ReadLine();        }        private void Work()        {            workThread.Start();            Thread.Sleep(1 * 1000);            Console.WriteLine("aborting");            workThread.Abort();            Console.WriteLine("aborted");        }        private void DoWork()        {            try            {                Console.WriteLine("started");            }            catch (Exception)            {                throw;            }            finally            {                for (int i = 0; i < 3; i++)                {                    Console.WriteLine("ThreadState:" + workThread.ThreadState);                    Thread.Sleep(1000);                }            }        }    }

输出：

startedThreadState:RunningabortingThreadState:AbortRequestedThreadState:AbortRequestedaborted

MSDN是这样解释的：

线程不一定会立即中止，或者根本不中止。 如果线程在作为中止过程的一部分被调用的 finally 块中做非常大量的计算，从而无限期延迟中止操作，则会发生这种情况。http://msdn.microsoft.com/zh-cn/library/5b50fdsz.aspx

其实追根溯源，问题是出在程序员对try-catch-finally的滥用上，finally块应该是用来做一些收尾工作，而不是等待操作。

finally 块用于清除 try 块中分配的任何资源，以及运行任何即使在发生异常时也必须执行的代码。http://msdn.microsoft.com/zh-cn/library/zwc8s4fz%28v=vs.80%29.aspx

 

找到了原因，解决方案也挺简单，将Thread.Sleep从finally块中移除即可。