【问题标题】:Code Review Checklist代码审查清单
【发布时间】:2014-03-01 12:05:17
【问题描述】:

请提供一些参数来评估代码效率,到目前为止,我的代码清单中包含以下内容:

  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. 存在任何无法访问的代码,如果可能,请修改该代码(如果存在)。

【问题讨论】:

标签: asp.net


【解决方案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

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

【讨论】:

    猜你喜欢
    • 1970-01-01
    • 1970-01-01
    • 2011-02-09
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多