Android 代码质检员

解决什么问题

Android 项目的代码审查通常依赖个人经验，不同开发者关注的维度差异很大——有人盯着架构分层，有人盯着内存泄漏，有人盯着 XML 里的硬编码颜色值。等到 Code Review 时，往往只能就眼前的问题讨论，缺少一套覆盖全面的检查清单。这个技能的目标是把架构师级别的审查经验固化成可重复执行的流程，让机器先扫一遍，把确定性的问题挑出来，把需要人工判断的留给人。

如何工作

技能的核心是一套六维度检查体系：

• arch — 架构设计：分层是否清晰（Clean Architecture / MVVM / MVI），模块边界是否正确，依赖注入是否规范
• solid — SOLID 原则：单一职责、开闭原则、里氏替换等六大原则的具体检查项，包括类超过 300 行、函数超过 40 行的预警线
• pattern — 设计模式：覆盖创建型、结构型、行为型共 23 种模式的典型 Android 场景与常见误用
• perf / security — 平台最佳实践：内存管理、线程安全、生命周期、性能、安全、代码质量
• ui-res — UI 资源规范：检测颜色、尺寸、字符串的硬编码，支持 XML、Kotlin/Java、Compose 三种场景
• syntax — 语法与类加载安全：!! 非空断言、类型转换、可变集合暴露、四大组件内部类引用泄漏

支持五种调用方式：file（单文件）、dir（目录）、diff（Git 变更）、module（模块）、workflow（完整 Review + 修改流程）。每种方式输出结构化报告，每条问题标注 severity（ERROR / WARN / INFO）、dimension、location（文件名 + 行号）和具体修改建议。workflow 子命令还会在审查后生成优化方案，按约束条件逐条执行修改。

适用边界

技能做的是静态分析，无法覆盖运行时行为、业务逻辑正确性和性能实测数据。它能告诉你 !! 在哪里、Handler 为什么泄漏、布局嵌套了几层，但不能告诉你这段代码在低端机上的实际帧率。把它当作初筛工具，而不是最终判断。