51Testing软件测试论坛

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

QQ登录

只需一步,快速开始

微信登录,快人一步

手机号码,快捷登录

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

[原创] 记一次review代码

[复制链接]
  • TA的每日心情
    无聊
    4 天前
  • 签到天数: 11 天

    连续签到: 2 天

    [LV.3]测试连长

    跳转到指定楼层
    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-9-23 22:30 , Processed in 0.065860 second(s), 23 queries .

    Powered by Discuz! X3.2

    © 2001-2024 Comsenz Inc.

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