Có cần thiết phải bao bọc StreamWriter trong một khối sử dụng không?

Question

Một vài ngày trước, tôi đăng tải một số mã như thế này:

StreamWriter writer = new StreamWriter(Response.OutputStream); 
writer.WriteLine("col1,col2,col3"); 
writer.WriteLine("1,2,3"); 
writer.Close(); 
Response.End(); 

tôi đã nói rằng thay vì tôi nên quấn StreamWriter trong một khối sử dụng trong trường hợp ngoại lệ. Thay đổi như vậy sẽ làm cho nó trông như thế này:

using(StreamWriter writer = new StreamWriter(Response.OutputStream)) 
{ 
    writer.WriteLine("col1,col2,col3"); 
    writer.WriteLine("1,2,3"); 
    writer.Close(); //not necessary I think... end of using block should close writer 
} 
Response.End(); 

Tôi không chắc chắn tại sao đây là một thay đổi có giá trị. Nếu một ngoại lệ xảy ra mà không có khối sử dụng, người viết và phản hồi sẽ vẫn được dọn dẹp, đúng không? Khối sử dụng đạt được gì cho tôi?

Answer 1

Nope luồng sẽ vẫn mở trong ví dụ đầu tiên, vì lỗi sẽ phủ nhận việc đóng.

Toán tử sử dụng buộc gọi Dispose() được cho là để làm sạch đối tượng lên và đóng tất cả các kết nối đang mở khi nó thoát khỏi khối.

Answer 2

Nếu trường hợp ngoại lệ xảy ra mà không có khối sử dụng và giết chương trình, bạn sẽ bị ngắt kết nối. Khối sử dụng sẽ luôn đóng kết nối cho bạn, tương tự như khi bạn sử dụng try {} catch {} cuối cùng {}

Answer 3

Theo ý kiến ​​của tôi, cần phải bọc bất kỳ lớp nào triển khai IDisposable trong khối sử dụng. Thực tế là một lớp thực hiện IDisposable có nghĩa là lớp đó có các tài nguyên cần được làm sạch.

Answer 4

sử dụng các cuộc gọi chặn bỏ() khi kết thúc. Nó chỉ là một cách tiện dụng để đảm bảo rằng các nguồn lực được làm sạch một cách kịp thời.

Answer 5

Trong hầu hết mọi trường hợp, nếu một lớp thực hiện IDisposable, và nếu bạn đang tạo một thể hiện của lớp đó, thì bạn cần khối sử dụng.

Answer 6

Cuối cùng, người viết sẽ được dọn dẹp. Khi điều này xảy ra là tùy thuộc vào bộ thu gom rác, người sẽ nhận thấy rằng Dispose cho lệnh chưa được gọi và gọi nó. Tất nhiên, GC có thể không chạy trong vài phút, vài giờ hoặc vài ngày tùy theo tình hình. Nếu người viết đang nắm giữ một khóa độc quyền trên nói, một tập tin, không có quá trình khác sẽ có thể mở nó, mặc dù bạn đã hoàn thành lâu dài.

Khối sử dụng đảm bảo cuộc gọi Dispose luôn được thực hiện và do đó luôn luôn gọi Close, bất kể dòng điều khiển nào xảy ra.

Answer 7

Bao bì các StreamWriter trong một khối using là khá nhiều tương đương với đoạn mã sau:

StreamWriter writer; 
try 
{ 
    writer = new StreamWriter(Response.OutputStream); 
    writer.WriteLine("col1,col2,col3"); 
    writer.WriteLine("1,2,3"); 
} 
catch 
{ 
    throw; 
} 
finally 
{ 
    if (writer != null) 
    { 
     writer.Close();  
    } 
} 

Trong khi bạn rất tốt có thể viết mã này chính mình, nó là như vậy dễ dàng hơn để chỉ cần đặt nó trong một khối sử dụng .

Answer 8

Quy tắc chung của tôi là, nếu tôi thấy Vứt bỏ được liệt kê trong intellisense, tôi bọc nó trong một khối sử dụng.

Answer 9

Trong khi thực hành tốt là luôn luôn nhúng các lớp dùng một lần như StreamWriter, như những người khác chỉ ra, trong trường hợp này không quan trọng.

Response.OutputStream sẽ được xử lý bởi cơ sở hạ tầng ASP.NET khi nó xử lý xong yêu cầu của bạn.

StreamWriter giả định nó "sở hữu" Luồng được truyền tới hàm tạo và do đó sẽ đóng luồng khi được xử lý.Nhưng trong mẫu bạn cung cấp, luồng đã được khởi tạo bên ngoài mã của bạn, vì vậy sẽ có một chủ sở hữu khác chịu trách nhiệm dọn dẹp.

Answer 10

Tôi sẽ đưa ra ý kiến ​​bất đồng ý kiến. Câu trả lời cho câu hỏi cụ thể "Có cần thiết phải bao bọc StreamWriter trong một khối sử dụng không?" Trên thực tế, bạn không được cuộc gọi Vứt bỏ trên một StreamWriter, bởi vì Vứt bỏ nó được thiết kế tồi và làm điều sai trái.

Sự cố với StreamWriter là, khi bạn Vứt bỏ nó, nó sẽ loại bỏ luồng cơ bản. Nếu bạn đã tạo StreamWriter với tên tệp, và nó đã tạo ra một FileStream riêng trong nội bộ, thì hành vi này sẽ hoàn toàn thích hợp. Nhưng nếu, như ở đây, bạn đã tạo StreamWriter với luồng hiện tại, thì hành vi này hoàn toàn là Điều sai (tm). Nhưng nó vẫn làm.

Mã như thế này sẽ không làm việc:

var stream = new MemoryStream(); 
using (var writer = new StreamWriter(stream)) { ... } 
stream.Position = 0; 
using (var reader = new StreamReader(stream)) { ... } 

vì khi using khối của StreamWriter disposes StreamWriter, mà sẽ lần lượt vứt bỏ con suối. Vì vậy, khi bạn cố gắng đọc từ luồng, bạn sẽ nhận được một ObjectDisposedException.

StreamWriter là vi phạm nghiêm trọng quy tắc "dọn dẹp mớ hỗn độn của riêng bạn". Nó cố gắng làm sạch đống lộn xộn của người khác, cho dù họ có muốn hay không.

(Hãy tưởng tượng nếu bạn thử điều này trong thực tế đời sống. Hãy thử giải thích với cảnh sát tại sao bạn đã đột nhập vào nhà của người khác và bắt đầu ném tất cả các công cụ của họ vào thùng rác ...)

Vì lý do đó, tôi xem xét StreamWriter (và StreamReader, mà làm điều tương tự) là một trong số rất ít các lớp học, nơi "nếu nó thực hiện IDisposable, bạn nên gọi Dispose" là sai. Không bao giờ gọi Vứt bỏ trên StreamWriter được tạo trên luồng hiện có. Gọi Flush() để thay thế.

Sau đó, chỉ cần đảm bảo bạn dọn sạch Luồng khi cần. (Như Joe đã chỉ ra, ASP.NET phân phối Response.OutputStream cho bạn, vì vậy bạn không cần phải lo lắng về nó ở đây.)

Cảnh báo: nếu bạn không loại bỏ StreamWriter, thì bạn do cần gọi Flush() khi bạn viết xong. Nếu không, bạn có thể có dữ liệu vẫn đang được đệm trong bộ nhớ mà không bao giờ làm cho nó vào dòng đầu ra.

Quy tắc của tôi cho StreamReader là, giả vờ rằng nó không triển khai IDisposable. Chỉ cần để nó đi khi bạn đã hoàn tất.

Quy tắc của tôi cho StreamWriter là, hãy gọi Dọn dẹp nơi bạn có thể gọi là Vứt bỏ. (Điều này có nghĩa là bạn phải sử dụng số try .. finally thay vì using.)