軟件測試技術交流羣
: 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之類的,可以使用“==比較值”