polish

Author: ngocnhan-tran1996

src/main/java/org/springframework/data/aot/DefaultAotContext.java

@@ -32,14 +32,15 @@
  * Default {@link AotContext} implementation.
  *
  * @author Mark Paluch
+ * @author Ngoc Nhan
  * @since 3.0
  */
 class DefaultAotContext implements AotContext {
 
 	private final ConfigurableListableBeanFactory factory;
 
 	public DefaultAotContext(BeanFactory beanFactory) {
-		factory = beanFactory instanceof ConfigurableListableBeanFactory ? (ConfigurableListableBeanFactory) beanFactory
+		factory = beanFactory instanceof ConfigurableListableBeanFactory configurableListableBeanFactory ? configurableListableBeanFactory
 				: new DefaultListableBeanFactory(beanFactory);
 	}
 

src/main/java/org/springframework/data/convert/DefaultConverterBuilder.java

@@ -39,6 +39,7 @@
  * @author Oliver Gierke
  * @author Mark Paluch
  * @author Johannes Englmeier
+ * @author Ngoc Nhan
  * @since 2.0
  * @see ConverterBuilder#writing(Class, Class, Function)
  * @see ConverterBuilder#reading(Class, Class, Function)
@@ -93,12 +94,12 @@ private ConvertiblePair invertedPair() {
 
 	DefaultConverterBuilder<S, T> withWriting(Optional<Function<? super S, ? extends T>> writing) {
 		return this.writing == writing ? this
-				: new DefaultConverterBuilder<S, T>(this.convertiblePair, writing, this.reading);
+				: new DefaultConverterBuilder<>(this.convertiblePair, writing, this.reading);
 	}
 
 	DefaultConverterBuilder<S, T> withReading(Optional<Function<? super T, ? extends S>> reading) {
 		return this.reading == reading ? this
-				: new DefaultConverterBuilder<S, T>(this.convertiblePair, this.writing, reading);
+				: new DefaultConverterBuilder<>(this.convertiblePair, this.writing, reading);
 	}
 
 	private static class ConfigurableGenericConverter<S, T> implements GenericConverter {

src/main/java/org/springframework/data/convert/DefaultTypeMapper.java

@@ -38,6 +38,7 @@
  * @author Thomas Darimont
  * @author Christoph Strobl
  * @author Mark Paluch
+ * @author Ngoc Nhan
  */
 public class DefaultTypeMapper<S> implements TypeMapper<S>, BeanClassLoaderAware {
 
@@ -199,8 +200,8 @@ public void writeType(TypeInformation<?> info, S sink) {
 	@Override
 	public void setBeanClassLoader(ClassLoader classLoader) {
 		for (TypeInformationMapper mapper : mappers) {
-			if (mapper instanceof BeanClassLoaderAware) {
-				((BeanClassLoaderAware) mapper).setBeanClassLoader(classLoader);
+			if (mapper instanceof BeanClassLoaderAware beanClassLoaderAware) {
+				beanClassLoaderAware.setBeanClassLoader(classLoader);
 			}
 		}
 	}

src/main/java/org/springframework/data/domain/SliceImpl.java

@@ -25,6 +25,7 @@
  *
  * @author Oliver Gierke
  * @author Keegan Witt
+ * @author Ngoc Nhan
  * @since 1.8
  */
 public class SliceImpl<T> extends Chunk<T> {
@@ -74,7 +75,7 @@ public String toString() {
 		String contentType = "UNKNOWN";
 		List<T> content = getContent();
 
-		if (content.size() > 0) {
+		if (!content.isEmpty()) {
 			contentType = content.get(0).getClass().getName();
 		}
 

src/main/java/org/springframework/data/domain/Sort.java

@@ -44,6 +44,7 @@
  * @author Mark Paluch
  * @author Johannes Englmeier
  * @author Jan Kurella
+ * @author Ngoc Nhan
  */
 public class Sort implements Streamable<org.springframework.data.domain.Sort.Order>, Serializable {
 
@@ -204,7 +205,7 @@ public Sort and(Sort sort) {
 
 		Assert.notNull(sort, "Sort must not be null");
 
-		List<Order> these = new ArrayList<Order>(this.toList());
+		List<Order> these = new ArrayList<>(this.toList());
 
 		for (Order order : sort) {
 			these.add(order);

src/main/java/org/springframework/data/history/AnnotationRevisionMetadata.java

@@ -37,6 +37,7 @@
  * @author Jens Schauder
  * @author Mark Paluch
  * @author Johannes Englmeier
+ * @author Ngoc Nhan
  */
 public class AnnotationRevisionMetadata<N extends Number & Comparable<N>> implements RevisionMetadata<N> {
 
@@ -120,20 +121,20 @@ private static <T> Lazy<Optional<T>> detectAnnotation(Object entity, Class<? ext
 
 	private static Instant convertToInstant(Object timestamp) {
 
-		if (timestamp instanceof Instant) {
-			return (Instant) timestamp;
+		if (timestamp instanceof Instant instant) {
+			return instant;
 		}
 
-		if (timestamp instanceof LocalDateTime) {
-			return ((LocalDateTime) timestamp).atZone(ZoneOffset.systemDefault()).toInstant();
+		if (timestamp instanceof LocalDateTime localDateTime) {
+			return localDateTime.atZone(ZoneOffset.systemDefault()).toInstant();
 		}
 
-		if (timestamp instanceof Long) {
-			return Instant.ofEpochMilli((Long) timestamp);
+		if (timestamp instanceof Long milli) {
+			return Instant.ofEpochMilli(milli);
 		}
 
-		if (timestamp instanceof Date) {
-			return ((Date) timestamp).toInstant();
+		if (timestamp instanceof Date date) {
+			return date.toInstant();
 		}
 
 		throw new IllegalArgumentException(String.format("Can't convert %s to Instant", timestamp));

src/main/java/org/springframework/data/mapping/AccessOptions.java

@@ -32,6 +32,7 @@
  * propagation settings for how to handle intermediate collection and map values when setting values.
  *
  * @author Oliver Drotbohm
+ * @author Ngoc Nhan
  * @since 2.3
  */
 public class AccessOptions {
@@ -186,7 +187,7 @@ public <T> GetOptions registerHandler(PersistentProperty<?> property, Class<T> t
 			Assert.isTrue(type.isAssignableFrom(property.getType()), () -> String
 					.format("Cannot register a property handler for %s on a property of type %s", type, property.getType()));
 
-			Function<Object, T> caster = (Function<Object, T>) it -> type.cast(it);
+			Function<Object, T> caster = type::cast;
 
 			return registerHandler(property, caster.andThen(handler));
 		}

src/main/java/org/springframework/data/mapping/PropertyPath.java

@@ -41,6 +41,7 @@
  * @author Mark Paluch
  * @author Mariusz Mączkowski
  * @author Johannes Englmeier
+ * @author Ngoc Nhan
  */
 public class PropertyPath implements Streamable<PropertyPath> {
 
@@ -377,7 +378,7 @@ public static PropertyPath from(String source, TypeInformation<?> type) {
 			Iterator<String> parts = iteratorSource.iterator();
 
 			PropertyPath result = null;
-			Stack<PropertyPath> current = new Stack<PropertyPath>();
+			Stack<PropertyPath> current = new Stack<>();
 
 			while (parts.hasNext()) {
 				if (result == null) {

src/main/java/org/springframework/data/mapping/context/DefaultPersistentPropertyPath.java

@@ -33,6 +33,7 @@
  *
  * @author Oliver Gierke
  * @author Christoph Strobl
+ * @author Ngoc Nhan
  */
 class DefaultPersistentPropertyPath<P extends PersistentProperty<P>> implements PersistentPropertyPath<P> {
 
@@ -135,15 +136,15 @@ public String toPath(String delimiter, Converter<? super P, String> converter) {
 	@Override
 	public P getLeafProperty() {
 
-		Assert.state(properties.size() > 0, "Empty PersistentPropertyPath should not exist");
+		Assert.state(!properties.isEmpty(), "Empty PersistentPropertyPath should not exist");
 
 		return properties.get(properties.size() - 1);
 	}
 
 	@Override
 	public P getBaseProperty() {
 
-		Assert.state(properties.size() > 0, "Empty PersistentPropertyPath should not exist");
+		Assert.state(!properties.isEmpty(), "Empty PersistentPropertyPath should not exist");
 
 		return properties.get(0);
 	}
@@ -208,9 +209,8 @@ public Iterator<P> iterator() {
 	 */
 	public boolean containsPropertyOfType(@Nullable TypeInformation<?> type) {
 
-		return type == null //
-				? false //
-				: properties.stream() //
+		return type != null //
+				&& properties.stream() //
 						.anyMatch(property -> type.equals(property.getTypeInformation().getActualType()));
 	}
 

src/main/java/org/springframework/data/mapping/model/PersistentEntityIsNewStrategy.java

@@ -29,6 +29,7 @@
  * @author Oliver Gierke
  * @author Mark Paluch
  * @author Johannes Englmeier
+ * @author Ngoc Nhan
  * @soundtrack Scary Pockets - Crash Into Me (Dave Matthews Band Cover feat. Julia Nunes) -
  *             https://www.youtube.com/watch?v=syGlBNVGEqU
  */
@@ -100,8 +101,8 @@ public boolean isNew(Object entity) {
 			return false;
 		}
 
-		if (value instanceof Number) {
-			return ((Number) value).longValue() == 0;
+		if (value instanceof Number number) {
+			return number.longValue() == 0;
 		}
 
 		throw new IllegalArgumentException(