diff options
author | Gary Gregory <garydgregory@gmail.com> | 2020-12-22 16:06:55 -0500 |
---|---|---|
committer | Gary Gregory <garydgregory@gmail.com> | 2020-12-22 16:06:55 -0500 |
commit | 44b0bb5273db99db4a191b8b24f5676bd7a9c523 (patch) | |
tree | 794cb47446514b6e01762f0296060932f22a61f6 /src/main/java/org/apache/commons/lang3/builder | |
parent | 5aab41b3650db0fafeaf900a5b8c20342ea6726e (diff) | |
download | apache-commons-lang-44b0bb5273db99db4a191b8b24f5676bd7a9c523.tar.gz |
Use ArraySorter.
Diffstat (limited to 'src/main/java/org/apache/commons/lang3/builder')
-rw-r--r-- | src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java | 5 | ||||
-rw-r--r-- | src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java | 7 |
2 files changed, 5 insertions, 7 deletions
diff --git a/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java b/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java index 0a95a74fb..75eb3e38b 100644 --- a/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java +++ b/src/main/java/org/apache/commons/lang3/builder/HashCodeBuilder.java @@ -20,12 +20,12 @@ package org.apache.commons.lang3.builder; import java.lang.reflect.AccessibleObject; import java.lang.reflect.Field; import java.lang.reflect.Modifier; -import java.util.Arrays; import java.util.Collection; import java.util.Comparator; import java.util.HashSet; import java.util.Set; +import org.apache.commons.lang3.ArraySorter; import org.apache.commons.lang3.ArrayUtils; import org.apache.commons.lang3.Validate; @@ -191,8 +191,7 @@ public class HashCodeBuilder implements Builder<Integer> { try { register(object); // The elements in the returned array are not sorted and are not in any particular order. - final Field[] fields = clazz.getDeclaredFields(); - Arrays.sort(fields, Comparator.comparing(Field::getName)); + final Field[] fields = ArraySorter.sort(clazz.getDeclaredFields(), Comparator.comparing(Field::getName)); AccessibleObject.setAccessible(fields, true); for (final Field field : fields) { if (!ArrayUtils.contains(excludeFields, field.getName()) diff --git a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java index 66eb15cc2..63ad959d1 100644 --- a/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java +++ b/src/main/java/org/apache/commons/lang3/builder/ReflectionToStringBuilder.java @@ -26,6 +26,7 @@ import java.util.Collection; import java.util.Comparator; import java.util.List; +import org.apache.commons.lang3.ArraySorter; import org.apache.commons.lang3.ArrayUtils; import org.apache.commons.lang3.ClassUtils; import org.apache.commons.lang3.Validate; @@ -640,8 +641,7 @@ public class ReflectionToStringBuilder extends ToStringBuilder { return; } // The elements in the returned array are not sorted and are not in any particular order. - final Field[] fields = clazz.getDeclaredFields(); - Arrays.sort(fields, Comparator.comparing(Field::getName)); + final Field[] fields = ArraySorter.sort(clazz.getDeclaredFields(), Comparator.comparing(Field::getName)); AccessibleObject.setAccessible(fields, true); for (final Field field : fields) { final String fieldName = field.getName(); @@ -801,8 +801,7 @@ public class ReflectionToStringBuilder extends ToStringBuilder { this.excludeFieldNames = null; } else { //clone and remove nulls - this.excludeFieldNames = toNoNullStringArray(excludeFieldNamesParam); - Arrays.sort(this.excludeFieldNames); + this.excludeFieldNames = ArraySorter.sort(toNoNullStringArray(excludeFieldNamesParam)); } return this; } |