From 0544328c7965478e585be4b50781935bd7059d18 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Roland=20H=C3=A4der?= <roland@mxchange.org>
Date: Wed, 19 Oct 2022 17:08:05 +0200
Subject: [PATCH] Continued: - also validate these parameters, e.g. please
 don't set 2nd parameter "cause" to   NULL, but use the other method with one
 parameter instead!

---
 .../beans/local/logger/LoggerBean.java        | 132 ++++++++++++++++++
 1 file changed, 132 insertions(+)

diff --git a/src/java/org/mxchange/jcoreeelogger/beans/local/logger/LoggerBean.java b/src/java/org/mxchange/jcoreeelogger/beans/local/logger/LoggerBean.java
index 74d63eb..36d208a 100644
--- a/src/java/org/mxchange/jcoreeelogger/beans/local/logger/LoggerBean.java
+++ b/src/java/org/mxchange/jcoreeelogger/beans/local/logger/LoggerBean.java
@@ -50,78 +50,210 @@ public class LoggerBean implements LoggerBeanLocal {
 
 	@Override
 	public void logDebug (final String message) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().debug(message);
 	}
 
 	@Override
 	public void logDebug (final String message, final Throwable cause) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		} else if (null == cause) {
+			// Throw NPE again
+			throw new NullPointerException("cause is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().debug(message, cause);
 	}
 
 	@Override
 	public void logError (final String message) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().error(message);
 	}
 
 	@Override
 	public void logError (final String message, final Throwable cause) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		} else if (null == cause) {
+			// Throw NPE again
+			throw new NullPointerException("cause is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().error(message, cause);
 	}
 
 	@Override
 	public void logException (final Throwable throwable) {
+		// Validate parameter
+		if (null == throwable) {
+			// Throw NPE again
+			throw new NullPointerException("throwable is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().catching(throwable);
 	}
 
 	@Override
 	public void logFatal (final String message, final Throwable cause) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		} else if (null == cause) {
+			// Throw NPE again
+			throw new NullPointerException("cause is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().fatal(message, cause);
 	}
 
 	@Override
 	public void logFatal (final String message) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().fatal(message);
 	}
 
 	@Override
 	public void logInfo (final String message) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().info(message);
 	}
 
 	@Override
 	public void logInfo (final String message, final Throwable cause) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		} else if (null == cause) {
+			// Throw NPE again
+			throw new NullPointerException("cause is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().info(message, cause);
 	}
 
 	@Override
 	public void logTrace (final String message) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().trace(message);
 	}
 
 	@Override
 	public void logTrace (final String message, final Throwable cause) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		} else if (null == cause) {
+			// Throw NPE again
+			throw new NullPointerException("cause is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().trace(message, cause);
 	}
 
 	@Override
 	public void logWarning (final String message) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().warn(message);
 	}
 
 	@Override
 	public void logWarning (final String message, final Throwable cause) {
+		// Validate parameters
+		if (null == message) {
+			// Throw NPE
+			throw new NullPointerException("message is null"); //NOI18N
+		} else if (message.isEmpty()) {
+			// Throw IAE
+			throw new IllegalArgumentException("message is empty"); //NOI18N
+		} else if (null == cause) {
+			// Throw NPE again
+			throw new NullPointerException("cause is null"); //NOI18N
+		}
+
 		// Deligate to logger
 		this.getLogger().warn(message, cause);
 	}
-- 
2.39.5