常见误用场景：在订单支付环节中，为了防止用户不小心多次点击支付按钮而导致的订单重复支付问题，我们用 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类：
1 public class Locker
2 {
3 private const int ExpireMinutes = 10;
4
5 private static readonly Timer _timer;
6 private static readonly Dictionary<string, LockObj> _dict = new Dictionary<string, LockObj>();
7
8 static Locker()
9 {
10 _timer = new Timer(60000);
11 _timer.Elapsed += (s, e) =>
12 {
13 RemovedExired();
14 };
15 _timer.Start();
16 }
17
18 public static void Run(string key, Action action)
19 {
20 LockObj lockObj = null;
21 lock (_dict)
22 {
23 if (!_dict.ContainsKey(key))
24 {
25 _dict[key] = new LockObj();
26 }
27 lockObj = _dict[key];
28 lockObj.Time = DateTime.Now;
29 }
30 lock (lockObj)
31 {
32 action();
33 }
34 }
35
36 public static void RemovedExired()
37 {
38 lock (_dict)
39 {
40 var keys = _dict.Where(x => x.Value.IsExpired()).Select(x => x.Key).ToList();
41 foreach (var key in keys)
42 {
43 _dict.Remove(key);
44 }
45 }
46 }
47
48 private class LockObj
49 {
50 public DateTime Time { private get; set; }
51
52 public bool IsExpired()
53 {
54 return this.Time < DateTime.Now.AddMinutes(-ExpireMinutes);
55 }
56 }
57 }
总结
lock（字符串）其实最大的用处就是类似锁定当前订单的操作，lock一个常量字符串就没有多大意义，正如MSDN所说，不推荐使用。