Polishing.

Original pull request #3612

Author: arefbehboudi

spring-data-jpa/src/main/java/org/springframework/data/jpa/domain/JpaSort.java

@@ -309,7 +309,7 @@ public String toString() {
 				builder.append(attribute.getName()).append(".");
 			}
 
-			return builder.length() == 0 ? "" : builder.substring(0, builder.lastIndexOf("."));
+			return builder.isEmpty() ? "" : builder.substring(0, builder.lastIndexOf("."));
 		}
 	}
 

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/aot/JpaRuntimeHints.java

@@ -88,9 +88,8 @@ public void registerHints(RuntimeHints hints, @Nullable ClassLoader classLoader)
 
 		// streaming results requires reflective access to jakarta.persistence.Query#getResultAsStream
 		hints.reflection().registerType(jakarta.persistence.Query.class, MemberCategory.INTROSPECT_PUBLIC_METHODS);
-		hints.reflection().registerType(jakarta.persistence.Query.class, hint -> {
-			hint.withMethod("getResultStream", Collections.emptyList(), ExecutableMode.INVOKE);
-		});
+		hints.reflection().registerType(jakarta.persistence.Query.class, hint ->
+				hint.withMethod("getResultStream", Collections.emptyList(), ExecutableMode.INVOKE));
 
 		hints.reflection().registerType(NamedEntityGraph.class,
 				hint -> hint.onReachableType(EntityGraph.class).withMembers(MemberCategory.INVOKE_PUBLIC_METHODS));

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/Querydsl.java

@@ -77,15 +77,11 @@ public Querydsl(EntityManager em, PathBuilder<?> builder) {
 	 */
 	public <T> AbstractJPAQuery<T, JPAQuery<T>> createQuery() {
 
-		switch (provider) {
-			case ECLIPSELINK:
-				return new JPAQuery<>(em, EclipseLinkTemplates.DEFAULT);
-			case HIBERNATE:
-				return new JPAQuery<>(em, HQLTemplates.DEFAULT);
-			case GENERIC_JPA:
-			default:
-				return new JPAQuery<>(em);
-		}
+        return switch (provider) {
+            case ECLIPSELINK -> new JPAQuery<>(em, EclipseLinkTemplates.DEFAULT);
+            case HIBERNATE -> new JPAQuery<>(em, HQLTemplates.DEFAULT);
+            default -> new JPAQuery<>(em);
+        };
 	}
 
 	/**
@@ -202,18 +198,11 @@ private NullHandling toQueryDslNullHandling(org.springframework.data.domain.Sort
 
 		Assert.notNull(nullHandling, "NullHandling must not be null");
 
-		switch (nullHandling) {
-
-			case NULLS_FIRST:
-				return NullHandling.NullsFirst;
-
-			case NULLS_LAST:
-				return NullHandling.NullsLast;
-
-			case NATIVE:
-			default:
-				return NullHandling.Default;
-		}
+        return switch (nullHandling) {
+            case NULLS_FIRST -> NullHandling.NullsFirst;
+            case NULLS_LAST -> NullHandling.NullsLast;
+            default -> NullHandling.Default;
+        };
 	}
 
 	/**

spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java

@@ -98,6 +98,8 @@
 public class SimpleJpaRepository<T, ID> implements JpaRepositoryImplementation<T, ID> {
 
 	private static final String ID_MUST_NOT_BE_NULL = "The given id must not be null";
+	private static final String IDS_MUST_NOT_BE_NULL = "Ids must not be null";
+	private static final String ENTITIES_MUST_NOT_BE_NULL = "Entities must not be null";
 
 	private final JpaEntityInformation<T, ?> entityInformation;
 	private final EntityManager entityManager;
@@ -203,7 +205,7 @@ public void delete(T entity) {
 	@Transactional
 	public void deleteAllById(Iterable<? extends ID> ids) {
 
-		Assert.notNull(ids, "Ids must not be null");
+		Assert.notNull(ids, IDS_MUST_NOT_BE_NULL);
 
 		for (ID id : ids) {
 			deleteById(id);
@@ -214,7 +216,7 @@ public void deleteAllById(Iterable<? extends ID> ids) {
 	@Transactional
 	public void deleteAllByIdInBatch(Iterable<ID> ids) {
 
-		Assert.notNull(ids, "Ids must not be null");
+		Assert.notNull(ids, IDS_MUST_NOT_BE_NULL);
 
 		if (!ids.iterator().hasNext()) {
 			return;
@@ -255,7 +257,7 @@ public void deleteAllByIdInBatch(Iterable<ID> ids) {
 	@Transactional
 	public void deleteAll(Iterable<? extends T> entities) {
 
-		Assert.notNull(entities, "Entities must not be null");
+		Assert.notNull(entities, ENTITIES_MUST_NOT_BE_NULL);
 
 		for (T entity : entities) {
 			delete(entity);
@@ -266,7 +268,7 @@ public void deleteAll(Iterable<? extends T> entities) {
 	@Transactional
 	public void deleteAllInBatch(Iterable<T> entities) {
 
-		Assert.notNull(entities, "Entities must not be null");
+		Assert.notNull(entities, ENTITIES_MUST_NOT_BE_NULL);
 
 		if (!entities.iterator().hasNext()) {
 			return;
@@ -387,7 +389,7 @@ public List<T> findAll() {
 	@Override
 	public List<T> findAllById(Iterable<ID> ids) {
 
-		Assert.notNull(ids, "Ids must not be null");
+		Assert.notNull(ids, IDS_MUST_NOT_BE_NULL);
 
 		if (!ids.iterator().hasNext()) {
 			return Collections.emptyList();
@@ -638,7 +640,7 @@ public <S extends T> S saveAndFlush(S entity) {
 	@Transactional
 	public <S extends T> List<S> saveAll(Iterable<S> entities) {
 
-		Assert.notNull(entities, "Entities must not be null");
+		Assert.notNull(entities, ENTITIES_MUST_NOT_BE_NULL);
 
 		List<S> result = new ArrayList<>();