Kotlin 代码审查：让资深开发者抓狂的那些事

资深开发者如何看待你的 Kotlin 代码：基于真实案例的剖析

你写好了代码，推送到仓库，然后屏息等待批准。但等来的不是绿色勾号，而是资深开发者的一堆评论。熟悉吗？如果你用 Kotlin 编写代码，很可能遇到过这种情况：你自认为完美的代码，在 code review 之后变得一无是处。

在本文中，我们将分析 资深开发者最常见的意见 在 Kotlin 代码审查 中频繁出现的情况。我们不会教你语法——我们将展示如何编写代码，使其一次通过审查。开始吧！

1. 空安全：“为什么不用 Elvis 操作符？”

Kotlin 的主要特性之一是 空安全。但许多开发者，尤其是从 Java 转来的，仍然以“检查是否为空再执行”的风格编写代码。资深开发者一眼就能看出来。

糟糕示例（Java 风格）

fun getUserName(user: User?): String {\n    return if (user != null) {\n        user.name\n    } else {\n        "Unknown"\n    }\n}

优秀示例（Kotlin 风格）

fun getUserName(user: User?): String {\n    return user?.name ?: "Unknown"\n}

使用 Elvis 操作符 (?:) 和 安全调用 (?.) 使代码简洁且易读。资深开发者喜欢你在不损失清晰度的情况下节省代码行数。记住：if (x != null) { ... } else { ... } 是不合时宜的。

2. 扩展函数：“你的工具类在哪？”

当你看到类似 StringUtils.capitalize(str) 的静态工具类时，有经验的 Kotlin 开发者会皱眉。扩展函数 正是让 Kotlin 如此优雅的原因。

糟糕示例

object StringUtils {\n    fun capitalize(text: String): String {\n        return text.replaceFirstChar { it.uppercase() }\n    }\n}\n\n// 使用\nval result = StringUtils.capitalize("hello")

优秀示例

fun String.capitalizeFirst(): String {\n    return this.replaceFirstChar { it.uppercase() }\n}\n\n// 使用\nval result = "hello".capitalizeFirst()

资深开发者会说：“你为什么要创建一个垃圾对象？直接扩展类就行了！”使用 扩展函数——这能提高可读性，并遵循 Kotlin 最佳实践 原则。

3. 使用密封类代替枚举：“你的枚举已经过时了”

Kotlin 中的枚举很好，但 密封类 提供了更大的灵活性。特别是当需要在同一层次结构中存储不同类型的数据时。

糟糕示例（枚举）

enum class Result {\n    SUCCESS,\n    ERROR,\n    LOADING\n}\n\nfun handleResult(result: Result) {\n    when (result) {\n        Result.SUCCESS -> // ...\n        Result.ERROR -> // ...\n        Result.LOADING -> // ...\n    }\n}

优秀示例（密封类）

sealed class Result {\n    data class Success(val data: List<Item>) : Result()\n    data class Error(val message: String, val code: Int) : Result()\n    object Loading : Result()\n}\n\nfun handleResult(result: Result) {\n    when (result) {\n        is Result.Success -> println("Data: ${result.data}")\n        is Result.Error -> println("Error: ${result.message}")\n        Result.Loading -> println("Loading...")\n    }\n}

资深开发者会赞赏你使用 密封类 来表示状态。这是 Kotlin 资深开发者建议 中的要点之一——让类型层次结构显式化。

4. 作用域函数：“你忘了 let、apply 和 run”

作用域函数（let、apply、run、with、also）是强大的