| 电池管理系统代码 评 审 报 告 | |||||||||||
| 项目编号 | 编制部门 | ||||||||||
| 项目名称 | 电池管理系统BMS | 编 制 人 | |||||||||
| 产品型号 | BCU16-1/BMU14-2 | 编制日期 | |||||||||
| 产品名称 | 电池管理系统BMS | 参加人员 | 技术部全体人员 | ||||||||
| 评审时间 | |||||||||||
| 评审地点 | |||||||||||
| 评 审 内 容 | |||||||||||
| 评审对象：BCU16-1/BMU14-2.C语言代码（含策略代码） 评审内容： （1）清晰度 －代码是否清晰容易理解？（是，清晰容易理解） －代码是否符合编码标准？（是，符合相关的C语言编码规范的要求） －代码是否可以重做使得更加清晰？（否，无需重做，很清晰） （2）可维护性 －其他程序员可以维护此代码吗？（是，其他人员可以进行维护） －很好的注释和适当的格式化了吗？（是，有完整内部格式化的注释） （3）准确性 －代码完成了想要实现的事情了吗？（是，符合驱动设计方案及策略设计方案的要求） －如果使用了算法，实现得正确吗？（有，经单元测试，其算法正确） （4）可靠性和健壮性 －代码容错了吗？（有，有代码容错处理） －代码处理了不正常的情况吗？代码处理了不正常的输入了吗？（有，不正常的情况及输入都有处理） －当遇到不能预见的条件时，很好地处理了失败了？（有，很好地处理了失败） （5）安全性 －代码易受非法访问攻击、恶意使用或修改吗？（无，不涉及） （6）伸缩性 －代码是否有瓶颈从而阻止系统的增长来适应增加的负载、数据、用户或输入？（无，不涉及） （7）重用性 －代码能够在其他应用程序中使用吗？（是，部分代码可以在其他应用程序中使用） －代码可以做得更通用吗？（是，可以） （8）效率 －代码高效的使用了内存、CPU、带宽或其他系统资源吗？（是，主要是内存使用的优化） －是否可以进行再次优化？（是，可以的，目前可以不进行） | |||||||||||
| 附 件 | BCU16-1/BMU14-2.C语言代码 共 1份 | ||||||||||
| 评 审 意 见 | |||||||||||
| No | 指摘·问题 | 处理·对策 | 改善日期 | 确认 | |||||||
| 1 | 建议兼容C++编码的要求，以便扩展； | 适当进行修改，兼容C++编码要求； | 2015/06/27 | ||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| / / | |||||||||||
| 备注：其它内容可以参考相关的会议纪要。 | |||||||||||
| 评 审 结 论 | |||||||||||
| BCU16-1/BMU14-2.C语言代码经代码评审，其符合了清晰度、可维护性、准确性等方面的要求，符合驱动设计方案及策略设计方案的要求。 评审小组认为此代码通过评审。 | |||||||||||
| 评 审 签 字 | |||||||||||
年 月 日 | |||||||||||
| 发 行 | |||||||||||
