标签:
General
代码可以正常工作
小伙伴的代码写得有些凌乱,主要有些被废弃的cpp仍未删除,增加了阅读难度
作为个人代码,代码并没有统一规范,不过我和伙伴的代码都大致符合通用规范
并没有出现重复或者冗余的代码,但是留下了编写程序中的痕迹,比如无用类和cpp文件未删除等情况
封装度很高,计算的过程进行了封装(calculation类)
没有发现可以被替代的全局变量
这位同学的代码其实挺干净的,没有被注释掉的代码。
没有错误。简洁清楚…
没有可以被替换为库函数的代码段。
这个代码中没有任何用于调试的代码段。
Security
进行了基础的输入安全性检查,比如输入文件为空或者不存在会出现error提示
没有用到第三方代码……
输出值没有经过检查
无效的参数确实进行了处理
Documentation
小伙伴的习惯非常好,有注释,代码简洁明了,易于理解
Test
单元测试都没有完成
标签:
原文地址:http://www.cnblogs.com/cheney223/p/4847616.html