パラメータ取得処理をラムダではなく個々の型で管理するように変更

Author: k163377

src/main/kotlin/com/mapk/kmapper/ParameterForMap.kt

@@ -15,33 +15,33 @@ internal class ParameterForMap<T : Any> private constructor(val param: KParamete
     // リストの長さが小さいと期待されるためこの形で実装しているが、理想的にはmap的なものが使いたい
     private val converters: Set<Pair<KClass<*>, KFunction<T>>> = clazz.getConverters()
 
-    private val convertCache: ConcurrentMap<KClass<*>, (Any) -> Any?> = ConcurrentHashMap()
+    private val convertCache: ConcurrentMap<KClass<*>, ParameterProcessor> = ConcurrentHashMap()
 
     fun <U : Any> mapObject(value: U): Any? {
         val valueClazz: KClass<*> = value::class
 
         // 取得方法のキャッシュが有ればそれを用いる
-        convertCache[valueClazz]?.let { return it(value) }
+        convertCache[valueClazz]?.let { return it.process(value) }
 
         // パラメータに対してvalueが代入可能（同じもしくは親クラス）であればそのまま用いる
         if (clazz.isSuperclassOf(valueClazz)) {
-            convertCache.putIfAbsent(valueClazz) { it }
+            convertCache.putIfAbsent(valueClazz, ParameterProcessor.Plain)
             return value
         }
 
         val converter: KFunction<*>? = converters.getConverter(valueClazz)
 
-        val lambda: (Any) -> Any? = when {
+        val processor: ParameterProcessor = when {
             // converterに一致する組み合わせが有れば設定されていればそれを使う
-            converter != null -> { { converter.call(it) } }
+            converter != null -> ParameterProcessor.UseConverter(converter)
             // 要求された値がenumかつ元が文字列ならenum mapperでマップ
-            javaClazz.isEnum && value is String -> { { EnumMapper.getEnum(javaClazz, it as String) } }
+            javaClazz.isEnum && value is String -> ParameterProcessor.ToEnum(javaClazz)
             // 要求されているパラメータがStringならtoStringする
-            clazz == String::class -> { { it.toString() } }
+            clazz == String::class -> ParameterProcessor.ToString
             else -> throw IllegalArgumentException("Can not convert $valueClazz to $clazz")
         }
-        convertCache.putIfAbsent(valueClazz, lambda)
-        return lambda(value)
+        convertCache.putIfAbsent(valueClazz, processor)
+        return processor.process(value)
     }
 
     companion object {
@@ -50,3 +50,23 @@ internal class ParameterForMap<T : Any> private constructor(val param: KParamete
         }
     }
 }
+
+private sealed class ParameterProcessor {
+    abstract fun process(value: Any): Any?
+
+    object Plain : ParameterProcessor() {
+        override fun process(value: Any): Any? = value
+    }
+
+    class UseConverter(private val converter: KFunction<*>) : ParameterProcessor() {
+        override fun process(value: Any): Any? = converter.call(value)
+    }
+
+    class ToEnum(private val javaClazz: Class<*>) : ParameterProcessor() {
+        override fun process(value: Any): Any? = EnumMapper.getEnum(javaClazz, value as String)
+    }
+
+    object ToString : ParameterProcessor() {
+        override fun process(value: Any): Any? = value.toString()
+    }
+}