Polish

Author: philwebb

buildSrc/src/main/java/org/springframework/boot/build/architecture/ArchitectureRules.java

@@ -119,9 +119,9 @@ private static void allBeanPostProcessorBeanMethodsShouldBeStaticAndNotCausePrem
 			.stream()
 			.filter(notAnnotatedWithLazy)
 			.filter((parameter) -> notOfASafeType.test(parameter.getRawType()))
-			.forEach((parameter) -> events.add(SimpleConditionEvent.violated(parameter,
+			.forEach((parameter) -> addViolation(events, parameter,
 					parameter.getDescription() + " will cause eager initialization as it is "
-							+ notAnnotatedWithLazy.getDescription() + " and is " + notOfASafeType.getDescription())));
+							+ notAnnotatedWithLazy.getDescription() + " and is " + notOfASafeType.getDescription()));
 	}
 
 	private static ArchRule allBeanFactoryPostProcessorBeanMethodsShouldBeStaticAndHaveNoParameters() {
@@ -193,8 +193,8 @@ private static ArchCondition<? super JavaMethod> notSpecifyOnlyATypeThatIsTheSam
 			if (!properties.containsKey("type") && !properties.containsKey("name")) {
 				conditionalAnnotation.get("value").ifPresent((value) -> {
 					if (containsOnlySingleType((JavaType[]) value, item.getReturnType())) {
-						events.add(SimpleConditionEvent.violated(item, conditionalAnnotation.getDescription()
-								+ " should not specify only a value that is the same as the method's return type"));
+						addViolation(events, item, conditionalAnnotation.getDescription()
+								+ " should not specify only a value that is the same as the method's return type");
 					}
 				});
 			}
@@ -265,6 +265,10 @@ public void check(T item, ConditionEvents events) {
 		};
 	}
 
+	private static void addViolation(ConditionEvents events, Object correspondingObject, String message) {
+		events.add(SimpleConditionEvent.violated(correspondingObject, message));
+	}
+
 	private static String shouldUse(String string) {
 		return string + " should be used instead";
 	}