递归筛选和映射属性列表

Question

我使用静态编程语言反射来检查具有特定注释的属性是否为空。

给出以下示例：

data class DataClass(
    @SomeRandomAnnotation
    val otherAnnotated: String?,
    val inner: InnerClass
)

data class AnotherDataClass(
    @SomeRandomAnnotation
    val annotatedProperty: String?,
    val dataClass: DataClass
) {

    fun checkCreditAnalysisConstrain() {
        print(checkConstrain(this))
    }
}

以及检查它的函数：

fun checkConstrain(parentClass: Any): List<String> {
    val filter = parentClass::class.memberProperties.filter {
        if (memberIsDataClass(it)) checkConstrain(getMemberPropertyInstance(parentClass, it))

        hasAnnotation(it) && propertyIsNull(it, parentClass)
    }
    return filter.map { formatResult(parentClass, it) }
}

这个想法是，函数将迭代我的类的属性，检查它们是否有注释，并检查值是否为空。如果属性是一个数据类，代码会递归地计算Childs的属性。

之后，我映射结果，将KProperty转换为一个简单的字符串，该字符串可供人阅读，包含类名和属性名。

问题是上面的代码没有按预期工作。返回的属性只是来自第一级类的属性。

如果不进行筛选，而是运行forEach并打印结果，则会得到预期的属性。所以我很确定这和过滤器内部的循环有关。

你认为有没有更实用的方法？我只是担心我不需要一个“临时”列表，向列表中添加值，然后重置它。

Answer 1

您的函数递归地调用自身，但不处理该递归调用的返回列表。这就是为什么你只会得到顶级课程的成绩。

另外，在我看来，你不应该依赖于过滤器调用产生的副作用。它可能会工作，但函数的文档并不能保证集合中的每个项都会准确调用它一次。因此，应该有一个单独的for循环来执行递归调用，并且应该将结果添加到现有结果中。

fun checkConstrain(parent: Any): List<String> {
    val memberProperties = parent::class.memberProperties
    var result = memberProperties
        .filter { hasAnnotation(it) && propertyIsNull(it, parent) }
        .map { formatResult(parent, it) }
    memberProperties.filter { memberIsDataClass(it) }
        .mapNotNull { getMemberPropertyInstance(parent, it) }
        .forEach { result += checkConstrain(it) }
    return result
}

您没有为您使用的几个函数提供代码。这是我给他们用的：

val KProperty<*>.returnTypeClass get() = this.returnType.classifier as? KClass<*>
fun <T> memberIsDataClass(member: KProperty<T>) = member.returnTypeClass?.isData == true
fun <T> getMemberPropertyInstance(parent: Any, property: KProperty<T>) = property.getter.call(parent)
fun <T> hasAnnotation(property: KProperty<T>) = property.annotations.firstOrNull { it.annotationClass == SomeRandomAnnotation::class } != null
fun <T> propertyIsNull(property: KProperty<T>, parent: Any) = getMemberPropertyInstance(parent, property) == null
fun formatResult(parent: Any, property: KProperty<*>) = "$parent's property(${property.name}) is annotated with SomeRandomAnnotation and is null."