Список C # <Stream>удалить / закрыть

Question

Я настраиваю службу подписки для отправки отчетов различным людям в нашей компании по расписанию. Я планирую отправлять отчеты по электронной почте, и система отчетов, которую я использую, может экспортировать в виде потока PDF (вместо записи временных файлов). Большинство людей получат более одного отчета, поэтому я пытаюсь прикрепить их к одному письму, делая что-то вроде

List<Stream> reports = new List<Stream>();
//looping code for each users set of reports
Stream stream = ReportSource.ReportDocument.ExportToStream(PortableDocFormat)
reports.Add(stream);
stream.Flush();  //unsure
stream.Close();  //unsure
//end looping code

SmtpClient smtpClient = new SmtpClient(host, port);
MailMessage message = new MailMessage(from, to, subject, body);

foreach (Stream report in reports)
{
    message.Attachments.Add(new Attachment(report, "application/pdf"));
}                
smtpClient.Send(message);

В чем я не уверен, так это в том, что я должен очищать и закрывать поток сразу после добавления его в список. Или мне нужно потом зациклить список для очистки и удаления? Я пытаюсь избежать любой возможной утечки памяти.

Answer 1

Почему бы не создать класс StreamCollection, который реализует IDisposable:

public class StreamCollection : Collection<Stream>, IDisposable { }

В методе Dispose этого класса вы можете перебрать все потоки и правильно закрыть / удалить каждый поток. Тогда ваш код будет выглядеть так:

using (var reports = new StreamCollection())
{
   //looping code for each users set of reports
   reports.Add(ReportSource.ReportDocument.ExportToStream(PortableDocFormat));
   //end looping codeSmtpClient 

   smtpClient = new SmtpClient(host, port);
   MailMessage message = new MailMessage(from, to, subject, body);

   foreach (Stream report in reports)
   {    
      message.Attachments.Add(new Attachment(report, "application/pdf"));
   }

   smtpClient.Send(message);
}

Answer 2

Вы можете создать DisposableList, который вы можете заключить в оператор using:

public class DisposableList<T> : List<T>, IDisposable where T : IDisposable {

    // any constructors you need...

    public void Dispose() {
        foreach (T obj in this) {
            obj.Dispose();
        }
    }
}

Answer 3

Я не вижу логики в закрытии потоков сразу после добавления его в список. Судя по предоставленному вами коду, ссылки на эти потоки используются в других местах. Если потоки закрыты, то что хорошего они служат?

Answer 4

Зависит от того, используются ли потоки позже при создании вложения. Я предполагаю, что это означает, что вы захотите расположить потоки в конце.

Не забудьте попробовать, наконец, это. В противном случае они не будут утилизированы в случае возникновения исключения.

Answer 5

Нет никакого вреда при выполнении Flush () / Close (). Если вы хотите быть абсолютно уверенным, то вы должны сделать , используя выражение:

using (Stream stream = ReportSource.ReportDocument.ExportToStream(PortableDocFormat))
{
    reports.Add(stream);
    stream.Flush();  //unsure
}

Таким образом, исключения не повлияют на ваш код.