代码审查清单

时间:2014-03-01 12:05:18

标签: asp.net

请提供一些参数来评估代码效率,直到现在我在代码清单中包含以下内容:

  1. 警告在代码中(否/是)
  2. 按工具报告进行代码分析
  3. 未使用
  4. 单元测试用例
  5. 压痕
  6. 空参考例外
  7. 命名惯例
  8. 代码可重用性
  9. 代码一致性
  10. 评论
  11. 代码可读性
  12. 使用泛型
  13. 速度
  14. 处置非托管资源
  15. 异常处理
  16. 代码长度(行数)每个方法30-40行
  17. 是否使用嵌套For / Foreach循环?
  18. 使用Linq或Lambda
  19. 根据范围使用访问说明符(私有,公共,受保护,内部,受保护的内部)
  20. 在需要维护解耦的地方使用接口
  21. 根据其用法和需要将课程标记为密封或静态或抽象。
  22. 如果需要多个连接,请使用Stringbuilder而不是字符串,从而节省堆内存。
  23. 存在任何无法访问的代码,如果可能,修改代码(如果存在)。

1 个答案:

答案 0 :(得分:1)

我将从定义“软件效率”开始。本文提供了一个提示:https://www.keenesystems.com/blog/defining-efficiency-as-a-software-requirement

根据ISO 25010:效率是“与用户实现目标的准确性和完整性相关的资源消耗”

然后,它可能是“性能效率”,意思是“相对于在指定条件下使用的资源量的性能”和诸如以下的标准

  • 时间行为
  • 资源利用
  • 容量

其他规范包括ISO / IEC 9126-1,ISO / IEC 25062和ISO 9241-11

来自https://en.wikipedia.org/wiki/ISO_9241#ISO_9241-11

“系统效率:为了评估系统效率,研究人员记录了参与者完成每个任务所花费的时间(以秒为单位)。”

也很有趣:which code is consuming less power?

最后:“生产率(也称为效率)是指 生产出一定数量资源的产品。对于软件,生产力通常是通过规模(ESLOC)除以工作时间来衡量的。“请参阅Department of Defense Software Factbook

总结一下。我认为您应该更新列表,并专注于您真正想要和需要衡量的内容,以及有关系统或软件的通用内容以及例如语言特定的效率标准。