chore: failed validation logging review #562

Author: fugerit79

fj-doc-val-core/src/main/java/org/fugerit/java/doc/val/core/basic/AbstractDocTypeValidator.java

@@ -16,11 +16,15 @@
 
 import lombok.Getter;
 import lombok.extern.slf4j.Slf4j;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 @Slf4j
 public abstract class AbstractDocTypeValidator implements DocTypeValidator {
 
-	@Override
+    private static final Logger logger = LoggerFactory.getLogger( AbstractDocTypeValidator.class );
+
+    @Override
 	public String toString() {
 		return this.getClass().getSimpleName()+"[mimeType=" + mimeType + ", supportedExtensions=" + supportedExtensions + "]";
 	}
@@ -74,4 +78,8 @@ public boolean checkCompatibility() {
 		return Boolean.TRUE;
 	}
 
+    protected void logFailedCheck( String type, Exception e ) {
+        logger.warn( "Failed check on {} : {}", type, e.toString() );
+    }
+
 }

fj-doc-val-poi/src/main/java/org/fugerit/java/doc/val/poi/DocValidator.java

@@ -6,13 +6,9 @@
 import org.fugerit.java.doc.val.core.DocTypeValidationResult;
 import org.fugerit.java.doc.val.core.DocTypeValidator;
 import org.fugerit.java.doc.val.core.basic.AbstractDocTypeValidator;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class DocValidator extends AbstractDocTypeValidator {
 
-	private static final Logger logger = LoggerFactory.getLogger( DocValidator.class );
-	
 	public static final String EXTENSION = "DOC";
 
 	public static final String MIME_TYPE = "application/msword";
@@ -29,7 +25,7 @@ public DocTypeValidationResult validate(InputStream is) {
 		try ( HWPFDocument workbook = new HWPFDocument( is ) ) {
 			result = DocTypeValidationResult.newOk();
 		} catch (Exception e) {
-			logger.warn( "Failed check on pdf : {}", e.toString() );
+			this.logFailedCheck( EXTENSION, e );
 			result.withMainException( e );
 		}
 		return result;

fj-doc-val-poi/src/main/java/org/fugerit/java/doc/val/poi/DocxValidator.java

@@ -6,13 +6,9 @@
 import org.fugerit.java.doc.val.core.DocTypeValidationResult;
 import org.fugerit.java.doc.val.core.DocTypeValidator;
 import org.fugerit.java.doc.val.core.basic.AbstractDocTypeValidator;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class DocxValidator extends AbstractDocTypeValidator {
 
-	private static final Logger logger = LoggerFactory.getLogger(DocxValidator.class);
-
 	public static final String EXTENSION = "DOCX";
 
 	public static final String MIME_TYPE = "application/vnd.openxmlformats-officedocument.wordprocessingml.document";
@@ -29,7 +25,7 @@ public DocTypeValidationResult validate(InputStream is) {
 		try (XWPFDocument workbook = new XWPFDocument(is)) {
 			result = DocTypeValidationResult.newOk();
 		} catch (Exception e) {
-			logger.warn("Failed check on pdf : {}", e.toString());
+            this.logFailedCheck( EXTENSION, e );
 			result.withMainException( e );
 		}
 		return result;

fj-doc-val-poi/src/main/java/org/fugerit/java/doc/val/poi/XlsValidator.java

@@ -6,13 +6,9 @@
 import org.fugerit.java.doc.val.core.DocTypeValidationResult;
 import org.fugerit.java.doc.val.core.DocTypeValidator;
 import org.fugerit.java.doc.val.core.basic.AbstractDocTypeValidator;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class XlsValidator extends AbstractDocTypeValidator {
 
-	private static final Logger logger = LoggerFactory.getLogger( XlsValidator.class );
-	
 	public static final String EXTENSION = "XLS";
 
 	public static final String MIME_TYPE = "application/vnd.ms-excel";
@@ -29,7 +25,7 @@ public DocTypeValidationResult validate(InputStream is) {
 		try ( HSSFWorkbook workbook = new HSSFWorkbook( is ) ) {
 			result = DocTypeValidationResult.newOk();
 		} catch (Exception e) {
-			logger.warn( "Failed check on pdf : {}", e.toString() );
+            this.logFailedCheck( EXTENSION, e );
 			result.withMainException( e );
 		}
 		return result;

fj-doc-val-poi/src/main/java/org/fugerit/java/doc/val/poi/XlsxValidator.java

@@ -6,13 +6,9 @@
 import org.fugerit.java.doc.val.core.DocTypeValidationResult;
 import org.fugerit.java.doc.val.core.DocTypeValidator;
 import org.fugerit.java.doc.val.core.basic.AbstractDocTypeValidator;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class XlsxValidator extends AbstractDocTypeValidator {
 
-	private static final Logger logger = LoggerFactory.getLogger( XlsxValidator.class );
-	
 	public static final String EXTENSION = "XLSX";
 
 	public static final String MIME_TYPE = "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet";
@@ -29,7 +25,7 @@ public DocTypeValidationResult validate(InputStream is) {
 		try ( XSSFWorkbook workbook = new XSSFWorkbook( is ) ) {
 			result = DocTypeValidationResult.newOk();
 		} catch (Exception e) {
-			logger.warn( "Failed check on pdf : {}", e.toString() );
+            this.logFailedCheck( EXTENSION, e );
 			result.withMainException( e );
 		}
 		return result;