Merge pull request #6 from k163377/improve_performance

Improve performance.

Author: k163377

.idea/codeStyles/Project.xml

@@ -22,7 +22,7 @@ data class Sample(
 
 val function: KFunction<Sample> = ::Sample
 
-val fastKFunction: FastKFunction<Sample> = FastKFunction(function)
+val fastKFunction: FastKFunction<Sample> = FastKFunction.of(function)
 
 // call by vararg
 val result: Sample = fastKFunction.call(1, 2, 3, 4, 5)
@@ -58,7 +58,7 @@ data class Sample(
 
 val sample = Sample(1, 2)
 
-val fastKFunction = FastKFunction(sample::instanceFun, sample)
+val fastKFunction = FastKFunction.of(sample::instanceFun, sample)
 ```
 
 Depending on how you get the `KFunction`, the `instance parameter` may be required.
@@ -72,7 +72,7 @@ Calling with `vararg` or `Collection` is faster if you don't need to use the def
  can get them in the order in which they are defined.
 
 ```kotlin
-val fastKFunction: FastKFunction<Sample> = FastKFunction(function)
+val fastKFunction: FastKFunction<Sample> = FastKFunction.of(function)
 
 // call by vararg
 val result: Sample = fastKFunction.call(1, 2, 3, 4, 5)
@@ -93,7 +93,7 @@ data class Sample(
     val arg3: String? = null
 )
 
-private val fastKFunction: FastKFunction<Sample> = FastKFunction(::Sample)
+private val fastKFunction: FastKFunction<Sample> = FastKFunction.of(::Sample)
 
 fun map(src: Map<String, Any?>): Sample {
     return fastKFunction.generateBucket()

README.md

@@ -16,7 +16,7 @@ open class CallConstructorBenchmark {
     private val argumentMap: Map<KParameter, Any?> = function.parameters.associateWith { it.index + 1 }
 
     private val javaConstructor: Constructor<Constructor5> = function.javaConstructor!!
-    private val fastKFunction: FastKFunction<Constructor5> = FastKFunction(function, null)
+    private val fastKFunction: FastKFunction<Constructor5> = FastKFunction.of(function, null)
     private val argumentBucket: ArgumentBucket = fastKFunction.generateBucket()
         .apply { (0 until 5).forEach { this[it] = it + 1 } }
 

src/jmh/kotlin/com/mapk/fastkfunction/CallConstructorBenchmark.kt

@@ -17,8 +17,8 @@ open class CallInstanceMethodBenchmark {
     private val argumentMap: Map<KParameter, Any?> = function.parameters.associateWith { it.index + 1 }
 
     private val javaMethod: Method = function.javaMethod!!
-    private val fastKFunctionWithoutInstance: FastKFunction<Constructor5> = FastKFunction(function, null)
-    private val fastKFunctionWithInstance: FastKFunction<Constructor5> = FastKFunction(function, instance)
+    private val fastKFunctionWithoutInstance: FastKFunction<Constructor5> = FastKFunction.of(function, null)
+    private val fastKFunctionWithInstance: FastKFunction<Constructor5> = FastKFunction.of(function, instance)
     private val argumentBucket: ArgumentBucket = fastKFunctionWithoutInstance.generateBucket()
         .apply { (0 until 5).forEach { this[it] = it + 1 } }
 

src/jmh/kotlin/com/mapk/fastkfunction/CallInstanceMethodBenchmark.kt

@@ -27,14 +27,14 @@ open class CallObjectMethodBenchmark {
     private val javaMethod: Method = functionByMethodReference.javaMethod!!
 
     private val fastKFunctionByMethodReferenceWithoutInstance: FastKFunction<Constructor5> =
-        FastKFunction(functionByMethodReference)
+        FastKFunction.of(functionByMethodReference)
     private val fastKFunctionByMethodReferenceWithInstance: FastKFunction<Constructor5> =
-        FastKFunction(functionByMethodReference, objectInstance)
+        FastKFunction.of(functionByMethodReference, objectInstance)
 
     private val fastKFunctionByReflectionWithoutInstance: FastKFunction<Constructor5> =
-        FastKFunction(functionByReflection)
+        FastKFunction.of(functionByReflection)
     private val fastKFunctionByReflectionWithInstance: FastKFunction<Constructor5> =
-        FastKFunction(functionByReflection, objectInstance)
+        FastKFunction.of(functionByReflection, objectInstance)
 
     private val argumentBucket: ArgumentBucket = fastKFunctionByMethodReferenceWithoutInstance.generateBucket()
         .apply { (0 until 5).forEach { this[it] = it + 1 } }

src/jmh/kotlin/com/mapk/fastkfunction/CallObjectMethodBenchmark.kt

@@ -23,11 +23,12 @@ open class CallTopLevelExtensionFunBenchmark {
     private val javaMethod: Method = functionByMethodReference.javaMethod!!
 
     private val fastKFunctionByMethodReferenceWithoutInstance: FastKFunction<Constructor5> =
-        FastKFunction(functionByMethodReference)
+        FastKFunction.of(functionByMethodReference)
     private val fastKFunctionByMethodReferenceWithInstance: FastKFunction<Constructor5> =
-        FastKFunction(functionByMethodReference, receiverInstance)
+        FastKFunction.of(functionByMethodReference, receiverInstance)
 
-    private val fastKFunctionFromClass: FastKFunction<Constructor5> = FastKFunction(functionFromClass, receiverInstance)
+    private val fastKFunctionFromClass: FastKFunction<Constructor5> =
+        FastKFunction.of(functionFromClass, receiverInstance)
 
     private val argumentBucket: ArgumentBucket = fastKFunctionByMethodReferenceWithoutInstance.generateBucket()
         .apply { (0 until 5).forEach { this[it] = it + 1 } }

src/jmh/kotlin/com/mapk/fastkfunction/CallTopLevelExtensionFunBenchmark.kt

@@ -17,7 +17,7 @@ open class CallTopLevelFunBenchmark {
     private val argumentMap: Map<KParameter, Any?> = function.parameters.associateWith { it.index + 1 }
 
     private val javaMethod: Method = function.javaMethod!!
-    private val fastKFunction: FastKFunction<Constructor5> = FastKFunction(function, null)
+    private val fastKFunction: FastKFunction<Constructor5> = FastKFunction.of(function, null)
     private val argumentBucket: ArgumentBucket = fastKFunction.generateBucket()
         .apply { (0 until 5).forEach { this[it] = it + 1 } }