51Testing软件测试论坛

标题: 记一次review代码 [打印本页]

作者: 八戒你干嘛    时间: 2019-3-1 14:55
标题: 记一次review代码

一直觉得良好的编程习惯,代码层次很重要,但是总是自娱自乐。现在到了一家新公司,老大要求比较严,新写的代码让review,说实话内心还是有些小小的抵触的,毕竟我也工作三年了。结果review的时候,真是生无可恋啊。


给我review代码的其实也只工作三年,不过一直待在一号店。可见待在一家成熟点的公司还是蛮好的。


1、命名问题,命名一定要含义

    a)所有的List命名为 xxxList,不要直接用list命名,所有的数组命名 xxxArr

    b)除了静态常量,不要用下划线命名,用驼峰


2、判空,要用相应的工具类

    我之前循环list,先对对list判空,都是list != null。其实这样已经满足要求了,不会出现空指针的。因为如果list.size()为0的话,那么它是不会进入循环的


3、前期要确定好变量好实体类中成员变量的类型,否则后面改动可能比较大。特别是需要公用的类。


4、正则表达式需要定义在实工具类中。


5、代码层次要清晰,最好注释能有类似 1 2 3 4这样的注释


6、= && || 等运算符 加空格


7、对于比较简单的方法,unchecked这种注解可以加到方法上,例如dao层的方法,通常只有一句代码








欢迎光临 51Testing软件测试论坛 (http://bbs.51testing.com/) Powered by Discuz! X3.2