如何改进三种工作略有不同的方法

Question

我有以下方法，它们根据调用对象的位置返回对象的特定值。

 fun getRoamingStatusErrorItem(): SettingItem {
        return SettingRoamingItem(
            isRoaming = false,
            processingText = "",
            isEnabled = false,
            isErrorVisible = true,
            isProgressVisible = false)
    }

    fun getRoamingStatusProgressItem(): SettingItem {
        return SettingRoamingItem(
            isRoaming = false,
            processingText = "",
            isEnabled = false,
            isErrorVisible = false,
            isProgressVisible = true)
    }

    fun getRoamingStatusProcessingItem(text: String): SettingItem {
        return SettingRoamingItem(
            isRoaming = false,
            processingText = text,
            isEnabled = false,
            isErrorVisible = false,
            isProgressVisible = false)
    }

正如您所看到的，这些方法只是表示对象类型的不同状态；我想了解这是否是一种很好的编码实践来使用它，或者我可以以更好的方式重构它

Answer 1

您可以为SettinRoamingItem使用Builder模式，并为默认值设置isRoaming=false、Process ingText = "", isEnable=false、isErrorViable=true、isProgressViable=false。然后使用类似. with ProcessingText（"text"）. enableVisibleError（）. build（）的内容来获取getter所需的实例。

Answer 2

在不改变整个班级的情况下，你能做的事情不多。唯一“简单”的方法是添加另一个没有isRoaming参数的构造函数：在所有三个示例中，该值都是false。当您始终传递相同的值时：不要传递该值，而是使用构造函数伸缩来提供该值作为“默认值”。

除此之外，如果您愿意投入更多的时间，您应该在这里研究如何使用生成器模式。比如：

SettingRoamingItemBuilder().withText("").withProgressVisible(true)..
  .build()

关键是要确定所有这些参数中最常见的值，并将其用作默认值，这样只有那些偏离默认值的地方才需要调用以相应地更改默认设置。

Answer 3

如果SettinRoamingItem在您的控制之下，您可以给它一些默认值，例如：

class SettingRoamingItem(
  val isRoaming : Boolean = false,
  val processingText : String = "",
  val isEnabled : Boolean = false,
  val isErrorVisible : Boolean = false,
  val isProgressVisible : Boolean = false)

然后您的函数调用可以简化为：

fun getRoamingStatusErrorItem() = SettingRoamingItem(isErrorVisible = true)
fun getRoamingStatusProgressItem() = SettingRoamingItem(isProgressVisible = true)
fun getRoamingStatusProcessingItem(text : String) = SettingRoamingItem(processingText = text)

但是接下来:你为什么想要这样的功能呢？