如何正確處理用LINQ創建的對象

Question

我有以下創建一次性對象實例的方法。

Public Overridable Sub TransformXmlDocumentsToFileStream(ByVal stream As System.IO.Stream, ByVal xmlDocuments As IEnumerable(Of String), ByVal transformContext As XslTransformContext) 
    Dim readers As IEnumerable(Of XmlReader) = _ 
     (From document In xmlDocuments _ 
     Select XmlReader.Create(New System.IO.StringReader(document))) 

    With transformContext 
     TransformXmlDocumentsToFileStream(stream, readers, transformContext) 
    End With 
End Sub 

然後我遍歷對象在另一種方法中：

For Each reader In readers 
    Using reader 
     transform.Transform(reader, writer) 
    End Using 
Next 

在Visual Studio代碼分析器被給予警告：

CA2000：Microsoft.Reliability：在方法「TransformHelper .TransformXmlDocumentsToFileStream（Stream，IEnumerable（Of String），XslTransformContext）'，對象'New StringReader（document）'不會沿着所有異常路徑放置。調用System.IDisposable.Dispose在對象'New StringReader（document）'之前，所有對它的引用超出範圍。

由於沒有對StringReader沒有提到，我不能把它在使用塊或以其他方式處置它。可以忽略這個警告嗎？ StringReader應該在讀者超出範圍並被垃圾收集時處理，是否正確？

Answer 1

對不起，在C＃中的答案。管理IDisposables的一個簡單方法是使調用new的方法也調用Dispose。

List<StringReader> stringReaders = new List<StringReader>(); 
    //note: Select is deferred. stringReaders is captured. 
IEnumerable<XmlReader> readers = 
    xmlDocuments.Select(document => 
    { 
    StringReader sr = new System.IO.StringReader(document); 
    stringReaders.Add(sr); 
    return XmlReader.Create(sr); 
    }); 
try 
{ 
    //this method enumerates readers, causing all of the allocations. 
    TransformXmlDocumentsToFileStream(stream, readers, transformContext); 
} 
finally 
{ 
    foreach(StringReader x in stringReaders) 
    { 
    x.Dispose(); 
    } 
} 

Answer 2

Dispose當對象恰好是垃圾回收，這可能是沒有或對象之後很長一段時間超出範圍時調用。您應該明確地處理您的StringReader。

如果我讓你可以像這樣

Dim stringReaders As IEnumerable(Of System.IO.StringReader) = _ 
     (From document In xmlDocuments _ 
     Select New System.IO.StringReader(document)).ToList() 

    Try 
     Dim readers As IEnumerable(Of XmlReader) = _ 
      (From stringReader In stringReaders _ 
      Select XmlReader.Create(stringReader)) 
     With transformContext 
      TransformXmlDocumentsToFileStream(stream, readers, transformContext) 
     End With 

    Finally 
     For Each stringReader In stringReaders 
      stringReader.Dispose() 
     Next 
    End Try