Skip to content
This repository was archived by the owner on Jan 20, 2023. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ plugins {
}

group = "com.mapk"
version = "0.24"
version = "0.25"

java {
sourceCompatibility = JavaVersion.VERSION_1_8
Expand Down
13 changes: 13 additions & 0 deletions src/main/kotlin/com/mapk/conversion/KConvert.kt
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
package com.mapk.conversion

import kotlin.reflect.KClass

@Target(AnnotationTarget.ANNOTATION_CLASS)
@Retention(AnnotationRetention.RUNTIME)
@MustBeDocumented
annotation class KConvertBy(val converters: Array<KClass<out AbstractKConverter<*, *, *>>>)

abstract class AbstractKConverter<A : Annotation, S : Any, D : Any>(protected val annotation: A) {
abstract val srcClass: KClass<S>
abstract fun convert(source: S?): D?
}
2 changes: 1 addition & 1 deletion src/main/kotlin/com/mapk/kmapper/BoundParameterForMap.kt
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ internal sealed class BoundParameterForMap<S> {
val propertyClazz = property.returnType.classifier as KClass<*>

// コンバータが取れた場合
paramClazz.getConverters()
(param.getConverters() + paramClazz.getConverters())
.filter { (key, _) -> propertyClazz.isSubclassOf(key) }
.let {
if (1 < it.size) throw IllegalArgumentException("${param.name} has multiple converter. $it")
Expand Down
4 changes: 3 additions & 1 deletion src/main/kotlin/com/mapk/kmapper/ParameterForMap.kt
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,9 @@ internal class ParameterForMap<T : Any> private constructor(
clazz.java
}
// リストの長さが小さいと期待されるためこの形で実装しているが、理想的にはmap的なものが使いたい
private val converters: Set<Pair<KClass<*>, KFunction<T>>> = clazz.getConverters()
@Suppress("UNCHECKED_CAST")
private val converters: Set<Pair<KClass<*>, KFunction<T>>> =
(param.getConverters() as Set<Pair<KClass<*>, KFunction<T>>>) + clazz.getConverters()

private val convertCache: ConcurrentMap<KClass<*>, ParameterProcessor> = ConcurrentHashMap()

Expand Down
20 changes: 17 additions & 3 deletions src/main/kotlin/com/mapk/kmapper/ParameterUtils.kt
Original file line number Diff line number Diff line change
@@ -1,19 +1,23 @@
package com.mapk.kmapper

import com.mapk.annotations.KConverter
import com.mapk.conversion.KConvertBy
import com.mapk.core.KFunctionWithInstance
import kotlin.reflect.KClass
import kotlin.reflect.KFunction
import kotlin.reflect.KParameter
import kotlin.reflect.full.companionObjectInstance
import kotlin.reflect.full.findAnnotation
import kotlin.reflect.full.functions
import kotlin.reflect.full.isSubclassOf
import kotlin.reflect.full.primaryConstructor
import kotlin.reflect.full.staticFunctions
import kotlin.reflect.jvm.isAccessible

internal fun <T : Any> KClass<T>.getConverters(): Set<Pair<KClass<*>, KFunction<T>>> =
convertersFromConstructors(this) + convertersFromStaticMethods(this) + convertersFromCompanionObject(this)

private fun <T> Collection<KFunction<T>>.getConverterMapFromFunctions(): Set<Pair<KClass<*>, KFunction<T>>> {
private fun <T> Collection<KFunction<T>>.getConvertersFromFunctions(): Set<Pair<KClass<*>, KFunction<T>>> {
return filter { it.annotations.any { annotation -> annotation is KConverter } }
.map { func ->
func.isAccessible = true
Expand All @@ -23,14 +27,14 @@ private fun <T> Collection<KFunction<T>>.getConverterMapFromFunctions(): Set<Pai
}

private fun <T : Any> convertersFromConstructors(clazz: KClass<T>): Set<Pair<KClass<*>, KFunction<T>>> {
return clazz.constructors.getConverterMapFromFunctions()
return clazz.constructors.getConvertersFromFunctions()
}

@Suppress("UNCHECKED_CAST")
private fun <T : Any> convertersFromStaticMethods(clazz: KClass<T>): Set<Pair<KClass<*>, KFunction<T>>> {
val staticFunctions: Collection<KFunction<T>> = clazz.staticFunctions as Collection<KFunction<T>>

return staticFunctions.getConverterMapFromFunctions()
return staticFunctions.getConvertersFromFunctions()
}

@Suppress("UNCHECKED_CAST")
Expand All @@ -49,6 +53,16 @@ private fun <T : Any> convertersFromCompanionObject(clazz: KClass<T>): Set<Pair<
} ?: emptySet()
}

@Suppress("UNCHECKED_CAST")
internal fun KParameter.getConverters(): Set<Pair<KClass<*>, KFunction<*>>> {
return annotations.mapNotNull { paramAnnotation ->
paramAnnotation.annotationClass
.findAnnotation<KConvertBy>()
?.converters
?.map { it.primaryConstructor!!.call(paramAnnotation) }
}.flatten().map { (it.srcClass) to it::convert as KFunction<*> }.toSet()
}

// 引数の型がconverterに対して入力可能ならconverterを返す
internal fun <T : Any> Set<Pair<KClass<*>, KFunction<T>>>.getConverter(input: KClass<out T>): KFunction<T>? =
this.find { (key, _) -> input.isSubclassOf(key) }?.second
Expand Down
4 changes: 3 additions & 1 deletion src/main/kotlin/com/mapk/kmapper/PlainParameterForMap.kt
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,9 @@ internal class PlainParameterForMap<T : Any> private constructor(
clazz.java
}
// リストの長さが小さいと期待されるためこの形で実装しているが、理想的にはmap的なものが使いたい
private val converters: Set<Pair<KClass<*>, KFunction<T>>> = clazz.getConverters()
@Suppress("UNCHECKED_CAST")
private val converters: Set<Pair<KClass<*>, KFunction<T>>> =
(param.getConverters() as Set<Pair<KClass<*>, KFunction<T>>>) + clazz.getConverters()

fun <U : Any> mapObject(value: U): Any? {
val valueClazz: KClass<*> = value::class
Expand Down
137 changes: 137 additions & 0 deletions src/test/kotlin/com/mapk/kmapper/ConversionTest.kt
Original file line number Diff line number Diff line change
@@ -0,0 +1,137 @@
package com.mapk.kmapper

import com.mapk.conversion.AbstractKConverter
import com.mapk.conversion.KConvertBy
import java.lang.IllegalArgumentException
import java.math.BigDecimal
import java.math.BigInteger
import kotlin.reflect.KClass
import kotlin.reflect.jvm.jvmName
import org.junit.jupiter.api.Assertions.assertEquals
import org.junit.jupiter.api.DisplayName
import org.junit.jupiter.api.Nested
import org.junit.jupiter.params.ParameterizedTest
import org.junit.jupiter.params.provider.EnumSource
import org.junit.jupiter.params.provider.ValueSource

@DisplayName("KConvertアノテーションによる変換のテスト")
class ConversionTest {
@Target(AnnotationTarget.VALUE_PARAMETER)
@Retention(AnnotationRetention.RUNTIME)
@MustBeDocumented
@KConvertBy([FromString::class, FromNumber::class])
annotation class ToNumber(val destination: KClass<out Number>)

class FromString(annotation: ToNumber) : AbstractKConverter<ToNumber, String, Number>(annotation) {
private val converter: (String) -> Number = when (annotation.destination) {
Double::class -> String::toDouble
Float::class -> String::toFloat
Long::class -> String::toLong
Int::class -> String::toInt
Short::class -> String::toShort
Byte::class -> String::toByte
BigDecimal::class -> { { BigDecimal(it) } }
BigInteger::class -> { { BigInteger(it) } }
else -> throw IllegalArgumentException("${annotation.destination.jvmName} is not supported.")
}

override val srcClass = String::class
override fun convert(source: String?): Number? = source?.let(converter)
}

class FromNumber(annotation: ToNumber) : AbstractKConverter<ToNumber, Number, Number>(annotation) {
private val converter: (Number) -> Number = when (annotation.destination) {
Double::class -> Number::toDouble
Float::class -> Number::toFloat
Long::class -> Number::toLong
Int::class -> Number::toInt
Short::class -> Number::toShort
Byte::class -> Number::toByte
BigDecimal::class -> { { BigDecimal.valueOf(it.toDouble()) } }
BigInteger::class -> { { BigInteger.valueOf(it.toLong()) } }
else -> throw IllegalArgumentException("${annotation.destination.jvmName} is not supported.")
}

override val srcClass = Number::class
override fun convert(source: Number?): Number? = source?.let(converter)
}

data class Dst(@ToNumber(BigDecimal::class) val number: BigDecimal)
data class NumberSrc(val number: Number)
data class StringSrc(val number: String)

enum class NumberSource(val values: Array<Number>) {
Doubles(arrayOf(1.0, -2.0, 3.5)),
Floats(arrayOf(4.1f, -5.09f, 6.00001f)),
Longs(arrayOf(7090, 800, 911)),
Ints(arrayOf(0, 123, 234)),
Shorts(arrayOf(365, 416, 511)),
Bytes(arrayOf(6, 7, 8))
}

@Nested
@DisplayName("KMapper")
inner class KMapperTest {
@ParameterizedTest
@EnumSource(NumberSource::class)
@DisplayName("Numberソース")
fun fromNumber(numbers: NumberSource) {
numbers.values.forEach {
val actual = KMapper(::Dst).map(NumberSrc(it))
assertEquals(0, BigDecimal.valueOf(it.toDouble()).compareTo(actual.number))
}
}

@ParameterizedTest
@ValueSource(strings = ["100", "2.0", "-500"])
@DisplayName("Stringソース")
fun fromString(str: String) {
val actual = KMapper(::Dst).map(StringSrc(str))
assertEquals(0, BigDecimal(str).compareTo(actual.number))
}
}

@Nested
@DisplayName("PlainKMapper")
inner class PlainKMapperTest {
@ParameterizedTest
@EnumSource(NumberSource::class)
@DisplayName("Numberソース")
fun fromNumber(numbers: NumberSource) {
numbers.values.forEach {
val actual = PlainKMapper(::Dst).map(NumberSrc(it))
assertEquals(0, BigDecimal.valueOf(it.toDouble()).compareTo(actual.number))
}
}

@ParameterizedTest
@ValueSource(strings = ["100", "2.0", "-500"])
@DisplayName("Stringソース")
fun fromString(str: String) {
val actual = PlainKMapper(::Dst).map(StringSrc(str))
assertEquals(0, BigDecimal(str).compareTo(actual.number))
}
}

@Nested
@DisplayName("BoundKMapper")
inner class BoundKMapperTest {
@ParameterizedTest
@EnumSource(NumberSource::class)
@DisplayName("Numberソース")
fun fromNumber(numbers: NumberSource) {
numbers.values.forEach {
val actual = BoundKMapper(::Dst, NumberSrc::class).map(NumberSrc(it))
assertEquals(0, BigDecimal.valueOf(it.toDouble()).compareTo(actual.number))
}
}

@ParameterizedTest
@ValueSource(strings = ["100", "2.0", "-500"])
@DisplayName("Stringソース")
fun fromString(str: String) {
val actual = BoundKMapper(::Dst, StringSrc::class).map(StringSrc(str))
assertEquals(0, BigDecimal(str).compareTo(actual.number))
}
}
}