前言在审查了100个电机控制项目的代码后我总结了一份完整的代码审查清单。这份清单可以帮助工程师在代码审查时不遗漏关键问题。一、命名规范10项1.1 变量命名变量名是否使用有意义的英文单词不用a/b/c/x/y是否遵循命名约定camelCase或snake_case一致布尔变量是否以is/has/can开头如isReady, hasError全局变量是否有前缀标识如g_count常量是否全大写如MAX_MOTOR_SPEED示例// ❌ 不好 int m; float v; int flag; // ✅ 好 int motorSpeed; float outputVoltage; bool isMotorRunning;1.2 函数命名函数名是否清晰表达功能不用func1/func2是否使用动词开头如initMotor, calculatePID获取值的函数是否用get前缀如getMotorSpeed设置值的函数是否用set前缀如setMotorSpeed二、逻辑正确性15项2.1 条件判断是否有悬挂else问题多个if嵌套时布尔条件是否正确是否有逻辑反转是否检查了所有可能的边界情况switch语句是否有default分支是否使用了魔法数字应该用常量替代示例// ❌ 不好 if (speed 0) if (voltage 12) startMotor(); else stopMotor(); // 哪个if的else不清楚 // ✅ 好 if (speed 0 voltage 12) { startMotor(); } else { stopMotor(); }2.2 循环循环条件是否正确是否会死循环循环变量是否正确更新是否有越界风险数组/指针访问嵌套循环是否有正确的初始化2.3 数据类型是否有类型不匹配的赋值整数除法是否产生精度丢失是否正确处理了有符号和无符号整数浮点数比较是否直接用应该用误差范围三、内存管理10项3.1 指针指针是否正确初始化不要野指针是否检查了malloc/calloc的返回值是否有内存泄漏malloc后没有free是否有double free问题指针使用后是否被设为NULL示例// ❌ 不好 int* data (int*)malloc(sizeof(int) * 100); // 使用data但没有检查是否分配成功 data[50] 5; // 如果malloc失败这是危险的 // ✅ 好 int* data (int*)malloc(sizeof(int) * 100); if (data NULL) { printf(Memory allocation failed\n); return -1; } // 使用data free(data); data NULL;3.2 数组是否有数组越界访问是否正确计算了数组大小是否有缓冲区溢出风险四、电机控制特定检查10项4.1 PID控制PID参数Kp/Ki/Kd是否有初值积分项是否有限制防止积分饱和微分项是否做了平滑处理防止噪声输出是否有限制不超过PWM范围是否处理了目标值跳跃防止微分尖峰4.2 PWM控制PWM频率是否合理通常1-20kHzPWM占空比是否正确计算0-100%是否处理了死区时间避免上下管同时导通是否有过流保护4.3 中断处理中断处理函数是否尽可能短是否避免了中断嵌套问题是否正确保存/恢复了寄存器状态是否有中断优先级冲突五、代码风格5项缩进是否一致Tab还是空格行长是否过长建议80字符是否有足够的注释复杂逻辑要说明函数长度是否合理不要超过100行是否去除了注释掉的代码死代码六、安全性5项是否检查了输入数据的有效性是否有整数溢出风险是否正确处理了错误情况是否有时序问题竞态条件是否有硬编码的密钥/密码如果有的话使用这个清单的方法方法1完整审查耗时30分钟逐项检查50个检查点记录发现的问题与代码作者讨论并修改方法2快速审查耗时10分钟重点检查命名规范、逻辑正确性、内存管理代码是否能运行、结果是否正确是否有明显的bug方法3自动化审查可以结合代码审查工具如cppcheck、splint来自动检查其中的一些项目。总结好的代码审查应该是✓系统化— 有清单不遗漏✓教育性— 帮助提升代码质量✓可执行— 能指出具体改进方向这份清单可以帮助你建立一个标准的代码审查流程。