Skip to content

Commit eef7c29

Browse files
author
Vincent Potucek
committed
add ObjectUtils.firstNonNull/getIfNull/getIfEmpty
Signed-off-by: Vincent Potucek <vpotucek@me.com>
1 parent 4c2a4f8 commit eef7c29

File tree

2 files changed

+11
-11
lines changed

2 files changed

+11
-11
lines changed

spring-core/src/main/java/org/springframework/util/ObjectUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ public static boolean isEmpty(@Nullable Object @Nullable [] array) {
127127
* @return An Optional containing the first non-null object, or an empty Optional if all objects are null.
128128
*/
129129
@SafeVarargs
130-
public static <T> Optional<T> getNonNull(@Nullable final T... objects) {
130+
public static <T> Optional<T> firstNonNull(@Nullable final T... objects) {
131131
return Optional.ofNullable(objects)
132132
.map(arr -> Arrays.stream(arr).filter(Objects::nonNull).findFirst())
133133
.orElse(Optional.empty());

spring-core/src/test/java/org/springframework/util/ObjectUtilsTests.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@
5555
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
5656
import static org.springframework.util.ObjectUtils.getIfEmpty;
5757
import static org.springframework.util.ObjectUtils.getIfNull;
58-
import static org.springframework.util.ObjectUtils.getNonNull;
58+
import static org.springframework.util.ObjectUtils.firstNonNull;
5959
import static org.springframework.util.ObjectUtils.isEmpty;
6060

6161
/**
@@ -148,20 +148,20 @@ void getIfEmptyObject() {
148148
}
149149

150150
@Test
151-
void getNonNullObject() {
151+
void firstNonNullObject() {
152152
String value = UUID.randomUUID().toString();
153153
String backup = UUID.randomUUID().toString();
154154

155-
assertThat(getNonNull(backup, value, value)).contains(backup);
156-
assertThat(getNonNull(value, null, backup)).contains(value);
157-
assertThat(getNonNull(backup, value, null)).contains(backup);
158-
assertThat(getNonNull(value, backup)).contains(value);
159-
assertThat(getNonNull(null, null, value)).contains(value);
155+
assertThat(firstNonNull(backup, value, value)).contains(backup);
156+
assertThat(firstNonNull(value, null, backup)).contains(value);
157+
assertThat(firstNonNull(backup, value, null)).contains(backup);
158+
assertThat(firstNonNull(value, backup)).contains(value);
159+
assertThat(firstNonNull(null, null, value)).contains(value);
160160

161161
String _null = null;
162-
assertThat(getNonNull(null, null, _null)).isEmpty();
163-
assertThat(getNonNull(null, "null", backup)).contains("null");
164-
assertThat(getNonNull(null, null)).isEmpty();
162+
assertThat(firstNonNull(null, null, _null)).isEmpty();
163+
assertThat(firstNonNull(null, "null", backup)).contains("null");
164+
assertThat(firstNonNull(null, null)).isEmpty();
165165
}
166166

167167
@Test

0 commit comments

Comments
 (0)