51Testing软件测试论坛

 找回密码
 (注-册)加入51Testing

QQ登录

只需一步,快速开始

微信登录,快人一步

查看: 1531|回复: 0
打印 上一主题 下一主题

[原创] 记一次review代码

[复制链接]

该用户从未签到

跳转到指定楼层
1#
发表于 2019-3-1 14:55:53 | 只看该作者 回帖奖励 |倒序浏览 |阅读模式

一直觉得良好的编程习惯,代码层次很重要,但是总是自娱自乐。现在到了一家新公司,老大要求比较严,新写的代码让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层的方法,通常只有一句代码



分享到:  QQ好友和群QQ好友和群 QQ空间QQ空间 腾讯微博腾讯微博 腾讯朋友腾讯朋友
收藏收藏
回复

使用道具 举报

本版积分规则

关闭

站长推荐上一条 /1 下一条

小黑屋|手机版|Archiver|51Testing软件测试网 ( 沪ICP备05003035号 关于我们

GMT+8, 2024-6-2 17:27 , Processed in 0.062379 second(s), 23 queries .

Powered by Discuz! X3.2

© 2001-2024 Comsenz Inc.

快速回复 返回顶部 返回列表