Kotlin优化：将Fragment重复调用方法封装为单一参数化方法

这是个非常典型的代码重复场景，咱们可以通过提取通用方法来彻底消除冗余，同时让代码更易维护！

分析重复逻辑 #

你目前的三个方法做的事情几乎完全一致：

• 获取FragmentManager实例
• 创建目标Fragment的实例
• 调用show()方法弹出对话框

唯一的差异就是具体的Fragment类和对应的tag字符串，这正是我们可以抽成参数的部分。

重构方案 #

首先，我们创建一个通用的私有方法，把重复的逻辑封装进去：

private fun openDialogFragment(fragment: Fragment, tag: String) {
    // 用?.let避免fragmentManager为空的安全问题
    fragmentManager?.let { fm ->
        fragment.show(fm, tag)
    }
}

然后，原来的三个方法就可以简化成简单的调用：

private fun openAboutDialogue() {
    openDialogFragment(GetStartedFragment(), "About the App")
}

private fun openNewRouteDialogue() {
    openDialogFragment(NewRouteFragment(), "NewRoute")
}

private fun openEndRouteDialogue() {
    openDialogFragment(TrafficDataFragment(), "GetTraffic")
}

额外优化（可选） #

如果你的Fragment后续需要传入初始化参数（比如用setArguments()），还可以把方法扩展成接受一个Fragment初始化的lambda，让灵活性更高：

private fun <T : Fragment> openDialogFragment(fragmentCreator: () -> T, tag: String) {
    fragmentManager?.let { fm ->
        val fragment = fragmentCreator()
        fragment.show(fm, tag)
    }
}

// 调用方式（比如给Fragment传参数）
private fun openAboutDialogueWithParams() {
    openDialogFragment({
        GetStartedFragment().apply {
            arguments = Bundle().apply {
                putString("KEY_NAME", "value")
            }
        }
    }, "About the App")
}

这样不管Fragment有没有参数，都能通过这个通用方法来处理，扩展性更强。

内容的提问来源于stack exchange，提问作者user9590653