là tốt để sử dụng cố gắng cuối cùng chỉ để đảm bảo rằng một cái gì đó được thực hiện khi phương pháp kết thúc?

Question

Phương thức của tôi trả về ở nhiều điểm. Tôi xây dựng newData trong quá trình thực hiện cũng ở nhiều điểm. Bất kể nơi tôi trở về tôi cần phải lưu và lưu trữ kết quả xây dựng. Không bỏ lỡ "return" Tôi chỉ bao quanh mã với khối try-finally vì vậy bây giờ tôi chắc chắn rằng newData sẽ được lưu trữ.

List<X> newData = new List<X>(); 
try 
{ 
    .... 
    update newData 
    ..... 
    return; 
    ..... 
    .... 
    update newData 
    .... 
    update newData 
    return; 
    ..... 
    return; 
} finally 
{ 
    // copy newData to data  
} 

Nhưng tôi không bắt bất kỳ ngoại lệ nào và mã này không có ý định làm việc với ngoại lệ. Có thể chấp nhận được nói chung hoặc bạn có thể đề xuất một cách tiếp cận tốt hơn?

Answer 1

tôi sẽ đề nghị refactoring mã trong khối try vào một phương pháp mới:

data = CreateList(); 

... 

private List<X> CreateList() 
{ 
    List<X> list = new List<X>(); 
    // It's fine to return list from any point here... 
} 

Answer 2

Việc sử dụng cuối cùng được dự định như là một bản sao lưu nếu có điều gì không (nghĩ về nó như một cơ chế không an toàn), bao gồm cả nguyên tử, vv ..

Nói chung việc xây dựng phương pháp của bạn là sai và bạn có thể cấu trúc lại nó, vì nhận được tất cả những lợi nhuận đó thường có nghĩa là bạn có thể tiếp cận một số thứ khác (ví dụ: sử dụng công tắc như ai đó trong các nhận xét được đề xuất) .