Code Review中应该关注的点

  1. Magic number/string
  1. If statement, you should always use single line or brackets
  1. Provide default value of Enum
  1. Difference between abstract class and interface
  1. Excess inheritance--the inheritance hierarchy is too deep
  1. Use string.Concat, string.Format, StringBuilder instead of +
  1. Const vs Readonly
  2. Break down large method into small methods
  1. Using nest class to hidden information is not necessary and hurts maintainability
  1. Do not use new keyword to hidden base class‘s method, please use polymorphism
  2. Static class may substitute singleton pattern for simplification
  3. Private members/property access are not necessary if not protecting; use auto properties instead
  4. Use IDisposable correctly to prevent resource leak for unmanaged resources.
  1. Tell browser the encoding you are using with meta tag
  2. Make sure browsers know you want to use edge rendering if you are building HTML5 websites
  3. Eliminating embedded javascript and style sheets to help improve caching of your code
  4. Use CDN so that browser caching can improve startup time for new users.
  5. Reduced round-trips with fewer CSS files means you are less likely to run into browser limits of CSS
  6. The less code inside a Razor file the better
  7. Creating cached objects inside per-request classes is not necessary and adds complexity
  8. Routing tables are still a valid way of doing routing for much your code
时间: 2024-11-15 22:48:57

Code Review中应该关注的点的相关文章

什么是Code Review(转)

Code Review是一种通过复查代码提高代码质量的过程,在XP方法中占有极为重要的地位,也已经成为软件工程中一个不可缺少的环节.本文通过对Code Review的一些概念和经验的探讨,就如何进行Code Review和Code Review中应该注意什么提出一些建议. 本文中涉及的问题大部分针对JAVA类代码.同时本文不涉及Code Review过程和组织. 关键词: Code Review,JAVA, XP ,代码质量 ,软件工程 一.Code Review简介1 Code Review的

Code Review最佳实践

Code Review最佳实践 原文链接 : Code Review Best Practices 原文作者 : Kevin London 译文出自 : 开发技术前线 www.devtf.cn 译者 : ayyb1988 校对者: chaossss 状态 : 完成 在Wiredrive上,我们做了很多的Code Review.在此之前我从来没有做过,这对于我来说是一个全新的体验,下面来总结一下在Code Review中做的事情以及说说Code Review的最好方式. 简单的说,Code Rev

Code Review(转)

Code Review是一种通过复查代码提高代码质量的过程,在XP方法中占有极为重要的地位,也已经成为软件工程中一个不可缺少的环节.本文通过对Code Review的一些概念和经验的探讨,就如何进行Code Review和Code Review中应该注意什么提出一些建议. 本文中涉及的问题大部分针对JAVA类代码.同时本文不涉及Code Review过程和组织. 关键词: Code Review,JAVA, XP ,代码质量 ,软件工程 一.Code Review简介1 Code Review的

什么是Code Review

Code Review是一种通过复查代码提高代码质量的过程,在XP方法中占有极为重要的地位,也已经成为软件工程中一个不可缺少的环节.本文通过对Code Review的一些概念和经验的探讨,就如何进行Code Review和Code Review中应该注意什么提出一些建议. 本文中涉及的问题大部分针对JAVA类代码.同时本文不涉及Code Review过程和组织. 关键词: Code Review,JAVA, XP ,代码质量 ,软件工程 一.Code Review简介 1 Code Review

如何在python脚本开发做code review

在软件项目开发中,我们经常提到一个词"code review".code review中文翻译过来就是代码评审或复查,简而言之就是编码完成后由其他人通过阅读代码来检查代码的质量(可编译.可运行.可读.可维护.可复用),这些性质都比较抽象,但是一般都可以通过以下的检查点来实现: 检查代码的命名方式是否符合规范,代码的可读和可维护必须要求所有参与编码的同事使用的命名有统一的规范(注意每个人有自己的代码风格,但是要符合可读性的代码规范): 检查代码的注释,注释一般包括:1.类要有类用途和使用

[读后感]从Code Review 谈如何做技术

还有9个电,争取把这篇发出去,里面有太同共鸣,只不过之前没能写出来, 一是文笔有限,总结不够明确,本文至少总结出了我想总结的6个观点,看来总结能力还是要提高: 二是不确认这是对的,所以不敢贸然写出来,看来奔四的程序员都有这些共同的想法,并非我一人,还有许多人... 着实说,代码审查,以前想过,但没做过: 代码审查确实很不错,不懂开发的测试人员其实从某种角度是用于粗暴地替代代码审查, 结果可知,花在修复 Bug 上的时间要比编码时间多 N 倍, 我想我们以敏捷方式来对付它,逐层皮儿地扒着做,做完一

Code Review的一些典型内容

如下是Code Review中一些典型的内容: 一.常规项: 1.代码能够工作么?它有没有实现预期的功能,逻辑是否正确等. 2.所有的代码是否简单易懂? 3.代码符合你所遵循的编程规范么?这通常包括大括号的位置,变量名和函数名,行的长度,缩进,格式和注释. 4.是否存在多余的或者重复的代码? 5.代码是否尽可能的模块化了? 6.是否有可以被替换的全局变量? 7.是否有被注释掉的代码? 8.循环是否设置了长度和正确的终止条件? 9.是否有可以被库函数替代的代码? 10.是否有可以删除的日志或调试代

Code Review代码审查的思路

转来的:http://www.blogjava.net/qileilove/archive/2012/07/16/383199.html 1.关于Code Review 1.1 Code Review的目的 Code Review主要用来在软件工程过程中改进代码质量,通过Code Review可以达到如下目的目的: (1)在项目早期就能够发现代码中的BUG (2)帮助初级开发人员学习高级开发人员的经验,达到知识共享 (3)避免开发人员犯一些很常见,很普通的错误 (4)保证项目组人员的良好沟通 (

我是如何进行code review的

众所周知,代码审查是软件开发过程中十分重要的环节,楼主结合自己的实际工作经验,和大家分享一下在实际工作中代码审查是如何开展的, 笔者水平有限,若有错误和纰漏,还请大家指正. 代码审查的阻力 我想不通公司不同部门对代码审查这项工作的重视程度还是不一样的,对于代码审查的阻力总结了以下几点: 国内的整体环境,国内的公司,尤其是互联网公司,讲究速度致上,软件开发的迭代周期周期短,速度快,因为竞争太大,开发的产品要求快速上线,对代码审查不是很重视,先上线,出了问题再解决. 公司的规模,大公司重视流程,把代