Use Java 8 forEach method on Map

Author: jonathanborenstein

spring-beans/src/main/java/org/springframework/beans/factory/BeanFactoryUtils.java

@@ -264,12 +264,12 @@ public static <T> Map<String, T> beansOfTypeIncludingAncestors(ListableBeanFacto
 			if (hbf.getParentBeanFactory() instanceof ListableBeanFactory) {
 				Map<String, T> parentResult = beansOfTypeIncludingAncestors(
 						(ListableBeanFactory) hbf.getParentBeanFactory(), type);
-				for (Map.Entry<String, T> entry : parentResult.entrySet()) {
+				parentResult.entrySet().forEach(entry -> {
 					String beanName = entry.getKey();
 					if (!result.containsKey(beanName) && !hbf.containsLocalBean(beanName)) {
 						result.put(beanName, entry.getValue());
-					}
-				}
+					}	
+				});
 			}
 		}
 		return result;
@@ -313,12 +313,13 @@ public static <T> Map<String, T> beansOfTypeIncludingAncestors(
 			if (hbf.getParentBeanFactory() instanceof ListableBeanFactory) {
 				Map<String, T> parentResult = beansOfTypeIncludingAncestors(
 						(ListableBeanFactory) hbf.getParentBeanFactory(), type, includeNonSingletons, allowEagerInit);
-				for (Map.Entry<String, T> entry : parentResult.entrySet()) {
+				
+				parentResult.entrySet().forEach(entry -> {
 					String beanName = entry.getKey();
 					if (!result.containsKey(beanName) && !hbf.containsLocalBean(beanName)) {
 						result.put(beanName, entry.getValue());
 					}
-				}
+				});
 			}
 		}
 		return result;

spring-beans/src/main/java/org/springframework/beans/factory/support/AbstractBeanFactory.java

@@ -1177,11 +1177,11 @@ protected void registerCustomEditors(PropertyEditorRegistry registry) {
 			}
 		}
 		if (!this.customEditors.isEmpty()) {
-			for (Map.Entry<Class<?>, Class<? extends PropertyEditor>> entry : this.customEditors.entrySet()) {
+			this.customEditors.entrySet().forEach(entry -> {
 				Class<?> requiredType = entry.getKey();
 				Class<? extends PropertyEditor> editorClass = entry.getValue();
 				registry.registerCustomEditor(requiredType, BeanUtils.instantiateClass(editorClass));
-			}
+			});
 		}
 	}
 

spring-core/src/main/java/org/springframework/util/ClassUtils.java

@@ -106,11 +106,11 @@ public abstract class ClassUtils {
 		primitiveWrapperTypeMap.put(Integer.class, int.class);
 		primitiveWrapperTypeMap.put(Long.class, long.class);
 		primitiveWrapperTypeMap.put(Short.class, short.class);
-
-		for (Map.Entry<Class<?>, Class<?>> entry : primitiveWrapperTypeMap.entrySet()) {
+		
+		primitiveWrapperTypeMap.entrySet().forEach(entry -> {
 			primitiveTypeToWrapperMap.put(entry.getValue(), entry.getKey());
 			registerCommonClasses(entry.getKey());
-		}
+		});
 
 		Set<Class<?>> primitiveTypes = new HashSet<>(32);
 		primitiveTypes.addAll(primitiveWrapperTypeMap.values());

spring-core/src/main/java/org/springframework/util/CollectionUtils.java

@@ -354,10 +354,10 @@ public static <K, V> MultiValueMap<K, V> toMultiValueMap(Map<K, List<V>> map) {
 	public static <K, V> MultiValueMap<K, V> unmodifiableMultiValueMap(MultiValueMap<? extends K, ? extends V> map) {
 		Assert.notNull(map, "'map' must not be null");
 		Map<K, List<V>> result = new LinkedHashMap<>(map.size());
-		for (Map.Entry<? extends K, ? extends List<? extends V>> entry : map.entrySet()) {
+		map.entrySet().forEach(entry -> {
 			List<? extends V> values = Collections.unmodifiableList(entry.getValue());
 			result.put(entry.getKey(), (List<V>) values);
-		}
+		});
 		Map<K, List<V>> unmodifiableMap = Collections.unmodifiableMap(result);
 		return toMultiValueMap(unmodifiableMap);
 	}
@@ -431,17 +431,13 @@ public void set(K key, V value) {
 
 		@Override
 		public void setAll(Map<K, V> values) {
-			for (Entry<K, V> entry : values.entrySet()) {
-				set(entry.getKey(), entry.getValue());
-			}
+			values.entrySet().forEach(entry -> set(entry.getKey(), entry.getValue()));
 		}
 
 		@Override
 		public Map<K, V> toSingleValueMap() {
 			LinkedHashMap<K, V> singleValueMap = new LinkedHashMap<>(this.map.size());
-			for (Entry<K, List<V>> entry : map.entrySet()) {
-				singleValueMap.put(entry.getKey(), entry.getValue().get(0));
-			}
+			map.entrySet().forEach(entry -> singleValueMap.put(entry.getKey(), entry.getValue().get(0)));
 			return singleValueMap;
 		}
 

spring-core/src/main/java/org/springframework/util/LinkedCaseInsensitiveMap.java

@@ -169,9 +169,8 @@ public void putAll(Map<? extends String, ? extends V> map) {
 		if (map.isEmpty()) {
 			return;
 		}
-		for (Map.Entry<? extends String, ? extends V> entry : map.entrySet()) {
-			put(entry.getKey(), entry.getValue());
-		}
+		map.entrySet().forEach(entry -> put(entry.getKey(), entry.getValue()));
+		
 	}
 
 	@Override

spring-core/src/main/java/org/springframework/util/LinkedMultiValueMap.java

@@ -100,17 +100,15 @@ public void set(K key, V value) {
 
 	@Override
 	public void setAll(Map<K, V> values) {
-		for (Entry<K, V> entry : values.entrySet()) {
-			set(entry.getKey(), entry.getValue());
-		}
+		values.entrySet().forEach(entry -> set(entry.getKey(), entry.getValue()));	
 	}
 
 	@Override
 	public Map<K, V> toSingleValueMap() {
 		LinkedHashMap<K, V> singleValueMap = new LinkedHashMap<>(this.targetMap.size());
-		for (Entry<K, List<V>> entry : this.targetMap.entrySet()) {
-			singleValueMap.put(entry.getKey(), entry.getValue().get(0));
-		}
+		this.targetMap.entrySet().forEach(entry ->
+			singleValueMap.put(entry.getKey(), entry.getValue().get(0)));
+		
 		return singleValueMap;
 	}
 
@@ -186,9 +184,9 @@ public Set<Entry<K, List<V>>> entrySet() {
 	 */
 	public LinkedMultiValueMap<K, V> deepCopy() {
 		LinkedMultiValueMap<K, V> copy = new LinkedMultiValueMap<>(this.targetMap.size());
-		for (Map.Entry<K, List<V>> entry : this.targetMap.entrySet()) {
-			copy.put(entry.getKey(), new LinkedList<>(entry.getValue()));
-		}
+		this.targetMap.entrySet().forEach(entry -> 
+			copy.put(entry.getKey(), new LinkedList<>(entry.getValue())));
+		
 		return copy;
 	}
 

spring-webflux/src/main/java/org/springframework/web/reactive/result/method/AbstractHandlerMethodMapping.java

@@ -192,11 +192,12 @@ protected void detectHandlerMethods(final Object handler) {
 		if (logger.isDebugEnabled()) {
 			logger.debug(methods.size() + " request handler methods found on " + userType + ": " + methods);
 		}
-		for (Map.Entry<Method, T> entry : methods.entrySet()) {
+		
+		methods.entrySet().forEach(entry -> {
 			Method invocableMethod = AopUtils.selectInvocableMethod(entry.getKey(), userType);
 			T mapping = entry.getValue();
 			registerHandlerMethod(handler, invocableMethod, mapping);
-		}
+		});
 	}
 
 	/**

spring-webmvc/src/main/java/org/springframework/web/servlet/handler/SimpleUrlHandlerMapping.java

@@ -114,7 +114,7 @@ protected void registerHandlers(Map<String, Object> urlMap) throws BeansExceptio
 			logger.warn("Neither 'urlMap' nor 'mappings' set on SimpleUrlHandlerMapping");
 		}
 		else {
-			for (Map.Entry<String, Object> entry : urlMap.entrySet()) {
+			urlMap.entrySet().forEach(entry -> {
 				String url = entry.getKey();
 				Object handler = entry.getValue();
 				// Prepend with slash if not already present.
@@ -126,7 +126,7 @@ protected void registerHandlers(Map<String, Object> urlMap) throws BeansExceptio
 					handler = ((String) handler).trim();
 				}
 				registerHandler(url, handler);
-			}
+			});
 		}
 	}