返回声明应该在锁内还是外?

我只是意识到,在我的代码中的一些地方,我有锁内部的返回语句,有时在外面。 哪一个是最好的?

1)

void example() { lock (mutex) { //... } return myData; } 

2)

 void example() { lock (mutex) { //... return myData; } } 

我应该使用哪一个?

从本质上讲,哪个代码更简单。 单点退出是一个不错的理想,但我不会弯曲代码变形只是为了实现它…而如果替代是声明一个局部variables(锁之外),初始化它(在锁内)和然后返回它(在锁之外),然后我会说,在锁内的一个简单的“返回foo”是一个简单得多。

为了显示IL中的差异,让代码:

 static class Program { static void Main() { } static readonly object sync = new object(); static int GetValue() { return 5; } static int ReturnInside() { lock (sync) { return GetValue(); } } static int ReturnOutside() { int val; lock (sync) { val = GetValue(); } return val; } } 

(请注意,我很高兴地认为, ReturnInside是C#的一个简单/清洁的位)

并看看IL(释放模式等):

 .method private hidebysig static int32 ReturnInside() cil managed { .maxstack 2 .locals init ( [0] int32 CS$1$0000, [1] object CS$2$0001) L_0000: ldsfld object Program::sync L_0005: dup L_0006: stloc.1 L_0007: call void [mscorlib]System.Threading.Monitor::Enter(object) L_000c: call int32 Program::GetValue() L_0011: stloc.0 L_0012: leave.s L_001b L_0014: ldloc.1 L_0015: call void [mscorlib]System.Threading.Monitor::Exit(object) L_001a: endfinally L_001b: ldloc.0 L_001c: ret .try L_000c to L_0014 finally handler L_0014 to L_001b } method private hidebysig static int32 ReturnOutside() cil managed { .maxstack 2 .locals init ( [0] int32 val, [1] object CS$2$0000) L_0000: ldsfld object Program::sync L_0005: dup L_0006: stloc.1 L_0007: call void [mscorlib]System.Threading.Monitor::Enter(object) L_000c: call int32 Program::GetValue() L_0011: stloc.0 L_0012: leave.s L_001b L_0014: ldloc.1 L_0015: call void [mscorlib]System.Threading.Monitor::Exit(object) L_001a: endfinally L_001b: ldloc.0 L_001c: ret .try L_000c to L_0014 finally handler L_0014 to L_001b } 

所以在IL层面上,他们[给予或者采用一些名字]是相同的(我学到了一些东西; -p)。 因此,唯一明智的比较就是本地编码风格的(高度主观的)规律。为了简单起见,我更喜欢ReturnInside ,但是我也不会为此感到兴奋。

它没有任何区别; 它们都被编译器翻译成相同的东西。

为了澄清,无论是有效地翻译成以下语义的东西:

 T myData; Monitor.Enter(mutex) try { myData= // something } finally { Monitor.Exit(mutex); } return myData; 

我肯定会把回报锁在里面。 否则,冒着另一个线程进入锁并在返回语句之前修改你的variables的风险,因此使得原来的调用者接收到不同于预期的值。

如果认为外面的锁看起来更好,但要小心,如果您最终将代码更改为:

 return f(...) 

如果f()需要被调用,那么它显然需要在锁内部,因为这样保持返回锁内的一致性是有道理的。

这取决于,

我要在这里反对粮食。 我一般会回到里面的锁。

通常variablesmydata是一个局部variables。 我喜欢在初始化时声明局部variables。 我很less有数据在我的锁之外初始化我的返回值。

所以你的比较实际上是有缺陷的。 理想的情况下,两种select之间的区别就像你写的那样,这似乎是对情况1的点头,实际上它有点丑陋。

 void example() { int myData; lock (foo) { myData = ...; } return myData } 

 void example() { lock (foo) { return ...; } } 

我发现案例2更容易阅读,更难搞砸,特别是对于短片段。

为了让开发人员更容易阅读代码,我会build议第一个select。

对于它的价值, MSDN上的文档有一个从锁内返回的例子。 从这里的其他答案看来,它看起来与IL非常相似,但是对于我来说,从锁内返回似乎更安全一些,因为这样就不会有被另一个线程覆盖的返回variables的风险。

外面看起来更干净