软件测试技术交流群 : 429183023
(由于刚入职新公司才写的这篇文章,会将工作中遇到的陆续更新)
更新于2017/3/8
我觉得作为一个Tester | QA 不仅仅是保证业务逻辑上的正确,其实也应该保证开发人员(RD)所写代码的质量。我意识到这一点,是之前公司一个Qunar出来的同事每次都会做代码的Review,主要是检查代码的业务逻辑。当然,我也就每次都会Review代码,后来也会发现很多开发所写的代码不仅仅是业务逻辑上容易查出问题,其实命名规范、单词拼写错、push代码前不格式化、多处重复代码未重构等等问题,但是这些问题,目前是RD团队里面是没有人来检查的。那么,我觉得作为一个QA,也作为第三人完全是有必要站出来检查代码,保证代码质量的。确保主分支代码的优秀,可以给后来的新入职员工带来极大的方便,也方便以后的代码的扩展,同时也能够提高自身的代码能力,对自身职业发展起到帮助。
一、格式问题
1、命名规范:例如
String finishedcount = "Test";
这个虽然不影响逻辑,但是造成此变量不易阅读,应该改为finishedCount。
2、删除未使用变量或者引用
为了代码整体的简洁明了,应该删除无用变量和引用。
3、单词拼写错误或者变量命名不够直接
1) 单词拼写错误绝对不能忍,必须让开发改
2) 变量命名尽量能够体现变量名的含义(不要怕单词长,能体现含义就行,用一个除了你谁都看不出来什么含义的单词缩写,简直是要回去上C编程了。)
例如:
ReceiveOder re = new ReceiveOder();
应收订单,你说你命名为re,在其他引用的地方,天晓得re是应收订单啊
二、逻辑问题
三、代码优化
1、引用数据类型的值比较
昨日跟进一个新需求,发现开发在比较两个Long类型数据值的时候,使用“==”,马上直接截图甩给开发。
关于引用类型和java基本数据类型的比较:
引用类型的值比较,建议使用equals()函数(因为针对对象来说“==”是比较引用地址是否一致,也就是比较两个数据是否指向同一个地址),基本数据类型int/char/double之类的,可以使用“==比较值”