我应该摆脱大开关外壳吗？

Question

我有一个工厂，其中包括许多HTML属性生成器，该生成器根据属性的类型返回其中一个，因此我想看看是否有更好的方法。

class AttributeHtmlGeneratorFactory {
    fun create(property: String): AttributeHtmlGenerator {
        when (property) {
            "animation" -> {
                return AnimationHtmlGenerator()
            }
            ...
            "left", "top" -> {
                return PositionHtmlGenerator()
            }
            ...
            "scaleX" , "scaleY", ... , "direction" -> {
                return UnusedAttributesHtmlGenerator()
            }

当开关中有20个开关案例时。

这是所有这些类都使用的界面

interface AttributeHtmlGenerator {
    fun generateHtml(member: KProperty1<HtmlComponentDataModel, *>, component: HtmlComponentDataModel ): String
}

，这是我在哪里以及如何使用所有这些界面：

        var result = ""

        HtmlComponentDataModel::class.memberProperties.forEach { member ->
            val generator = AttributeHtmlGeneratorFactory().create(member.name)
            result = result.plus(generator.generateHtml(member, component))
        }

        return result

此外，这是界面的简单实现：

class ButtonFillHtmlGenerator : AttributeHtmlGenerator {
    override fun generateHtml(member: KProperty1<HtmlComponentDataModel, *>, component: HtmlComponentDataModel): String {
        var result = ""

        member.get(component)?.let {
            result = result.plus("background-color:${it};")
        }

        return result
    }
}

无论如何是否有可以使情况变得更好？

Answer 1

如果您只想重新格式化何时陈述，我建议您喜欢这样做：

fun create(property: String): AttributeHtmlGenerator = when (property)
{
    "animation" -> AnimationHtmlGenerator()
    "left", "top" -> PositionHtmlGenerator()
    "scaleX", "scaleY", "direction" -> UnusedAttributesHtmlGenerator()
    else -> error("No generator found for property $property")
}

如果要跨模块划分此逻辑，则使用 map 。

class AttributeHtmlGeneratorFactory {
    private val generatorMap = mutableMapOf<String, () -> AttributeHtmlGenerator>()

    init {
        assignGeneratorToProperties("animation") { AnimationHtmlGenerator() }
        assignGeneratorToProperties("left", "top") { PositionHtmlGenerator() }
    }

    fun create(property: String): AttributeHtmlGenerator {
        return generatorMap[property]?.invoke() ?: error("No generator found for property $property")
    }

    fun assignGeneratorToProperties(vararg properties: String, provider: () -> AttributeHtmlGenerator) {
        properties.forEach {
            generatorMap[it] = provider
        }
    }
}

这样，您可以在代码的一部分中调用godegeneratortoperties，从而将初始化逻辑分开。

在性能方面，当您有几种情况时，/if-else语句确实是表现的，但哈希图的表现要优于许多元素。您可以根据案件决定使用什么。