Include more null checks in transforming collections

Author: nicolaiparlog

src/main/java/org/codefx/libfx/collection/transform/AbstractTransformingList.java

@@ -139,6 +139,8 @@ public O set(int index, O element) {
 
 	@Override
 	public void replaceAll(UnaryOperator<O> operator) {
+		Objects.requireNonNull(operator, "The argument 'operator' must not be null.");
+
 		UnaryOperator<I> operatorOnInner = inner -> transformToInner(operator.apply(transformToOuter(inner)));
 		getInnerList().replaceAll(operatorOnInner);
 	}
@@ -153,6 +155,8 @@ public O remove(int index) {
 
 	@Override
 	public void sort(Comparator<? super O> comparator) {
+		Objects.requireNonNull(comparator, "The argument 'comparator' must not be null.");
+
 		Comparator<I> comparatorOfInner = (leftInner, rightInner) ->
 				comparator.compare(
 						transformToOuter(leftInner),

src/main/java/org/codefx/libfx/collection/transform/AbstractTransformingMap.java

@@ -173,13 +173,16 @@ public OV putIfAbsent(OK key, OV value) {
 
 	@Override
 	public void putAll(Map<? extends OK, ? extends OV> outerMap) {
+		Objects.requireNonNull(outerMap, "The argument 'outerMap' must not be null.");
+
 		Map<IK, IV> asInner = new TransformToReadOnlyInnerMap(outerMap);
 		getInnerMap().putAll(asInner);
 	}
 
 	@Override
 	public OV compute(OK key, BiFunction<? super OK, ? super OV, ? extends OV> remappingFunction) {
 		Objects.requireNonNull(remappingFunction, "The argument 'remappingFunction' must not be null.");
+
 		return transformToOuterValue(getInnerMap().compute(
 				transformToInnerKey(key),
 				transformToInnerKeyValueToValueFunction(remappingFunction)
@@ -189,6 +192,7 @@ public OV compute(OK key, BiFunction<? super OK, ? super OV, ? extends OV> remap
 	@Override
 	public OV computeIfAbsent(OK key, Function<? super OK, ? extends OV> mappingFunction) {
 		Objects.requireNonNull(mappingFunction, "The argument 'mappingFunction' must not be null.");
+
 		return transformToOuterValue(getInnerMap().computeIfAbsent(
 				transformToInnerKey(key),
 				transformToInnerToKeyValueFunction(mappingFunction)
@@ -198,6 +202,7 @@ public OV computeIfAbsent(OK key, Function<? super OK, ? extends OV> mappingFunc
 	@Override
 	public OV computeIfPresent(OK key, BiFunction<? super OK, ? super OV, ? extends OV> remappingFunction) {
 		Objects.requireNonNull(remappingFunction, "The argument 'remappingFunction' must not be null.");
+
 		return transformToOuterValue(getInnerMap().computeIfPresent(
 				transformToInnerKey(key),
 				transformToInnerKeyValueToValueFunction(remappingFunction)
@@ -207,6 +212,7 @@ public OV computeIfPresent(OK key, BiFunction<? super OK, ? super OV, ? extends
 	@Override
 	public OV merge(OK key, OV value, BiFunction<? super OV, ? super OV, ? extends OV> remappingFunction) {
 		Objects.requireNonNull(remappingFunction, "The argument 'remappingFunction' must not be null.");
+
 		return transformToOuterValue(getInnerMap().merge(
 				transformToInnerKey(key),
 				transformToInnerValue(value),
@@ -232,6 +238,8 @@ public boolean replace(OK key, OV oldValue, OV newValue) {
 
 	@Override
 	public void replaceAll(BiFunction<? super OK, ? super OV, ? extends OV> function) {
+		Objects.requireNonNull(function, "The argument 'function' must not be null.");
+
 		getInnerMap().replaceAll(transformToInnerKeyValueToValueFunction(function));
 	}
 
@@ -283,6 +291,7 @@ public void clear() {
 	@Override
 	public void forEach(BiConsumer<? super OK, ? super OV> action) {
 		Objects.requireNonNull(action, "The argument 'action' must not be null.");
+
 		getInnerMap().forEach(transformToInnerKeyValueConsumer(action));
 	}