verify-quality

# ⚖ 校验关卡 · 代码质量 ## 核心原则 ``` 代码质量 = 可读性 + 可维护性 + 可测试性 劣质代码是技术债，技术债是道基裂痕 复杂度是 bug 的温床 ``` ## 自动检查 运行质量检查脚本（跨平台）： ```bash # 在 skill 目录下运行 node scripts/quality_checker.js <扫描路径> node scripts/quality_checker.js <扫描路径> -v # 详细模式 node scripts/quality_checker.js <扫描路径> --json # JSON 输出 ``` ## 检测指标 ### 复杂度指标 | 指标 | 阈值 | 超标后果 | |------|------|----------| | **圈复杂度** | ≤ 10 | 🟠 警告，建议拆分 | | **函数长度** | ≤ 50 行 | 🟠 警告，建议拆分 | | **文件长度** | ≤ 500 行 | 🟡 提示，考虑拆分 | | **参数数量** | ≤ 5 | 🟠 警告，考虑封装 | | **嵌套深度** | ≤ 4 | 🟠 警告，建议重构 | | **行长度** | ≤ 120 | 🔵 提示 | ### 命名规范 | 类型 | 规范 | 示例 | |------|------|------| | **类名** | PascalCase | `UserService`, `HttpClient` | | **函数名** | snake_case | `get_user`, `process_data` | | **常量** | UPPER_SNAKE | `MAX_RETRY`, `DEFAULT_TIMEOUT` | | **变量** | snake_case | `user_id`, `total_count` | ### 代码异味 | 异味 | 说明 | 严重度 | |------|------|--------| | 重复代码 | 相似代码块 > 10 行 | 🟠 High | | 过长参数列表 | 参数 > 5 个 | 🟡 Medium | | 魔法数字 | 未命名的常量 | 🟡 Medium | | 死代码 | 未使用的函数/变量 | 🔵 Low | | 注释代码 | 被注释的代码块 | 🔵 Low | ## 自动触发时机 | 场景 | 触发条件 | |------|----------| | 复杂模块 | 代码行数 > 200 | | 重构完成 | 重构任务完成时 | | 代码审查 | PR/MR 审查时 | | 提交前 | 代码提交前检查 | ## 校验流程 ``` 1. 扫描代码文件 2. 计算复杂度指标 3. 检测代码异味 4. 验证命名规范 5. 输出质量校验报告 ``` ## 校验报告格式 ``` ## 代码质量校验报告 ✓ 通过 | ✗ 未通过 ### 复杂度指标 - 平均函数复杂度: N - 超标函数数: N - 最大文件行数: N ### 代码异味 - 🟠 High: N - 🟡 Medium: N - 🔵 Low: N ### 问题清单 | 文件 | 行号 | 类型 | 严重度 | 描述 | |------|------|------|--------|------| | ... | ... | ... | ... | ... | ### 结论 可交付 / 需重构后交付 ``` ## 重构建议 ### 降低复杂度 ```python # 🔴 高复杂度 - 道基不稳 def process(data): if condition1: if condition2: if condition3: # 深层嵌套 pass # ✅ 低复杂度 - 道基稳固 def process(data): if not condition1: return if not condition2: return if not condition3: return # 主逻辑 ``` ### 消除重复 ```python # 🔴 重复代码 - 异端 def func1(): # 10行相同逻辑 pass def func2(): # 10行相同逻辑 pass # ✅ 提取公共函数 - 正道 def common_logic(): # 公共逻辑 pass def func1(): common_logic() def func2(): common_logic() ``` ---