C#: Sollen Streams in der aufgerufenen Methode geschlossen/entsorgt werden?

Question

Ich habe eine Methode, die ich in C# schreibe, die eine Zeichenfolge akzeptiert, die ein XML-Dokument enthält, und ein Array von Streams, die XSDs sind. Das String Dokument richtet sich gegen die XSDs validiert:

private static XmlValidationResult ValidateDocumentInternal(string document, params Stream[] xsdStreams) 
{ 
    XmlReaderSettings settings = new XmlReaderSettings 
    { 
     ValidationType = ValidationType.Schema 
    }; 

    foreach (var xsdStream in xsdStreams) 
    { 
     using (xsdStream) 
     { 
      XmlReader xmlReader = XmlReader.Create(xsdStream); 

      try 
      { 
       settings.Schemas.Add(null, xmlReader); 
      } 
      finally 
      { 
       xmlReader.Close(); 
      } 
     } 
    } 

    var validationErrors = new List<string>(); 

    settings.ValidationEventHandler += (object sender, System.Xml.Schema.ValidationEventArgs e) => 
    { 
     validationErrors.Add($"({e.Exception.LineNumber}): {e.Message}"); 
    }; 

    using (var stream = document.ToStream()) 
    { 
     var reader = XmlReader.Create(stream, settings); 

     while (reader.Read()) 
     { 
     } 
    } 

    return new XmlValidationResult 
    { 
     Success = validationErrors.Count == 0, 
     ValidationErrors = validationErrors 
    }; 
} 

Meine Frage ist, sollte diese Methode des XSD Stromes sein entsorgen oder soll, dass die Verantwortung des Anrufers sein? Stellen Sie sich den folgenden Code, der in dem Dokument gelangt und XSD und erwartet ValidateDocumentInternal die XSD Strom zu entsorgen:

var document = GetDocument(); 
Stream xsd = GetXSD(); 
var validationResult = ValidateDocumentInternal(document, xsd); 

oder sie sein sollte, wie (nicht in ValidateDocumentInternal des Stroms Entsorgung):

var document = GetDocument(); 
using (Stream xsd = GetXSD()) { 
    var validationResult = = ValidateDocumentInternal(document, xsd); 
} 

oder alternativ sollte ich einfach in eine bool sagen, ob zu entsorgen oder nicht?

Answer 1

Ich denke, es ist die Verantwortung des Anrufers - es ist ein Parameter von der Funktion von jemand anderem gegeben. Die Funktion kann nicht wissen, ob es in einem anderen Kontext verwendet wird und und ändern, dass es tun wird ist eigentlich ein "Nebeneffekt" ... den ich persönlich stark versuche zu vermeiden