Replace cascading if-else with a map lookup

Remove unused
org.apache.commons.beanutils2.MethodUtils.getPrimitiveWrapper(Class) in
favor of org.apache.commons.lang3.ClassUtils.primitiveToWrapper(Class)

Author: garydgregory

src/changes/changes.xml

@@ -49,6 +49,7 @@
       <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove unused org.apache.commons.beanutils2.MethodUtils.invokeStaticMethod(Class, String, Object[], Class[]) in favor of Apache Commons Lang's org.apache.commons.lang3.reflect.MethodUtils.</action>
       <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove unused org.apache.commons.beanutils2.MethodUtils.toNonPrimitiveClass(Class) in favor of org.apache.commons.lang3.ClassUtils.primitiveToWrapper(Class).</action>
       <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove unused org.apache.commons.beanutils2.MethodUtils.getPrimitiveType(Class) in favor of org.apache.commons.lang3.ClassUtils.wrapperToPrimitive(Class).</action>
+      <action dev="ggregory" type="remove" due-to="Gary Gregory">Remove unused org.apache.commons.beanutils2.MethodUtils.getPrimitiveWrapper(Class) in favor of org.apache.commons.lang3.ClassUtils.primitiveToWrapper(Class).</action>
     </release>
     <release version="2.0.0-M2" date="2025-05-25" description="This is a major release and requires Java 8.">
       <!-- FIX -->

src/main/java/org/apache/commons/beanutils2/MethodUtils.java

@@ -28,6 +28,7 @@
 import java.util.Objects;
 import java.util.WeakHashMap;
 
+import org.apache.commons.lang3.ClassUtils;
 import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
@@ -493,7 +494,7 @@ private static float getObjectTransformationCost(Class<?> srcClass, final Class<
         float cost = 0.0f;
         while (srcClass != null && !destClass.equals(srcClass)) {
             if (destClass.isPrimitive()) {
-                final Class<?> destClassWrapperClazz = getPrimitiveWrapper(destClass);
+                final Class<?> destClassWrapperClazz = ClassUtils.wrapperToPrimitive(destClass);
                 if (destClassWrapperClazz != null && destClassWrapperClazz.equals(srcClass)) {
                     cost += 0.25f;
                     break;
@@ -521,41 +522,6 @@ private static float getObjectTransformationCost(Class<?> srcClass, final Class<
         return cost;
     }
 
-    /**
-     * Gets the wrapper object class for the given primitive type class. For example, passing {@code boolean.class</code> returns <code>Boolean.class}
-     *
-     * @param primitiveType the primitive type class for which a match is to be found
-     * @return the wrapper type associated with the given primitive or null if no match is found
-     */
-    public static Class<?> getPrimitiveWrapper(final Class<?> primitiveType) {
-        // does anyone know a better strategy than comparing names?
-        if (boolean.class.equals(primitiveType)) {
-            return Boolean.class;
-        }
-        if (float.class.equals(primitiveType)) {
-            return Float.class;
-        }
-        if (long.class.equals(primitiveType)) {
-            return Long.class;
-        }
-        if (int.class.equals(primitiveType)) {
-            return Integer.class;
-        }
-        if (short.class.equals(primitiveType)) {
-            return Short.class;
-        }
-        if (byte.class.equals(primitiveType)) {
-            return Byte.class;
-        }
-        if (double.class.equals(primitiveType)) {
-            return Double.class;
-        }
-        if (char.class.equals(primitiveType)) {
-            return Character.class;
-        }
-        return null;
-    }
-
     /**
      * Returns the sum of the object transformation cost for each class in the source argument list.
      *
@@ -749,7 +715,7 @@ public static boolean isAssignmentCompatible(final Class<?> parameterType, final
         if (parameterType.isPrimitive()) {
             // this method does *not* do widening - you must specify exactly
             // is this the right behavior?
-            final Class<?> parameterWrapperClazz = getPrimitiveWrapper(parameterType);
+            final Class<?> parameterWrapperClazz = ClassUtils.wrapperToPrimitive(parameterType);
             if (parameterWrapperClazz != null) {
                 return parameterWrapperClazz.equals(parameterization);
             }