Почему StreamWriter должен быть открыт для доступа к моему MemoryStream?

Question

У меня есть тестовый код, который готовит MemoryStream, который в конечном итоге будет считаться объектом. Вот как я хочу, чтобы писать:

 var manager = new LeaderboardImportManager(leaderboard); 
     var columnNames = manager.ColumnNames; 

     var stream = new MemoryStream(); 
     using (var writer = new StreamWriter(stream)) 
     { 
      writer.WriteLine(string.Join(",", columnNames)); 
      foreach (var user in users) 
      { 
       var row = leaderboard.Metrics.Select(m => Faker.RandomNumber.Next().ToString()).ToList(); 
       row.Insert(0, user.UserName); 
       writer.WriteLine(string.Join(",", row)); 
      } 

      writer.Flush(); 
      stream.Position = 0; 
     } 

     return stream; 

Но когда я делаю это таким образом, мой объект поток становится нечитаемым, и мой тест не пройден, так что я должен сделать это следующим образом:

 var manager = new LeaderboardImportManager(leaderboard); 
     var columnNames = manager.ColumnNames; 

     var stream = new MemoryStream(); 
     var writer = new StreamWriter(stream); 

     writer.WriteLine(string.Join(",", columnNames)); 
     foreach (var user in users) 
     { 
      var row = leaderboard.Metrics.Select(m => Faker.RandomNumber.Next().ToString()).ToList(); 
      row.Insert(0, user.UserName); 
      writer.WriteLine(string.Join(",", row)); 
     } 

     writer.Flush(); 
     stream.Position = 0; 
     return stream; 

Это, конечно, мешает мне избавиться от моего объекта StreamWriter, который, как я понимаю, обязательно должен быть уничтожен.

Почему StreamWriter должен оставаться открытым, если я уже сбросил его содержимое на объект MemoryStream?

Я могу придумать некоторые очень неудобные способы обойти это, но я хотел бы знать, почему это не работает так, как я этого хочу, и есть ли что-то, что я могу сделать, чтобы заставить его работать путь. Любые советы приветствуются, спасибо!

Answer 1

Другие ответы указывают, что я должен использовать конструктор с параметром leaveOpen и установить его в true, но мне это не нравится, потому что конструктор также требует аргумент bufferSize.

Однако я понял, что я могу уйти с этим так же легко:

 // new method body, returns byte array 

     var stream = new MemoryStream(); 
     using (var writer = new StreamWriter(stream)) 
     { 
      writer.WriteLine(string.Join(",", columnNames)); 
      foreach (var user in users) 
      { 
       var row = leaderboard.Metrics.Select(m => Faker.RandomNumber.Next().ToString()).ToList(); 
       row.Insert(0, user.UserName); 
       writer.WriteLine(string.Join(",", row)); 
      } 

      writer.Flush(); 
      stream.Position = 0; 
     } 

     return stream.ToArray(); 

     // consumer opens a new stream using the bytes 

     using (var stream = new MemoryStream(this.GetCSVStream(leaderboard, users))) 
     { 
      mockFile.Setup(f => f.InputStream).Returns(stream); 
      this.service.UpdateEntries(update.ID, mockFile.Object); 
     } 

Answer 2

using вызов оператора Dispose метод на объект перед выходом из операционного блока { }.

В распоряжении StreamWriter имеется также Stream.

Это означает, что ваш Stream является недействительным объектом, прежде чем он будет возвращен.

Применить заявление using только для объектов, созданных и уничтоженных в текущей области действия (не возвращайте их как минимум).

Почему StreamWriter должен оставаться открытым, если я уже сбросил его содержимое в объект MemoryStream?

Как @mikez упоминалось, по умолчанию создается StreamWriter «владеет» основной поток, но вы можете избежать этого путем добавления leaveOpen = true в конструкторе.

new StreamWriter(stream = s, encoding = Encoding.UTF8, bufferSize = 128, leaveOpen = true) 

Answer 3

По умолчанию StreamWriter «владеет» поток он передается и закрыть его утилизировать. Используйте конструктор с leaveOpen boolean parameter. Установите его в true, чтобы не закрывать базовый Stream, когда автор находится в вашем первом примере.

Answer 4

StreamWriter автоматически закрывает Stream, если вы этого не скажете. Создайте его, как это вместо того, чтобы оставить Stream открытыми:

using (var writer = new StreamWriter(stream, System.Text.Encoding.UTF8, 1024, true)) 

В качестве альтернативы, если вы не хотите, чтобы передать дополнительные аргументы, используйте GetBuffer для доступа MemoryStream внутреннего буфера. Избегайте ToArray, так как это создает копию данных и в зависимости от вашего сценария может быть неэффективным.