Απλά θα ήθελα τα σχόλιά σας. Ο κώδικας αυτός προϋποθέτει οτι έχεις δύο IO.Streams, ένα για το xml και ένα για το xsd και τα κάνει validate. (Τα είχα embedded σε dlls). Αν σας αρέσει, να το βάλουμε στο library (οταν και αν φτιαχτεί). Αν οχι, πείτε μου τι δεν.

'Method to validate an XML file using a given XSD file.
'Both files must be in the form of System.IO.Streams
Public Shared Sub ValidateXML( _
   ByVal xmlSettingsStream As Stream _
   , ByVal xsdStream As Stream)

   Dim tr As XmlTextReader = New XmlTextReader(xmlSettingsStream)
   Dim sc As XmlSchemaCollection = New XmlSchemaCollection
   Dim vr As XmlValidatingReader = New XmlValidatingReader(tr)
   Dim objTrXsd As XmlTextReader = New XmlTextReader(xsdStream)

   Try
      sc.Add(Nothing, objTrXsd)
      vr.ValidationType = ValidationType.Schema
      vr.Schemas.Add(sc)
      
      While (vr.Read())
      End While

   Catch ex As Exception
   
      'Rethrow exception to caller. Expected XMLException here.
      Throw (ex)
   
   Finally
   
   'Cleanup
      Try
         vr = Nothing
         sc = Nothing
         tr = Nothing
      
      Catch exc As Exception
         'do nothing
      End Try
   End Try





End Sub




 

cap wrote:

...    Catch ex As Exception           'Rethrow exception to caller. Expected XMLException here.       Throw (ex)        Finally ...

Διορθώστε με αν κάνω λάθος, αλλά νομίζω ότι εφ' όσον δεν πρόκειται να κάνεις τίποτα για το exception και θέλεις απλά να το κάνεις propagate, είναι καλύτερο να κάνεις απλά Throw (χωρίς argument) έστι ώστε να μην χάσεις το call stack από τα exceptions.

Εναλλακτικά, μπορείς να κάνεις
Τhrow(new Exception("my error info",ex))
έτσι ώστε και πάλι να διατηρήσεις το αρχικό exception ως inner exception πλέον

Αν θες οπωσδήποτε παρατηρήσεις:

α) δεν νομίζω ότι απαιτείται το finally clean-up αφού όλα τα objects που χρησιμοποιείς είναι managed.

β) βάλε τα variable declarations και initializations μέσα στο try/catch block ώστε να πιάσεις και τα errors που μπορεί να συμβούν κατά το initialization των variables.

γ) μετέτραψε το sub σε ένα function που να επιστρέφει true ή false αντί να πετάς exceptions. Στην περίπτωση αυτή μπορείς να προσθέσεις μια τελευταία bool παράμετρο στο function που να καθορίζει τι θα κάνεις με τα exceptions.

π.χ. (σε C#)

public bool ValidateXML(Stream xml, Stream xsd, bool throwMeAllExceptions)
{
      try
      {
            <do your stuff>
      }
      catch (Exception e)
      {
            if (throwMeAllExceptions)
                  throw;
            else return false;
      }
      return true;
}

και προσθέτεις και ένα orverload ώστε να είναι optional η bool (νομίζω ότι σε VB μπρορείς να την κάνεις optional με κάποιο keyword)

public bool ValidateXML(Stream xml, Stream xsd)
{
      return ValidateXML(xml, xsd, false);
}

δ) αν θες να το κάνεις ακόμα καλύτερο μπορείς να πιάνεις χωριστά τα XMLExceptions από όλα τα υπόλοιπα οπότε αν πρόκειται για Validation πρόβλημα και σου έχει δώσει throwMeAllExceptions = false τότε του επιστρέφεις false για validation errors αλλά του πετάς όλα τα άλλα exceptions δηλαδή

public bool ValidateXML(Stream xml, Stream xsd, bool throwMeAllExceptions)
{
      try
      {
            <do your stuff>
      }
      catch (XMLException e)
      {
            if (throwMeAllExceptions)
                  throw;
            else return false;
      }
      return true;
}

οπότε θα το πιάσεις μόνο αν είναι XMLException. Όλα τα άλλα θα πάνε στους από πάνω handlers. Αντίστοιχα μπροείς να προσθέσεις ένα σκέτο catch{} το οποίοα θα πιάνει όλα τα υπόλοιπα exceptions και θα τα "πνίγει".

αυτά..

Είπα να ποστάρω εδώ το πρόβλημά μου, μιας και είναι κοινό το θέμα. Προσπαθώ να φτιάξω μια εφαρμογούλα που να διαβάζει xml documents και να κάνει κάποιους υπολογισμούς. Υπάρχει δύο απαιτήσεις όμως:

1)      Εάν το xml δεν είναι σύμφωνο με το αντίστοιχο xsd (validation fails), να μην επεξεργάζεται το αρχείο

2)      Να μπορούν να τρέξουν πολλά instances της εφαρμογής παράλληλα για διαφορετικά αρχεία.

Όσες απόπειρες για xml validation έχω δει στο νετ χρησιμοποιούν shared variables & methods, κάτι που ενδεχομένως όμως προκαλεί πρόβλημα για multiple instancing.

Έχει κανείς να προτείνει κάποιον εναλλακτικό τρόπο για xml validation?

Thanks in advance…

Εδώ που τα λέμε, δεν χρειάζεται καν το Catch αφού δεν πρόκειται να κάνεις κάτι με το exception. Επίσης, το καθάρισμα των XmlTextReader, XMLValidatingReader μπορείς να το κάνεις με ένα using, οπότε δεν χρειάζεται ούτε το Finally.