diff options
author | Gary Gregory <garydgregory@gmail.com> | 2022-11-30 15:03:49 -0500 |
---|---|---|
committer | Gary Gregory <garydgregory@gmail.com> | 2022-11-30 15:03:49 -0500 |
commit | 74ce18e283b7fcc05192aa7dc07f4c65741ad7d7 (patch) | |
tree | 1434500c0e0a665d80b05e64d2ddec0280d22fa7 | |
parent | 770e72d2f78361b14f3fe27caea41e5977d3c638 (diff) | |
download | apache-commons-lang-74ce18e283b7fcc05192aa7dc07f4c65741ad7d7.tar.gz |
Lookup key in map only once
-rw-r--r-- | src/main/java/org/apache/commons/lang3/ClassUtils.java | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/src/main/java/org/apache/commons/lang3/ClassUtils.java b/src/main/java/org/apache/commons/lang3/ClassUtils.java index 8b15a3c37..1d2b9c461 100644 --- a/src/main/java/org/apache/commons/lang3/ClassUtils.java +++ b/src/main/java/org/apache/commons/lang3/ClassUtils.java @@ -528,13 +528,8 @@ public class ClassUtils { */ public static Class<?> getClass(final ClassLoader classLoader, final String className, final boolean initialize) throws ClassNotFoundException { try { - final Class<?> clazz; - if (namePrimitiveMap.containsKey(className)) { - clazz = namePrimitiveMap.get(className); - } else { - clazz = Class.forName(toCanonicalName(className), initialize, classLoader); - } - return clazz; + Class<?> clazz = namePrimitiveMap.get(className); + return clazz != null ? clazz : Class.forName(toCanonicalName(className), initialize, classLoader); } catch (final ClassNotFoundException ex) { // allow path separators (.) as inner class name separators final int lastDotIndex = className.lastIndexOf(PACKAGE_SEPARATOR_CHAR); |