เป็นวิธีที่ดีในการโทรกลับภายในโดยใช้คำสั่ง {} หรือไม่

ฉันแค่อยากรู้ว่าวิธีการโทรreturnภายในusingบล็อกนั้นปลอดภัย / ดีไหม

สำหรับอดีต

using(var scope = new TransactionScope())
{
  // my core logic
  return true; // if condition met else
  return false;
  scope.Complete();
}

เรารู้ว่าในวงเล็บปีกกาสุดท้าย dispose()จะถูกเรียกออก แต่จะเกิดอะไรขึ้นในกรณีข้างต้นเนื่องจากreturnข้ามการควบคุมออกจากขอบเขตที่กำหนด (AFAIK) ...

1. ฉันscope.Complete()ถูกเรียก?
2. และสำหรับdispose()วิธีการของขอบเขต

การโทรreturnภายในusingบล็อกของคุณปลอดภัยอย่างสมบูรณ์เนื่องจากบล็อกที่ใช้เป็นเพียงtry/finallyบล็อก

ในตัวอย่างของคุณด้านบนหลังจากส่งคืนtrueขอบเขตจะถูกกำจัดและมูลค่าที่ส่งคืน return falseและscope.Complete()จะไม่ถูกเรียก Disposeอย่างไรก็ตามจะถูกเรียกโดยไม่คำนึงถึงเนื่องจากมันอยู่ในบล็อกสุดท้าย

โดยพื้นฐานแล้วรหัสของคุณจะเหมือนกับสิ่งนี้ (หากทำให้เข้าใจง่ายขึ้น):

var scope = new TransactionScope())
try
{
  // my core logic
  return true; // if condition met else
  return false;
  scope.Complete();
}
finally
{
  if( scope != null) 
    ((IDisposable)scope).Dispose();
}

โปรดทราบว่าธุรกรรมของคุณจะไม่มีวันผูกมัดเนื่องจากไม่มีทางที่scope.Complete()จะทำธุรกรรมได้

ไม่เป็นไร - finallyประโยค (ซึ่งเป็นสิ่งที่วงเล็บปีกกาปิดของส่วนusingคำสั่งทำภายใต้ประทุน) จะถูกดำเนินการเสมอเมื่อขอบเขตถูกทิ้งไว้ไม่ว่าจะอย่างไร

อย่างไรก็ตามนี่เป็นจริงสำหรับข้อความที่อยู่ในบล็อกสุดท้ายเท่านั้น (ซึ่งไม่สามารถตั้งค่าอย่างชัดเจนเมื่อใช้using) ดังนั้นในตัวอย่างของคุณscope.Complete()จะไม่ถูกเรียก (ฉันคาดว่าคอมไพเลอร์จะเตือนคุณเกี่ยวกับรหัสที่ไม่สามารถเข้าถึงได้)

โดยทั่วไปเป็นแนวทางที่ดี แต่ในกรณีของคุณถ้าคุณกลับมาก่อนที่จะโทรไปscope.Complete()มันก็จะทิ้ง TransactionScope ขึ้นอยู่กับการออกแบบของคุณ

ดังนั้นในตัวอย่างนี้จึงไม่มีการเรียก Complete () และขอบเขตจะถูกกำจัดโดยสมมติว่ามันกำลังสืบทอดอินเทอร์เฟซ IDisposable

scope.Complete returnแน่นอนควรจะเรียกว่าก่อนที่จะ คอมไพเลอร์จะแสดงคำเตือนและจะไม่มีการเรียกรหัสนี้

เกี่ยวกับreturnตัวมันเอง - ใช่มันปลอดภัยที่จะเรียกมันว่าusingงบภายใน การใช้ถูกแปลว่าพยายามบล็อกในที่สุดหลังฉากและในที่สุดการบล็อกจะถูกดำเนินการ

ในตัวอย่างที่คุณให้มามีปัญหา scope.Complete()ไม่เคยถูกเรียก ประการที่สองก็ไม่ได้เป็นวิธีที่ดีในการใช้returnคำสั่งภายในusingงบ อ้างถึงสิ่งต่อไปนี้:

using(var scope = new TransactionScope())
{
    //have some logic here
    return scope;      
}

ในตัวอย่างง่ายๆนี้ประเด็นก็คือ ค่าของscopeจะเป็นโมฆะเมื่อใช้คำสั่งเสร็จสิ้น

ดังนั้นจึงเป็นการดีกว่าที่จะไม่กลับภายในโดยใช้งบ

เพื่อให้แน่ใจว่าจะถูกเรียกว่าห่อมันด้วยscope.Complete() try/finallyสิ่งdisposeนี้เรียกว่าเนื่องจากคุณได้ห่อด้วยบล็อกusingทางเลือกtry/finally

using(var scope = new TransactionScope())
{
  try
  {
  // my core logic
  return true; // if condition met else
  return false;
  }
  finally
  {
   scope.Complete();
  }
}

ในตัวอย่างนี้ scope.Complete () จะไม่ดำเนินการ อย่างไรก็ตามคำสั่ง return จะล้างข้อมูลทุกอย่างที่กำหนดบนสแตก GC จะดูแลทุกอย่างที่ไม่มีการอ้างอิง ดังนั้นเว้นแต่จะมีวัตถุที่ GC ไม่สามารถหยิบขึ้นมาได้ก็ไม่มีปัญหา