C# - Làm cách nào để kiểm tra các câu lệnh scope.Complete() bị thiếu?

Question

Các lập trình viên trong nhóm của tôi đôi khi mở một giao dịch và quên bao gồm câu lệnh scope.Complete() (xem khối mã bên dưới). Bất kỳ ý tưởng về cách để một trong hai

(1) tìm kiếm giải pháp của chúng tôi cho thiếu scope.Complete() tuyên bố, hoặc

(2) có Visual Studio tự động làm nổi bật hoặc nâng cao một cảnh báo cho thiếu scope.Complete() báo cáo ?

Đây là dòng chúng tôi bỏ lỡ:

using(TransactionScope scope = new TransactionScope()) 
{ 
     /* Perform transactional work here */ 
     scope.Complete(); <-- we forget this line 
     /* Optionally, include a return statement */ 
} 

Những gì tôi đã cố gắng
 
Tôi đã cố gắng sử dụng một mẫu ReSharper Tuỳ chỉnh cho mục đích này, không có may mắn. Lý tưởng nhất là tôi sẽ tìm kiếm một cái gì đó như:

using(TransactionScope scope = new TransactionScope()) 
{ 
    $statements1$ 
    [^(scope.Complete();)] 
    $statements2$ 
} 

Tuy nhiên, ReSharper chỉ chấp nhận biểu thức thông thường để định danh, không cho phát biểu, vì vậy đây không xuất hiện để làm việc (http://www.jetbrains.com/resharper/webhelp/Reference__Search_with_Pattern.html).

Bất kỳ ý tưởng nào? Tôi cũng đang mở để sử dụng các plugin hoặc công cụ khác.

Cảm ơn,
Bến

Answer 1

NDepend chắc chắn có thể giúp đỡ, nhưng không thể kiểm tra 100% những gì bạn đang yêu cầu. NDepend không biết về phương thức body internals (thứ tự gọi phương thức). Vì vậy, tốt nhất, bạn có thể viết một code rule over LINQ (CQLinq) rằng sẽ kiểm tra xem nếu một phương pháp đang tạo ra một TransactionScope, ít nhất nó phải gọi TransactionScope.Complete():

warnif count > 0 
from m in Application.Methods 
where m.CreateA("System.Transactions.TransactionScope") && 
    !m.IsUsing("System.Transactions.TransactionScope.Complete()") 
select m 

Lưu ý rằng nếu các nhà phát triển đang xử lý kỷ luật, đủ để tránh tạo ra nhiều TransactionScope trong một phương pháp , quy tắc này sẽ phù hợp với bạn.

Answer 2

Ông có thể buộc các lập trình viên sử dụng một API tùy chỉnh thay vì những thứ scope.Complete cấp thấp?

Một đóng cửa sẽ buộc sử dụng .Complete():

public static void Do(this TransactionScope scope, Action action) { 
    using (scope) { 
    action(); 
    scope.Complete(); 
    } 
} 

Sau đó, bạn có thể làm:

new TransactionScope().Do(() => /* Transactional stuff */); 

Answer 3

Tôi không biết bất kỳ plugin R # hiện có nào để kiểm tra điều này, nhưng bạn chắc chắn có thể tạo một trong số của riêng bạn. Tất cả những gì bạn phải làm là phát hiện một câu lệnh bằng cách sử dụng một khai báo biến kiểu TransactionScope, sau đó lặp lại các câu lệnh có chứa tìm kiếm cuộc gọi Complete().

Nếu bạn quan tâm đến việc này, tôi khuyên bạn nên tải xuống ReSharper SDK và kiểm tra Plugin Development Guide.