Gọi return bên trong bằng câu lệnh {} có phải là một cách tốt không?

Tôi chỉ muốn biết cách gọi returnbên trong một usingkhối có an toàn / tốt hay không.

Đối với người yêu cũ.

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

Chúng ta biết dấu ngoặc nhọn cuối cùng dispose()sẽ bị tắt. Nhưng điều gì sẽ xảy ra trong trường hợp trên, vì returnnhảy điều khiển ra khỏi phạm vi đã cho (AFAIK) ...

1. Tôi scope.Complete()có được gọi không?
2. Và đối với dispose()method của scope cũng vậy .

Hoàn toàn an toàn khi gọi returnbên trong usingkhối của bạn , vì khối đang sử dụng chỉ là một try/finallykhối.

Trong ví dụ của bạn ở trên sau khi trả về true, phạm vi sẽ bị loại bỏ và giá trị được trả lại. return false, và scope.Complete()sẽ không được gọi. Disposetuy nhiên sẽ được gọi bất kể nó nằm bên trong khối cuối cùng.

Mã của bạn về cơ bản giống như sau (nếu điều đó làm cho nó dễ hiểu hơn):

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();
}

Xin lưu ý rằng giao dịch của bạn sẽ không bao giờ được cam kết vì không có cách nào scope.Complete()để thực hiện giao dịch.

Điều đó tốt - finallycác mệnh đề (là những gì dấu ngoặc nhọn đóng của usingmệnh đề làm dưới mui xe) luôn được thực thi khi phạm vi được để lại, bất kể như thế nào.

Tuy nhiên, điều này chỉ đúng với các câu lệnh nằm trong khối cuối cùng (không thể đặt rõ ràng khi sử dụng using). Do đó, trong ví dụ của bạn, scope.Complete()sẽ không bao giờ được gọi (tôi hy vọng trình biên dịch sẽ cảnh báo bạn về mã không thể truy cập được).

Nói chung, đó là một cách tiếp cận tốt. Nhưng trong trường hợp của bạn, nếu bạn quay lại trước khi gọi scope.Complete(), nó sẽ chỉ làm thùng rác TransactionScope. Phụ thuộc vào thiết kế của bạn.

Vì vậy, trong mẫu này, Complete () không được gọi và phạm vi được xử lý, giả sử nó đang kế thừa giao diện IDisposable.

scope.Complete chắc chắn nên được gọi trước return. Trình biên dịch sẽ hiển thị cảnh báo và mã này sẽ không bao giờ được gọi.

Về returnbản thân - vâng, có thể an toàn khi gọi nó là usingcâu lệnh bên trong . Việc sử dụng được dịch sang khối try-final phía sau cảnh và cuối cùng khối chắc chắn được thực thi.

Trong ví dụ bạn đã cung cấp, có một vấn đề; scope.Complete()không bao giờ được gọi. Thứ hai, nó không phải là một thực hành tốt để sử dụng returncâu lệnh bên trong usingcâu lệnh. Tham khảo nội dung sau:

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

Trong ví dụ đơn giản này, vấn đề là; giá trị của scopesẽ là null khi kết thúc sử dụng câu lệnh.

Vì vậy, tốt hơn là không quay lại bên trong bằng cách sử dụng các câu lệnh.

Để đảm bảo rằng di scope.Complete()chúc được gọi, hãy bọc nó bằng try/finally. Các disposeđược gọi là bởi vì bạn phải quấn nó với usingđó là một sự thay thế try/finallykhối.

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

Trong ví dụ này, scope.Complete () sẽ không bao giờ thực thi. Tuy nhiên, lệnh return sẽ dọn dẹp mọi thứ được gán trên ngăn xếp. GC sẽ xử lý mọi thứ không được tham chiếu. Vì vậy, trừ khi có một đối tượng mà GC không thể nhặt được, không có vấn đề gì.