C＃：ストリームを呼び出されたメソッドで閉じたり破棄したりする必要がありますか？

Question

私は、XML文書を含む文字列とXSDであるストリームの配列を受け入れるC＃で書いているメソッドを持っています。文字列の文書は、のXSDに対して検証されています

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 
    }; 
} 

私の質問は、この方法は、XSDの流れを処分しなければならない場合、またはそれは、呼び出し側の責任ですべきですか？

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

またはそれが（ValidateDocumentInternalにストリームを廃棄しない）ようにする必要があります：

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

または文書とXSDに渡すと期待しValidateDocumentInternalをXSDストリームを配置するには、次のコードを想像してみて代わりに、私はちょうどbool処分するかどうかを言って渡す必要がありますか？

Answer 1

私はそれが発信者の責任であると思います。それは、他の人が関数から与えたパラメータです。この関数は別のコンテキストで使用されているかどうかを知ることができず、それが行う変更は実際には「副作用」です...私が個人的に強くしているのは避けてください