review-logic

Installation
SKILL.md

设计与代码逻辑一致性校验

语言要求

所有输出内容必须使用中文,包括校验报告、问题描述、修正建议、总体评价等。

功能说明

校验代码实现的逻辑是否与设计文档一致,确保代码正确实现了系统设计方案,避免实现偏离设计。

文档依赖

进行逻辑一致性校验时,参考以下文档:

使用方式

当用户提供设计文档路径以及需要校验的代码时,执行逻辑一致性校验并生成报告。

输入要求

设计文档(必需):

  • 设计文档:/projectdocs/design/{需求名称}_design.md
  • 设计文档是逻辑校验的主要依据

需求/用户故事文档(可选):

  • 需求分析文档:/projectdocs/require/{需求名称}_requirements_{时间戳}.md
  • 用户故事文档:/projectdocs/require/{需求名称}_userstory_{时间戳}.md
  • 仅作为理解设计背景的辅助,不作为校验标准

待校验代码

  • 可以是单个文件
  • 可以是多个文件
  • 可以是整个模块

校验流程

1. 读取设计文档

  • 从设计文档中提取关键设计点和功能点
  • 识别业务流程、业务规则、约束条件
  • 提取预期的输入输出、边界条件
  • 理解核心业务逻辑与数据流

2. 分析代码实现

  • 理解代码的业务逻辑流程
  • 识别关键的业务处理点
  • 分析输入验证、数据处理、输出结果
  • 查找边界条件处理

3. 逐项对比

  • 将设计文档中的逻辑与代码实现逐一对应
  • 检查每个设计点是否被正确实现
  • 验证业务规则是否被遵守
  • 检查边界条件是否处理完整

4. 生成报告

  • 列出需求与实现的对应关系
  • 指出缺失的需求实现
  • 标注实现有误的部分
  • 提供修正建议

校验维度

1. 功能完整性

  • 需求覆盖:所有需求功能是否都已实现
  • 功能缺失:是否有需求功能未实现
  • 多余功能:是否实现了需求之外的功能

2. 业务逻辑正确性

  • 业务流程:是否按照需求的业务流程实现
  • 业务规则:是否正确实现了所有业务规则
  • 计算逻辑:计算公式、算法是否符合需求
  • 状态转换:状态流转是否符合需求定义

3. 数据处理一致性

  • 输入验证:输入参数验证是否符合需求要求
  • 数据转换:数据格式转换是否正确
  • 输出格式:输出结果是否符合需求定义
  • 数据范围:数据取值范围是否符合需求约束

4. 边界条件处理

  • 异常场景:异常情况处理是否符合设计要求
  • 极值处理:最大值、最小值处理是否正确
  • 空值处理:空值、null 处理是否符合设计要求
  • 并发处理:并发场景处理是否符合设计要求

5. 权限与安全

  • 权限控制:权限验证是否符合设计要求
  • 数据安全:敏感数据处理是否符合要求
  • 操作审计:审计日志是否按设计要求记录

校验报告格式

# 设计与代码逻辑一致性校验报告

## 概览
- 设计文档:xxx_design_20241230_100000.md
- 校验代码:UserService.java
- 校验时间:2024-xx-xx
- 一致性评分:xx%
- 问题总数:xx 个

## 设计覆盖情况

### ✅ 已实现的设计点(x 项)
1. **[设计编号/用例编号] 设计标题/用例名称**
   - 设计说明:...
   - 实现位置:UserService.createUser() (Line 25-50)
   - 实现方式:...
   - 一致性:✅ 完全一致

### ⚠️ 实现有偏差的设计点(x 项)
1. **[设计编号/用例编号] 设计标题/用例名称**
   - 设计说明:...
   - 实现位置:UserService.updateUser() (Line 60-80)
   - 问题描述:...
   - 偏差影响:...
   - 修正建议:...

### ❌ 未实现的设计点(x 项)
1. **[设计编号/用例编号] 设计标题/用例名称**
   - 设计说明:...
   - 影响范围:...
   - 建议:补充实现该设计点

### 🔍 多余的实现(x 项)
1. **UserService.extraMethod()**
   - 实现位置:Line 100-120
   - 问题:该功能不在需求范围内
   - 建议:确认是否需要保留

## 总体评价
- 设计覆盖率:xx%
- 逻辑一致性:xx%
- 主要问题:...
- 修复建议:...

校验清单

功能完整性

  • 所有必需功能是否都已实现
  • 是否有功能缺失
  • 是否有多余的功能实现

业务逻辑

  • 业务流程是否正确
  • 业务规则是否遵守
  • 计算逻辑是否准确
  • 状态转换是否正确

数据处理

  • 输入验证是否完整
  • 数据转换是否正确
  • 输出格式是否符合要求
  • 数据范围是否正确

边界条件

  • 异常场景是否处理
  • 极值情况是否考虑
  • 空值处理是否正确
  • 并发场景是否考虑

权限安全

  • 权限验证是否完整
  • 敏感数据是否保护
  • 审计日志是否记录

校验原则

  1. 以设计为准:设计文档是校验的主要标准
  2. 全面覆盖:检查所有设计点,不遗漏
  3. 细致对比:仔细对比设计与实现的差异
  4. 客观公正:基于事实,不掺杂主观判断
  5. 建设性意见:提供可操作的改进建议

注意事项

  1. 设计文档必须:必须提供设计文档,否则无法进行校验
  2. 理解设计意图:不仅检查表面实现,还要理解设计背后的业务意图
  3. 关注隐性设计要求:一些隐含的设计要求也要考虑(如性能、安全等非功能性要求)
  4. 考虑业务场景:结合实际业务场景判断实现是否合理
  5. 提供具体建议:不仅指出问题,还要提供修复建议
  6. 分段处理大文档:如果设计文档过大,采用分段读取策略
Weekly Installs
2
First Seen
Feb 28, 2026
Installed on
qoder2
gemini-cli2
antigravity2
codebuddy2
claude-code2
github-copilot2