在多线程编程中lock（string）{...}隐藏的机关 - Leo C.W_H5之家

常见误用场景：在订单支付环节中，为了防止用户不小心多次点击支付按钮而导致的订单重复支付问题，我们用 lock(订单号) 来保证对该订单的操作同时只允许一个线程执行。

这样的想法很好，至少比 lock(处理类的private static object)要好，因为lock订单号想要的效果是只锁当前1个订单的操作，而如果lock静态变量，那就是锁所有的订单，就会导致所有的订单进行排队，这显然是不合理的。

那么本文开篇说的lock（订单号）的做法可以实现想要的效果吗？我们先用一些代码来还原使用场景。

如果忽略用户信息及其他验证，那代码差不多是这样：

1 public ActionResult PayOrder(string orderNumber)
2 {
3     lock (orderNumber)
4     {
5         //订单支付，消息通知等耗时的操作
6     }
7     return View("Success");
8 }

这样的代码看起来好像没有什么问题，对于lock关键字，MSDN上面包括能够百度到的资料，好像都是说建议不要使用lock(string)，而原因都是同一个。以下这段话摘自MSDN关于lock字符串的建议：

由于进程中使用同一字符串的任何其他代码将共享同一个锁，所以出现 lock(“myLock”) 问题。

这句话隐藏了一个巨大的机关，那就是“同一字符串”。

什么叫“同一字符串”？请看代码：

static void Main(string[] args)
{
    var str1 = "abc";
    var str2 = "abc";
}

请问上面的str1和str2是同一字符串吗？答案是YES。

再看：

static void Main(string[] args)
{
    var str1 = "abc" + 123;
    var str2 = "abc" + 123;
}

上面的str1和str2还是同一字符串吗？答案就是NO了。

好了，再回到我们订单支付的问题上面来。在我们的代码中， lock(orderNumber) ，当用户手滑一不小心多点了几次，请问每次进入这个action的orderNumber是同一字符串吗？答案是NO。这就是说

上面处理订单的代码实际上并没有起到任何lock的作用。

实际上，字符串比较分两种，请看代码：

static void Main(string[] args)
{
    var str1 = "abc" + 123;
    var str2 = "abc" + 123;
    Console.WriteLine(str1 == str2);
    Console.WriteLine(object.ReferenceEquals(str1, str2));
}

上面的代码第一行输出True，第二行输出False。相信不用我解释你也明白MSDN所说的“同一字符串”了。

最后，再分享一个我们项目中用来解决lock（订单号）的方案。

调用方法：

public ActionResult PayOrder(string orderNumber)
{
    Locker.Run(orderNumber, () =>
    {
        //订单支付，消息通知等耗时的操作
    });
    return View("Success");
}

用到的Locker类：

			
public class Locker
{
   private const int ExpireMinutes = 10;

   private static readonly Timer _timer;
   private static readonly Dictionary<string, LockObj> _dict = new Dictionary<string, LockObj>();

   static Locker()
   {
       _timer = new Timer(60000);
       _timer.Elapsed += (s, e) =>
       {
           RemovedExired();
       };
       _timer.Start();
   }

   public static void Run(string key, Action action)
   {
       LockObj lockObj = null;
       lock (_dict)
       {
           if (!_dict.ContainsKey(key))
           {
               _dict[key] = new LockObj();
           }
           lockObj = _dict[key];
           lockObj.Time = DateTime.Now;
       }
       lock (lockObj)
       {
           action();
       }
   }

   public static void RemovedExired()
   {
       lock (_dict)
       {
           var keys = _dict.Where(x => x.Value.IsExpired()).Select(x => x.Key).ToList();
           foreach (var key in keys)
           {
               _dict.Remove(key);
           }
       }
   }

   private class LockObj
   {
       public DateTime Time { private get; set; }

       public bool IsExpired()
       {
           return this.Time < DateTime.Now.AddMinutes(-ExpireMinutes);
       }
   }
}
			

总结

lock（字符串）其实最大的用处就是类似锁定当前订单的操作，lock一个常量字符串就没有多大意义，正如MSDN所说，不推荐使用。