Question

Q:

I want to break the loop if some exception has been caught in the event.

The code :

XmlDocument x = new XmlDocument();
x.Load(targetFileName);

XmlReaderSettings settings = new XmlReaderSettings();
settings.CloseInput = true;
settings.ValidationEventHandler += Handler;

settings.ValidationType = ValidationType.Schema;
settings.Schemas.Add(null,System.Web.HttpContext.Current.Server.MapPath("~/importSchema/IntialSchema.xsd"));
settings.ValidationFlags =
     XmlSchemaValidationFlags.ReportValidationWarnings |
XmlSchemaValidationFlags.ProcessIdentityConstraints |
XmlSchemaValidationFlags.ProcessInlineSchema |
XmlSchemaValidationFlags.ProcessSchemaLocation;

using (StreamReader str_reader = new StreamReader(targetFileName))
{

    using (XmlReader validatingReader = XmlReader.Create(str_reader, settings))
    {
        while (validatingReader.Read())
        {
            //I wanna to break this loop if there is some caught error .
        }
    }
}

---

private static void Handler(object sender, ValidationEventArgs e)
{

    if (e.Severity == XmlSeverityType.Error || e.Severity == XmlSeverityType.Warning)
    {
        try
        {
            throw new Exception(
                 String.Format("Line: {0}, Position: {1} \"{2}\"",
                     e.Exception.LineNumber, e.Exception.LinePosition, e.Exception.Message));
        }
        catch (Exception ee)
        {
            Common.ErrMappingForInformix.WriteLog(ee.Message);
            ScheduleForm sf = new ScheduleForm();
            sf.ShowStatus("Error,Invalid xml file", "error", "", 1);

        }

    }

}

---

# Edit 1 :

XmlDocument x = new XmlDocument();
x.Load(targetFileName);

XmlReaderSettings settings = new XmlReaderSettings();
settings.CloseInput = true;
settings.ValidationEventHandler += (senderValidation, ee) =>
{
    if (ee.Severity == XmlSeverityType.Error || ee.Severity == XmlSeverityType.Warning)
    {
        try
        {
            this.validationFailed = true;
            throw new Exception(
                 String.Format("Line: {0}, Position: {1} \"{2}\"",
                     ee.Exception.LineNumber, ee.Exception.LinePosition, ee.Exception.Message));
        }
        catch (Exception ex)
        {

            Common.ErrMappingForInformix.WriteLog(ex.Message);
            this.ShowStatus("Error", "error", "", 1);
        }

    }
};

settings.ValidationType = ValidationType.Schema;
settings.Schemas.Add(null, System.Web.HttpContext.Current.Server.MapPath("~/importSchema/IntialSchema.xsd"));
settings.ValidationFlags =
     XmlSchemaValidationFlags.ReportValidationWarnings |
XmlSchemaValidationFlags.ProcessIdentityConstraints |
XmlSchemaValidationFlags.ProcessInlineSchema |
XmlSchemaValidationFlags.ProcessSchemaLocation;

using (StreamReader str_reader = new StreamReader(targetFileName))
{

    using (XmlReader validatingReader = XmlReader.Create(str_reader, settings))
    {
        while (validatingReader.Read())
        {
            //Loop through the document
            if (validationFailed)
            {
                break;
            }

        }
    }
}

It breaks the loop but it doesn’t execute the event handler.

Answer 1

You could set a flag to indicate there’s an error in the Handler method, then detect that flag and use break; if it’s set.

In this case, it might be better to use an inline anonymous delegate for the validation handler, so you don’t need to add a new member variable to the class containing the methods. (Adding a member variable isn’t so bad if the class only deals with XML reading/validation)

Edit: anonymous inline delegate means you can add an event handler that doesn’t have to be in a separate method, basically. For example:

bool validationFailed = false; // outside of the handler
settings.ValidationEventHandler += (sender, e) =>
{
    if (e.Severity == XmlS....
    // set validationFailed = true; in here to signal failure
};

// then detect validationFailed being true to know if/when to break;