diff options
author | Gary Gregory <garydgregory@gmail.com> | 2021-02-22 11:12:20 -0500 |
---|---|---|
committer | Gary Gregory <garydgregory@gmail.com> | 2021-02-22 11:12:20 -0500 |
commit | 91928eed34c1284f0d3c0db0635c39c0cb114047 (patch) | |
tree | cf04295c725f04d95415755407d207ce9988751f /src/main/java/org/apache/commons/lang3/RandomStringUtils.java | |
parent | 7e5be1cf84b9a1da900a4115b451aa3722929b57 (diff) | |
download | apache-commons-lang-91928eed34c1284f0d3c0db0635c39c0cb114047.tar.gz |
Combine nested 'if' statement in 'else' block to 'else if'.
Diffstat (limited to 'src/main/java/org/apache/commons/lang3/RandomStringUtils.java')
-rw-r--r-- | src/main/java/org/apache/commons/lang3/RandomStringUtils.java | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/src/main/java/org/apache/commons/lang3/RandomStringUtils.java b/src/main/java/org/apache/commons/lang3/RandomStringUtils.java index f089394b1..23d216115 100644 --- a/src/main/java/org/apache/commons/lang3/RandomStringUtils.java +++ b/src/main/java/org/apache/commons/lang3/RandomStringUtils.java @@ -372,18 +372,14 @@ public class RandomStringUtils { if (start == 0 && end == 0) { if (chars != null) { end = chars.length; + } else if (!letters && !numbers) { + end = Character.MAX_CODE_POINT; } else { - if (!letters && !numbers) { - end = Character.MAX_CODE_POINT; - } else { - end = 'z' + 1; - start = ' '; - } - } - } else { - if (end <= start) { - throw new IllegalArgumentException("Parameter end (" + end + ") must be greater than start (" + start + ")"); + end = 'z' + 1; + start = ' '; } + } else if (end <= start) { + throw new IllegalArgumentException("Parameter end (" + end + ") must be greater than start (" + start + ")"); } final int zero_digit_ascii = 48; |