From 44d4acc6218bf06843833eaf275ccf5f2d526999 Mon Sep 17 00:00:00 2001 From: Isira Seneviratne <31027858+Isira-Seneviratne@users.noreply.github.com> Date: Mon, 20 Apr 2020 20:57:41 +0000 Subject: [PATCH] Simplify some usages of the ternary operator with calls to Math.max() and Math.min(). (#512) --- src/main/java/org/apache/commons/lang3/StringUtils.java | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/main/java/org/apache/commons/lang3/StringUtils.java b/src/main/java/org/apache/commons/lang3/StringUtils.java index 3caf9e168..4f20fd82c 100644 --- a/src/main/java/org/apache/commons/lang3/StringUtils.java +++ b/src/main/java/org/apache/commons/lang3/StringUtils.java @@ -6421,9 +6421,8 @@ private static String replace(final String text, String searchString, final Stri return text; } final int replLength = searchString.length(); - int increase = replacement.length() - replLength; - increase = increase < 0 ? 0 : increase; - increase *= max < 0 ? 16 : max > 64 ? 64 : max; + int increase = Math.max(replacement.length() - replLength, 0); + increase *= max < 0 ? 16 : Math.min(max, 64); final StringBuilder buf = new StringBuilder(text.length() + increase); while (end != INDEX_NOT_FOUND) { buf.append(text, start, end).append(replacement); @@ -9168,7 +9167,7 @@ public static String truncate(final String str, final int offset, final int maxW return EMPTY; } if (str.length() > maxWidth) { - final int ix = offset + maxWidth > str.length() ? str.length() : offset + maxWidth; + final int ix = Math.min(offset + maxWidth, str.length()); return str.substring(offset, ix); } return str.substring(offset);