Trả lại luồng bộ nhớ từ hàm

Question

Theo ý kiến ​​của bạn, tốt hơn là trả lại luồng bộ nhớ mới được cấp phát từ một hàm hoặc chuyển nó vào hàm? Ví dụ,

void Foo(MemoryStream m) 
{ 
    m.Write(somebuffer, 0, somebuffer.Length); 
} 

hoặc

void MemoryStream Foo() 
{ 
    MemoryStream retval = new MemoryStream(); 
    retval.Write(somebuffer, 0, somebuffer.Length); 
    return retval; 
} 

Answer 1

Điều này giống như hỏi xem bạn có nên trả về chuỗi từ phương thức hoặc lấy StringBuilder và nối thêm vào chuỗi đó hay không. Câu trả lời phụ thuộc vào trường hợp sử dụng là gì.

Có khả năng người gọi sẽ muốn gọi phương thức của bạn bằng luồng hiện có chứa một số dữ liệu? Có thể họ muốn gọi nó nhiều lần bằng cách sử dụng cùng một luồng? Nếu vậy, phiên bản dùng MemoryStream sẽ hiệu quả hơn. Mặt khác, nếu họ chỉ muốn dữ liệu một lần, trả về nó như là một MemoryStream (hoặc, đơn giản hơn, như là một mảng byte) cũng có thể thích hợp hơn.

Thật không may từ mô tả chúng tôi không thể thực sự biết điều gì đang xảy ra. Tất nhiên, bạn có thể thực hiện cả hai như quá tải và gọi một từ khác.

Answer 2

Bạn có thể yên tâm gửi lại từ hàm. Bạn phải gọi Dispose() hoặc đặt nó bên trong mệnh đề using vì nó triển khai IDisposable.

Answer 3

Câu hỏi thứ hai của bạn tốt hơn. Tôi luôn cố gắng tránh biến đổi các đối tượng bên trong các hàm nếu có thể.

Answer 4

Không có hai sự khác biệt. Trong trường hợp đầu tiên người gọi sẽ có thể kiểm soát các dòng bộ nhớ và trong trường hợp thứ hai, bạn có thể làm một cái gì đó như thế này:

using (MemoryStream ms = Foo()) 
{ 
    //Do Stuff here. 
} 

Điều quan trọng nhất là phải nhớ để xử lý nó một cách chính xác.

Answer 5

Chuyển luồng bộ nhớ vào một hàm và trả về một luồng bộ nhớ cho một hàm không được sử dụng thay thế cho nhau. Các phương pháp bạn mô tả phục vụ hai mục đích khác nhau.

• Việc chuyển đến chức năng là khi bạn muốn chức năng thực hiện điều gì đó với thông số.

• Trả lại nội dung nào đó từ một hàm là khi người gọi được yêu cầu làm điều gì đó với kết quả.

Bạn đang nói về hai thứ khác nhau, táo và cam.

Answer 6

Tôi sẽ luôn chuyển luồng vào hàm. Điều này cho phép nó hoạt động với bất kỳ luồng nào của người gọi đang chọn, ví dụ: thẳng vào một tệp mà không có bất kỳ bộ đệm nào.

Answer 7

Tôi thích biểu mẫu được tiêm hơn. Nó loại bỏ sự liên kết trực tiếp giữa mã của bạn và MemoryStream và nó làm cho nó dễ kiểm tra hơn.

public void Foo(Stream stream) 
{ 
    stream.Write(somebuffer, 0, somebuffer.Length); 
} 

Bây giờ tôi có thể kiểm tra Foo với bất kỳ lớp nào triển khai Luồng bao gồm lớp giả.

Thông thường, tôi sẽ thực hiện việc tiêm trong hàm tạo của lớp chứ không phải trên các phương thức riêng lẻ nhưng ý tưởng về cơ bản giống nhau.

public class FooClass 
{ 
    public Stream FooStream { get; private set; } 

    public FooClass() : this(null) { } 

    public FooClass(Stream stream) 
    { 
     // provide a default if not specified 
     this.FooStream = stream ?? new MemoryStream(); 
    } 

    public void Foo() 
    { 
     this.FooStream.Write(somebuffer, 0, somebuffer.Length); 
    } 
} 

Answer 8

tôi sẽ có xu hướng hướng tới đầu tiên vì hai lý do:

1. Ngữ nghĩa của người "sở hữu" các dòng bộ nhớ rất rõ ràng. Người gọi đã tạo ra nó, do đó, tùy thuộc vào người gọi để loại bỏ nó (đây là vấn đề với các loại luồng khác chứa tài nguyên không được quản lý). luồng của bạn

Điều đó nói rằng, nếu mục đích chính của Foo là phương pháp nhà máy cho MemoryStreams (tương tự như một cái gì đó như File.Open, v.v.), cách tiếp cận thứ hai có ý nghĩa hơn.

Answer 9

Sau một vài suy nghĩ khác, tôi nghĩ nó đi xuống đến ngữ nghĩa dự định của phương pháp Foo. Là nó:

• Một hoạt động tạo ra một dòng (ví dụ File.Open().)
• Một hoạt động mà sửa đổi một dòng (ví dụ something.WriteXml().)

Nếu câu trả lời là "tạo ra một dòng suối", có nó trở lại một dòng. Nếu nó sửa đổi luồng, hãy truyền luồng vào.

Nếu câu trả lời là "một số trong cả hai", thì có thể có ý nghĩa khi tách phương thức sao cho chỉ có một trách nhiệm.

Answer 10

Vì luồng là tài nguyên cần xử lý rõ ràng (bộ nhớ, tệp, mạng), cách tốt nhất là áp dụng phương pháp RAII để xử lý chúng. Điều đó có nghĩa rằng chức năng khởi tạo chúng phải chịu trách nhiệm với việc phát hành (chúng tôi đã "sử dụng" keyord trong C# chỉ cho điều đó). Để bật mẫu này, tôi nói chấp nhận luồng dưới dạng tham số. Bằng cách đó, người gọi có thể quyết định thời điểm tạo và xử lý luồng. Trong khi bạn đang ở đó, phương thức của bạn chấp nhận bất kỳ việc triển khai luồng nào; có vẻ như nó chỉ cần làm việc cho MemoryStream.