minor changes (extract a method)

Author: gavinking

hibernate-core/src/main/java/org/hibernate/boot/internal/SessionFactoryOptionsBuilder.java

@@ -601,6 +601,12 @@ private static boolean disallowBatchUpdates(Dialect dialect, ExtractedDatabaseMe
 		return dialectAnswer != null ? !dialectAnswer : !meta.supportsBatchUpdates();
 	}
 
+	private static boolean hasStrategyConstructorSignature(Class<?>[] parameterTypes) {
+		return parameterTypes.length == 2
+			&& parameterTypes[0] == EntityMappingType.class
+			&& parameterTypes[1] == RuntimeModelCreationContext.class;
+	}
+
 	@SuppressWarnings("unchecked")
 	private SqmMultiTableMutationStrategy resolveSqmMutationStrategy(
 			String strategyName,
@@ -631,9 +637,7 @@ private SqmMultiTableMutationStrategy resolveSqmMutationStrategy(
 						else if ( parameterTypes.length == 0 ) {
 							emptyConstructor = constructor;
 						}
-						else if ( parameterTypes.length == 2
-									&& parameterTypes[0] == EntityMappingType.class
-									&& parameterTypes[1] == RuntimeModelCreationContext.class ) {
+						else if ( hasStrategyConstructorSignature( parameterTypes ) ) {
 							entityBasedConstructor = (Constructor<SqmMultiTableMutationStrategy>) declaredConstructor;
 						}
 					}
@@ -651,13 +655,12 @@ else if ( emptyConstructor != null ) {
 						}
 						catch (Exception e) {
 							throw new StrategySelectionException(
-									"Could not instantiate named strategy class [" +
-											strategyClass.getName() + "]",
+									"Could not instantiate named strategy class [" + strategyClass.getName() + "]",
 									e
 							);
 						}
-						throw new IllegalArgumentException(
-								"Cannot instantiate the class [" + strategyClass.getName() + "] because it does not have a constructor that accepts a dialect or an empty constructor" );
+						throw new IllegalArgumentException( "Cannot instantiate the class [" + strategyClass.getName()
+								+ "] because it does not have a constructor that accepts a dialect or an empty constructor" );
 					}
 					else {
 						return null;
@@ -675,9 +678,7 @@ private Constructor<SqmMultiTableMutationStrategy> resolveSqmMutationStrategyCon
 					strategySelector.selectStrategyImplementor( SqmMultiTableMutationStrategy.class, strategyName );
 			for ( var declaredConstructor : strategyClass.getDeclaredConstructors() ) {
 				final var parameterTypes = declaredConstructor.getParameterTypes();
-				if ( parameterTypes.length == 2
-						&& parameterTypes[0] == EntityMappingType.class
-						&& parameterTypes[1] == RuntimeModelCreationContext.class ) {
+				if ( hasStrategyConstructorSignature( parameterTypes ) ) {
 					return (Constructor<SqmMultiTableMutationStrategy>) declaredConstructor;
 				}
 			}
@@ -715,9 +716,7 @@ private SqmMultiTableInsertStrategy resolveSqmInsertStrategy(
 						else if ( parameterTypes.length == 0 ) {
 							emptyConstructor = constructor;
 						}
-						else if ( parameterTypes.length == 2
-									&& parameterTypes[0] == EntityMappingType.class
-									&& parameterTypes[1] == RuntimeModelCreationContext.class ) {
+						else if ( hasStrategyConstructorSignature( parameterTypes ) ) {
 							entityBasedConstructor = (Constructor<SqmMultiTableInsertStrategy>) declaredConstructor;
 						}
 					}
@@ -759,9 +758,7 @@ private Constructor<SqmMultiTableInsertStrategy> resolveSqmInsertStrategyConstru
 					strategySelector.selectStrategyImplementor( SqmMultiTableInsertStrategy.class, strategyName );
 			for ( var declaredConstructor : strategyClass.getDeclaredConstructors() ) {
 				final var parameterTypes = declaredConstructor.getParameterTypes();
-				if ( parameterTypes.length == 2
-						&& parameterTypes[0] == EntityMappingType.class
-						&& parameterTypes[1] == RuntimeModelCreationContext.class ) {
+				if ( hasStrategyConstructorSignature( parameterTypes ) ) {
 					return (Constructor<SqmMultiTableInsertStrategy>) declaredConstructor;
 				}
 			}
@@ -997,7 +994,8 @@ public SqmMultiTableMutationStrategy resolveCustomSqmMultiTableMutationStrategy(
 			}
 			catch (Exception e) {
 				throw new StrategySelectionException(
-						String.format( "Could not instantiate named strategy class [%s]", sqmMultiTableMutationStrategyConstructor.getDeclaringClass().getName() ),
+						String.format( "Could not instantiate named strategy class [%s]",
+								sqmMultiTableMutationStrategyConstructor.getDeclaringClass().getName() ),
 						e
 				);
 			}
@@ -1013,7 +1011,8 @@ public SqmMultiTableInsertStrategy resolveCustomSqmMultiTableInsertStrategy(Enti
 			}
 			catch (Exception e) {
 				throw new StrategySelectionException(
-						String.format( "Could not instantiate named strategy class [%s]", sqmMultiTableInsertStrategyConstructor.getDeclaringClass().getName() ),
+						String.format( "Could not instantiate named strategy class [%s]",
+								sqmMultiTableInsertStrategyConstructor.getDeclaringClass().getName() ),
 						e
 				);
 			}