https://yun7211.github.io/tags/%E4%BB%A3%E7%A0%81%E5%AE%A1%E6%9F%A5/Recent content in 代码审查 on Yun 的个人博客Hugo -- gohugo.iozh-cnMon, 18 May 2026 11:40:00 +0800https://yun7211.github.io/c-language/debug-quality/code-review-checklist/Mon, 18 May 2026 11:40:00 +0800https://yun7211.github.io/c-language/debug-quality/code-review-checklist/<h2 id="审查目标">审查目标 </h2><p>C 代码审查不是挑风格问题，而是提前发现运行时成本最高的问题：内存错误、资源泄漏、接口歧义、错误路径遗漏和不可测试的全局状态。</p> <h2 id="接口语义">接口语义 </h2><p>先看函数接口是否清楚：</p> <ul> <li>参数能否为 <code>NULL</code></li> <li>返回值如何表达错误</li> <li>调用方是否负责释放资源</li> <li>输出参数失败时是否有效</li> <li>函数是否修改传入缓冲区</li> </ul> <p>接口语义不清楚时，后续实现再漂亮也容易出问题。</p> <h2 id="内存和边界">内存和边界 </h2><p>重点检查长度、容量和生命周期：</p> <ul> <li>字符串空间是否包含 <code>\0</code></li> <li><code>memcpy</code>、<code>snprintf</code>、数组下标是否有边界</li> <li>释放路径是否覆盖所有失败分支</li> <li>指针释放后是否还会被访问</li> </ul> <h2 id="错误路径">错误路径 </h2><p>C 项目里很多 bug 只在异常路径出现。审查时要刻意走失败分支：申请失败、文件打开失败、设备不存在、协议字段不合法、部分初始化后返回错误。</p> <h2 id="小结">小结 </h2><p>好的代码审查清单能把经验固化下来。每次发现新问题，都应该把它变成下一次能提前检查的条目。</p>