Zasadniczo, co kiedykolwiek czyni kod prostszym. Pojedynczy punkt wyjścia jest fajnym ideałem, ale nie wyginałbym kodu z kształtu tylko po to, aby to osiągnąć ... A jeśli alternatywą jest zadeklarowanie zmiennej lokalnej (poza zamkiem), zainicjowanie jej (wewnątrz zamka) i potem zwracając go (poza zamek), wtedy powiedziałbym, że proste „return foo” wewnątrz zamka jest o wiele prostsze.
Aby pokazać różnicę w IL, zakodujmy:
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;
}
}
(zauważ, że z przyjemnością twierdzę, że ReturnInside
jest to prostszy / czystszy kawałek C #)
Spójrz na IL (tryb zwolnienia itp.):
.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
}
Więc na poziomie IL są one [podaj lub przyjmij kilka nazw] identyczne (nauczyłem się czegoś ;-p). W związku z tym jedynym sensownym porównaniem jest (wysoce subiektywne) prawo lokalnego stylu kodowania ... Wolę ReturnInside
prostotę, ale też nie byłbym podekscytowany.