From 1c8bc7132d980c1ff2dbd6b9af579c3b2fd8c63e Mon Sep 17 00:00:00 2001
From: bculkin2442 <bjculkin@mix.wvu.edu>
Date: Sun, 3 Apr 2016 19:22:48 -0400
Subject: General code refactoring and maintenance

---
 .../src/main/java/bjc/utils/data/GenHolder.java      | 20 ++++++++++++++++++++
 1 file changed, 20 insertions(+)

(limited to 'BJC-Utils2/src/main/java/bjc/utils/data/GenHolder.java')

diff --git a/BJC-Utils2/src/main/java/bjc/utils/data/GenHolder.java b/BJC-Utils2/src/main/java/bjc/utils/data/GenHolder.java
index 6854440..e042554 100644
--- a/BJC-Utils2/src/main/java/bjc/utils/data/GenHolder.java
+++ b/BJC-Utils2/src/main/java/bjc/utils/data/GenHolder.java
@@ -46,6 +46,10 @@ public class GenHolder<T> implements IHolder<T> {
 	 */
 	@Override
 	public void doWith(Consumer<T> action) {
+		if (action == null) {
+			throw new NullPointerException("Action must be non-null");
+		}
+
 		action.accept(heldValue);
 	}
 
@@ -56,6 +60,10 @@ public class GenHolder<T> implements IHolder<T> {
 	 */
 	@Override
 	public <NewT> IHolder<NewT> map(Function<T, NewT> transformer) {
+		if (transformer == null) {
+			throw new NullPointerException("Transformer must be non-null");
+		}
+
 		return new GenHolder<>(transformer.apply(heldValue));
 	}
 
@@ -66,6 +74,10 @@ public class GenHolder<T> implements IHolder<T> {
 	 */
 	@Override
 	public IHolder<T> transform(Function<T, T> transformer) {
+		if (transformer == null) {
+			throw new NullPointerException("Transformer must be non-null");
+		}
+
 		heldValue = transformer.apply(heldValue);
 
 		return this;
@@ -78,11 +90,19 @@ public class GenHolder<T> implements IHolder<T> {
 	 */
 	@Override
 	public <E> E unwrap(Function<T, E> unwrapper) {
+		if (unwrapper == null) {
+			throw new NullPointerException("Unwrapper must be null");
+		}
+
 		return unwrapper.apply(heldValue);
 	}
 
 	@Override
 	public String toString() {
+		if (heldValue == null) {
+			return "(null)";
+		}
+
 		return heldValue.toString();
 	}
 }
-- 
cgit v1.2.3