代碼質(zhì)量,一個(gè)可大可小的問(wèn)題。

一 ?規(guī)范
(1)命名
?變量:mNumberOfPlayer(number_of_player)
常量:INTEM_IMAGE(intem_image,intemImage)
(2)方法?
命名:采用小駝峰式
注意:
1、嚴(yán)格要求自己寫體量小的方法,一個(gè)方法在20行左右(getter、setter一行解決),方法過(guò)大一定要分拆,比如如多 if-else里面的內(nèi)容完全可以分別拆出2個(gè)單獨(dú)方法?
2、避免空方法(Methods should not be empty)
如果有特殊原因?qū)懥藘?nèi)容為空的方法,一定要有詳細(xì)的注釋說(shuō)明原因。
3、表達(dá)式不應(yīng)該太復(fù)雜(Expressions should not be too complex,maximum allowed 3)
?if(edge == DragEdge.Right && childRight <= getWidth()&& childBottom <= getHeight()&& ? ? ? ? ? ? ? ?childTop >= getPaddingTop()&& childLeft >= getPaddingLeft())
?4、字符串字面值不應(yīng)重復(fù)(String literals should not be duplicated)
HashMapmap =newHashMap();
map.put("ItemImage", R.drawable.wb_icon_myproject);
map.put("ItemImage", R.drawable.wb_icon_myproject2);
修改為:
constant String INTEM_IMAGE="ItemImage";
?map.put(INTEM_IMAGE, R.drawable.wb_icon_myproject2);
二 ?規(guī)避Bug
(1)java.lang.NullPointerException
空指針最為常見,也最容易規(guī)避,使用的時(shí)候一定要進(jìn)行null check,采取不信任原則:
方法形參要判空后才使用;
全局變量容易被系統(tǒng)回收或者更改,使用全局變量前建議判空;
第三方接口的調(diào)用,對(duì)返回值進(jìn)行判空。
(2)java.util.ConcurrentModificationException
異常示例:java.util.ArrayList$ArrayListIterator.next(ArrayList.java:573)
異常代碼示例:
mTimer.schedule(newTimerTask() {
? ? ? ? public voidrun() {
? ? ? ? ? ? ? ? ? initEleLocationData();
? ? ? ? ? }},0,DELAY_TIME_InitData);
mTimer.schedule(newTimerTask() {
?public void ?run() {
? ?if(mEleInfoList!=null) {
? ? ? ? ? ?for(HashMap mInfoMap :mEleInfoList) {
? ? ? ? ? ? ? ? ? ? ? calculateTime(mInfoMap);}
? }}},0,DELAY_TIME);
[原因分析]:該異常表示迭代器迭代過(guò)程中,迭代的對(duì)象發(fā)生了改變,如數(shù)據(jù)項(xiàng)增加或刪除。
由于迭代對(duì)象不是線程安全,在迭代的過(guò)程中,會(huì)檢查modCount是否和初始modCount即expectedModCount一致,如果不一致,則認(rèn)為數(shù)據(jù)有變化,迭代終止并拋出異常。常出現(xiàn)的場(chǎng)景是,兩個(gè)線程同時(shí)對(duì)集合進(jìn)行操作,線程1對(duì)集合進(jìn)行遍歷,而線程2對(duì)集合進(jìn)行增加、刪除操作,此時(shí)將會(huì)發(fā)生ConcurrentModificationException異常。
具體解決方法:多線程訪問(wèn)時(shí)要增加同步鎖,或者建議使用線程安全的集合:
1. 使用ConcurrentHashMap替換HashMap,CopyOnWriteArrayList替換ArrayList;
2. 或者使用使用Vector替換ArrayList,Vector是線程安全的。Vector的缺點(diǎn):大量數(shù)據(jù)操作時(shí),由于線程安全,性能比ArrayList低.
(3)java.lang.IllegalArgumentException: You must pass in a non null
(4)Switch ??"switch" statements should end with "default" clauses