Merge pull request #11 from ruke47/jul-logger

nemonik · web-flow · commit e1800bec033d · 2017-09-27T08:52:16.000-04:00
Use JUL logger instead of System.{out,err}.println
diff --git a/src/main/java/org/mitre/stix/ValidationErrorHandler.java b/src/main/java/org/mitre/stix/ValidationErrorHandler.java
@@ -8,20 +8,23 @@
 import org.xml.sax.SAXParseException;
 import org.xml.sax.SAXException;
 
+import java.util.logging.Logger;
+
 /**
  * Parsing and validating error handler
  * 
  * @author nemonik (Michael Joseph Walsh <github.com@nemonik.com>) *
  */
 public class ValidationErrorHandler implements ErrorHandler {
-
+	private static final Logger LOGGER = Logger.getLogger(ValidationErrorHandler.class.getName());
+	
 	private void log(String type, SAXParseException e) {
-		System.err.println("SAXParseException " + type);
-		System.err.println("\tPublic ID: " + e.getPublicId());
-		System.err.println("\tSystem ID: " + e.getSystemId());
-		System.err.println("\tLine     : " + e.getLineNumber());
-		System.err.println("\tColumn   : " + e.getColumnNumber());
-		System.err.println("\tMessage  : " + e.getMessage());
+		LOGGER.warning("SAXParseException " + type);
+		LOGGER.warning("\tPublic ID: " + e.getPublicId());
+		LOGGER.warning("\tSystem ID: " + e.getSystemId());
+		LOGGER.warning("\tLine     : " + e.getLineNumber());
+		LOGGER.warning("\tColumn   : " + e.getColumnNumber());
+		LOGGER.warning("\tMessage  : " + e.getMessage());
 	}
 
 	/* (non-Javadoc)
@@ -50,4 +53,4 @@ public void warning(SAXParseException e) throws SAXException {
 		log("WARNING", e);
 		throw e;
 	}
-}
+}
diff --git a/src/main/java/org/mitre/stix/ValidationEventHandler.java b/src/main/java/org/mitre/stix/ValidationEventHandler.java
@@ -5,26 +5,29 @@
 package org.mitre.stix;
 
 import javax.xml.bind.ValidationEvent;
+import java.util.logging.Logger;
 
 public class ValidationEventHandler implements javax.xml.bind.ValidationEventHandler {
+	private static final Logger LOGGER = Logger.getLogger(ValidationErrorHandler.class.getName());
+
 	/* (non-Javadoc)
 	 * @see javax.xml.bind.ValidationEventHandler#handleEvent(javax.xml.bind.ValidationEvent)
 	 */
 	public boolean handleEvent(ValidationEvent event) {
-		System.out.println("");
-		System.out.println("EventT");
-		System.out.println("\tSeverity: " + event.getSeverity());
-		System.out.println("\tMessage: " + event.getMessage());
-		System.out.println("\tLinked Excpetion: " + event.getLinkedException());
-		System.out.println("\tLocator");
-		System.out.println("\tLine Number: "
+		LOGGER.info("");
+		LOGGER.info("EventT");
+		LOGGER.info("\tSeverity: " + event.getSeverity());
+		LOGGER.info("\tMessage: " + event.getMessage());
+		LOGGER.info("\tLinked Excpetion: " + event.getLinkedException());
+		LOGGER.info("\tLocator");
+		LOGGER.info("\tLine Number: "
 				+ event.getLocator().getLineNumber());
-		System.out.println("\tColumn Number: "
+		LOGGER.info("\tColumn Number: "
 				+ event.getLocator().getColumnNumber());
-		System.out.println("\tOffset: " + event.getLocator().getOffset());
-		System.out.println("\tObject: " + event.getLocator().getObject());
-		System.out.println("\tNode: " + event.getLocator().getNode());
-		System.out.println("\tURL: " + event.getLocator().getURL());
+		LOGGER.info("\tOffset: " + event.getLocator().getOffset());
+		LOGGER.info("\tObject: " + event.getLocator().getObject());
+		LOGGER.info("\tNode: " + event.getLocator().getNode());
+		LOGGER.info("\tURL: " + event.getLocator().getURL());
 		return true;
 	}
-}
+}
