القائمة
基于腾讯 Kotlin 代码规范对 Kotlin 代码进行全面审查，涵盖变量、类型、类与对象、数据类、集合、函数、泛型、协程、元编程等方面，结合代码上下文提供详细的改进建议。触发词：kotlin代码审查、审查kotlin代码、检查kotlin代码、kotlin code review、review kotlin code
التثبيت باستخدام Codex أو Claude انسخ هذا Prompt والصقه في Codex أو Claude أو مساعد آخر ليراجع صفحة Skill ويثبّتها لك.
استنادا إلى تصنيف SOC المهني
查询和获取 Kuikly 第三方 UI 组件的详细信息。当用户询问 Kuikly 可用组件、KuiklyUI 第三方库，或需要特定 Kuikly 第三方组件的详细文档时使用此技能。该技能会自动同步组件仓库，并使用 deepwiki-mcp 获取全面的组件文档。
Kuikly UI 框架开发助手。帮助使用 Kuikly 组件（View、Text、Button、List、Modal、ActionSheet、Input 等 UI 组件）和模块（Router、Network、SP 等系统模块），自动提供正确的 import 语句、API 使用方法和完整代码示例。支持传统 Kuikly DSL（attr/event）和 Compose DSL 两种开发方式。解决页面创建、组件使用、布局实现、事件处理、编译错误等 Kuikly 开发问题。
| name | kotlin-code-review |
| description | 基于腾讯 Kotlin 代码规范对 Kotlin 代码进行全面审查，涵盖变量、类型、类与对象、数据类、集合、函数、泛型、协程、元编程等方面，结合代码上下文提供详细的改进建议。触发词：kotlin代码审查、审查kotlin代码、检查kotlin代码、kotlin code review、review kotlin code |
本 skill 提供基于项目 kotlin-standard 规范的 Kotlin 代码审查能力。审查涵盖以下维度：
按照以下优先级进行审查：
使用结构化格式输出审查结果（见输出格式部分）
val 而不是 varby lazylateinit，禁用 isInitialized// ❌ 反例
var count = 0 // 应该用 val
class Manager {
var result = 0 // 作用域过大
}
// ✅ 正例
val count = 0
class Manager {
fun calculate(): Int {
val result = 0 // 局部变量
return result
}
}
T?、?.、?: 处理空值!! 非空断言typealias 简化复杂类型// ❌ 反例
val name = person!!.address!!.city // 使用 !!
fun getData(): Any // 返回 Any
// ✅ 正例
val name = person?.address?.city ?: "Unknown"
fun getData(): List<String> // 明确类型
// ❌ 反例
class User {
val temperature: Double
get() = fetchFromAPI() // 属性不应有副作用
}
// ✅ 正例
class User {
fun fetchTemperature(): Double = fetchFromAPI()
}
copy()（浅复制）// ❌ 反例
data class User(
var name: String, // 可变属性
val tags: MutableList<String>, // 可变集合
val onLogin: () -> Unit // 函数类型
)
// ✅ 正例
data class User(
val name: String,
val tags: List<String>
)
forEach 中使用 returnSequence// ❌ 反例
fun getUsers(): MutableList<User> // 暴露可变集合
numbers.forEach { if (it > 5) return } // forEach 中 return
// ✅ 正例
fun getUsers(): List<User>
numbers.firstOrNull { it > 5 }
require 和 check 强制执行契约// ❌ 反例
fun setAge(age: Int) { this.age = age } // 缺少前置条件检查
resizeImage(1024, 768, 1.5) // 参数含义不明
// ✅ 正例
fun setAge(age: Int) {
require(age > 0) { "Age must be positive" }
this.age = age
}
resizeImage(width = 1024, height = 768, scaleFactor = 1.5)
reified 避免类型擦除// ❌ 反例
fun <T> isInstanceOf(value: Any): Boolean {
return value is T // 编译错误
}
// ✅ 正例
inline fun <reified T> isInstanceOf(value: Any): Boolean {
return value is T
}
GlobalScopeMutex// ❌ 反例
GlobalScope.launch { } // 生命周期不受控
synchronized(lock) { delay(100) } // 协程中使用线程锁
// ✅ 正例
viewModelScope.launch { }
mutex.withLock { }
// ❌ 反例
synchronized(Worker::class) { } // 类引用可能不唯一
val kClass = Class.forName(name).kotlin // 启动时使用 Kotlin 反射
// ✅ 正例
synchronized(this) { }
val jClass = Class.forName(name) // 使用 Java 反射
lateinitdata 关键字定义数据类as?Flow 不声明为 suspendrun {}ForEachOnRangeSpreadOperatorgcinline 优化 lambdaasSequence# Kotlin 代码审查报告
**文件**：`<文件路径>`
**审查时间**：<时间>
**总体评估**：优秀/良好/需改进
---
## 严重问题（P0）🔴
### 🔴 [严重] <问题标题>
**位置**：第 X 行
**问题**：<问题描述>
**违反规范**：<具体规范条目>
**风险**：<可能导致的后果>
**建议**：
\`\`\`kotlin
// 改进后的代码
\`\`\`
---
## 重要问题（P1）🟡
### 🟡 [重要] <问题标题>
**位置**：第 X 行
**问题**：<问题描述>
**违反规范**：<具体规范条目>
**建议**：
\`\`\`kotlin
// 改进后的代码
\`\`\`
---
## 建议（P2）🔵
### 🔵 [建议] <问题标题>
**位置**：第 X 行
**问题**：<问题描述>
**建议**：<改进建议>
---
## 正面反馈 ✅
### ✅ [好的实践] <标题>
**位置**：第 X 行
**原因**：<为什么这是好的实践>
---
## 总结
### 优点
- <列出代码的优点>
### 改进领域
- <列出需要改进的方面>
### 建议
<总体改进建议和优先级>
用户：kotlin代码审查 UserManager.kt
助手：[执行完整审查流程，生成结构化报告]
用户：检查这个文件的协程使用是否符合规范
助手：[重点审查协程相关代码]
用户：帮我review这段代码：
\`\`\`kotlin
fun getData() = GlobalScope.launch {
val result = api.fetch()!!
return result
}
\`\`\`
助手：[针对代码片段进行审查]
本 skill 基于以下规范文档：
references/variable.md - 变量使用规范references/type.md - 类型安全规范references/class_and_object.md - 类与对象规范references/data_class.md - 数据类规范references/collections.md - 集合操作规范references/function.md - 函数设计规范references/generics.md - 泛型规范references/coroutines.md - 协程规范references/metaprogramming.md - 元编程规范references/kotlin_best_practices.md - Kotlin 最佳实践