技能 · 编程
代码评审清单:安全/边界/命名 3 组
按安全、边界、命名三组清单评审代码,先看有没有注入和越权,再看空值与并发,最后看可读性,避免只挑格式问题。
适用边界:适合团队合并请求的常规评审;紧急热修和实验性分支可以只过安全与边界两组,格式问题交给自动化工具处理更省时间。
怎么做
- 先看安全组,检查外部输入是否校验、权限是否判断、日志是否泄露敏感信息。
- 再看边界组,关注空值、超长、并发以及异常分支是否都有处理路径。
- 最后看命名与结构,函数是否只做一件事,变量名能否直接读出含义。
- 把意见分成必须改和建议改两类,必须改的给出具体改法或示例代码。
交付前检查
- 安全组没有遗留未校验的外部输入和越权路径。
- 边界分支都有处理,空值和异常不会直接抛出到用户界面。
- 评论指向具体行并说明原因,没有只写这里不好。
可复制提示词
你是资深代码评审人。输入:【代码 diff】=____;【业务背景】=____;【项目规范】=____。请按安全、边界、命名三组输出评审意见:位置、问题、风险等级、建议改法。要求先安全后风格,每条指向具体代码,区分必须改与建议改。不要臆测未展示的上下文,最后给出合并前必须解决的三条。
相关工具
同类技能
JSON 转 TypeScript 类型:先定可选字段
把接口返回的 JSON 样本转成 TypeScript 类型,先判断哪些字段真的可选,避免把所有字段都标成 any 或者全加问号。
查看 →正则编写:先写测试串再写表达式
先写一批应该匹配和不该匹配的样本串,再写正则,逐条跑通才收工,避免表达式只在作者想到的那一种格式上正确。
查看 →SQL 慢查询定位:先看执行计划再改索引
先看执行计划和扫描行数,再判断是缺索引、索引失效还是写法导致全表扫描,改完用同一语句复测耗时。
查看 →更新时间:2026-10-29 · 步骤是通用做法,平台/客户规则以实际页面为准;不承诺排名、过稿或收益。
跨库学习路径
按共同关键词匹配,不是随机推荐;从任务到提示词/教程/项目继续走。