現在的位置: 首頁 > 編程語言 > 正文

Java編碼易疏忽哪些問題

2020年06月08日 編程語言 ⁄ 共 4619字 ⁄ 字型大小 評論關閉

  在Java編碼中,我們容易犯一些錯誤,也容易疏忽一些問題。下面學步園小編來講解下Java編碼易疏忽哪些問題?

  Java編碼易疏忽哪些問題

  1.糾結的同名

  現象

  很多類的命名相同(例如:常見於異常、常量、日誌等類),導致在import時,有時候張冠李戴,這種錯誤有時候很隱蔽。因為往往同名的類功能也類似,所以IDE不會提示warn。

  解決

  寫完代碼時,掃視下import部分,看看有沒有不熟悉的。替換成正確導入後,要注意下注釋是否也作相應修改。

  啟示

  命名盡量避開重複名,特別要避開與JDK中的類重名,否則容易導入錯,同時存在大量重名類,在查找時,也需要更多的辨別時間。

  2.想當然的API

  現象

  有時候調用API時,會想當然的通過名字直接自信滿滿地調用,導致很驚訝的一些錯誤:

  示例一:flag是true?

  雙擊代碼全選

  1

  booleanflag=Boolean.getBoolean("true");

  可能老是false。

  示例二:這是去年的今天嗎(今年是2012年,不考慮閏年)?結果還是2012年:

  Calendarcalendar=GregorianCalendar.getInstance();

  calendar.roll(Calendar.DAY_OF_YEAR,-365);

  下面的才是去年:

  calendar.add(Calendar.DAY_OF_YEAR,-365);

  解決辦法

  問自己幾個問題,這個方法我很熟悉嗎?有沒有類似的API?區別是什麼?就示例一而言,需要區別的如下:

  Boolean.valueOf(b)VSBoolean.parseBoolean(b)VSBoolean.getBoolean(b);

  啟示

  名字起的更詳細點,注釋更清楚點,不要不經了解、測試就想當然的用一些API,如果時間有限,用自己最為熟悉的API。

  3.有時候溢出並不難

  現象

  有時候溢出並不難,雖然不常復現:

  示例一:

  longx=Integer.MAX_VALUE+1;

  System.out.println(x);

  x是多少?竟然是-2147483648,明明加上1之後還是long的範圍。類似的經常出現在時間計算:

  數字1×數字2×數字3…

  示例二:

  在檢查是否為正數的參數校驗中,為了避免重載,選用參數number,於是下面代碼結果小於0,也是因為溢出導致:

  Numberi=Long.MAX_VALUE;

  System.out.println(i.intValue()>0);

  解決

  讓第一個操作數是long型,例如加上L或者l(不建議小寫字母l,因為和數字1太相似了);

  不確定時,還是使用重載吧,即使用doubleValue(),當參數是BigDecimal參數時,也不能解決問題。

  啟示

  對數字運用要保持敏感:涉及數字計算就要考慮溢出;涉及除法就要考慮被除數是0;實在容納不下了可以考慮BigDecimal之類。

  4.日誌跑哪了?

  現象

  有時候覺得log都打了,怎麼找不到?

  示例一:沒有stacktrace!

  }catch(Exceptionex){

  log.error(ex);

  }

  示例二:找不到log!

  }catch(ConfigurationExceptione){

  e.printStackTrace();

  }

  解決

  替換成log.error(ex.getMessage(),ex);

  換成普通的log4j吧,而不是System.out。

  啟示

  API定義應該避免讓人犯錯,如果多加個重載的log.error(Exception)自然沒有錯誤發生

  在產品代碼中,使用的一些方法要考慮是否有效,使用e.printStackTrace()要想下終端(Console)在哪。

  5.遺忘的volatile

  現象

  在DCL模式中,總是忘記加一個Volatile。

  privatestaticCacheImplinstance;//losevolatile

  publicstaticCacheImplgetInstance(){

  if(instance==null){

  synchronized(CacheImpl.class){

  if(instance==null){

  instance=newCacheImpl();

  }

  }

  }

  returninstance;

  }

  解決

  毋庸置疑,加上一個吧,synchronized鎖的是一塊代碼(整個方法或某個代碼塊),保證的是這」塊「代碼的可見性及原子性,但是instance==null第一次判斷時不再範圍內的。所以可能讀出的是過期的null。

  啟示

  我們總是覺得某些低概率的事件很難發生,例如某個時間並發的可能性、某個異常拋出的可能性,所以不加控制,但是如果可以,還是按照前人的「最佳實踐」來寫代碼吧。至少不用過多解釋為啥另闢蹊徑。

  Java編碼易疏忽哪些問題

  6.不要影響彼此

  現象

  在釋放多個IO資源時,都會拋出IOException,於是可能為了省事如此寫:

  publicstaticvoidinputToOutput(InputStreamis,OutputStreamos,

  booleanisClose)throwsIOException{

  BufferedInputStreambis=newBufferedInputStream(is,1024);

  BufferedOutputStreambos=newBufferedOutputStream(os,1024);

  ….

  if(isClose){

  bos.close();

  bis.close();

  }

  }

  假設bos關閉失敗,bis還能關閉嗎?當然不能!

  解決辦法

  雖然拋出的是同一個異常,但是還是各自捕獲各的為好。否則第一個失敗,後一個面就沒有機會去釋放資源了。

  啟示

  代碼/模塊之間可能存在依賴,要充分識別對相互的依賴。

  7.用斷言取代參數校驗

  現象

  如題所提,作為防禦式編程常用的方式:斷言,寫在產品代碼中做參數校驗等。例如:

  privatevoidsend(ListeventList){

  asserteventList!=null;

  }

  解決

  換成正常的統一的參數校驗方法。因為斷言默認是關閉的,所以起不起作用完全在於配置,如果採用默認配置,經歷了eventList!=null結果還沒有起到作用,徒勞無功。

  啟示

  有的時候,代碼起不起作用,不僅在於用例,還在於配置,例如斷言是否啟用、log級別等,要結合真實環境做有用編碼。

  8.用戶認知負擔有時候很重

  現象

  先來比較三組例子,看看那些看著更順暢?

  示例一:

  publicvoidcaller(inta,Stringb,floatc,Stringd){

  methodOne(d,z,b);

  methodTwo(b,c,d);

  }

  publicvoidmethodOne(Stringd,floatz,Stringb)

  publicvoidmethodTwo(Stringb,floatc,Stringd)

  示例二:

  雙擊代碼全選

  1

  2

  3

  4

  publicbooleanremove(Stringkey,longtimeout){

  Futurefuture=memcachedClient.delete(key);

  publicbooleandelete(Stringkey,longtimeout){

  Futurefuture=memcachedClient.delete(key);

  示例三:

  雙擊代碼全選

  1

  2

  publicstaticStringgetDigest(StringfilePath,DigestAlgorithmalgorithm)

  publicstaticStringgetDigest(StringfilePath,DigestAlgorithmdigestAlgorithm)

  解決

  保持參數傳遞順序;

  remove變成了delete,顯得突兀了點,統一表達更好;

  保持表達,少縮寫也會看起來流暢點。

  啟示

  在編碼過程中,不管是參數的順序還是命名都盡量統一,這樣用戶的認知負擔會很少,不要要用戶容易犯錯或迷惑。例如用枚舉代替string從而不讓用戶迷惑到底傳什麼string,諸如此類。

  9.忽視日誌記錄時機、級別

  現象

  存在下面兩則示例:

  示例一:該不該記錄日誌?

  雙擊代碼全選

  1

  2

  3

  4

  5

  catch(SocketExceptione)

  {

  LOG.error("servererror",e);

  thrownewConnectionException(e.getMessage(),e);

  }

  示例二:記什麼級別日誌?

  在用戶登錄系統中,每次失敗登錄:

  雙擊代碼全選

  1

  LOG.warn("Failedtologinby"+username+");

  移除日誌記錄:在遇到需要re-throw的異常時,如果每個人都按照先記錄後throw的方式去處理,那麼對一個錯誤會記錄太多的日誌,所以不推薦如此做;但是如果re-throw出去的exception沒有帶完整的trace(即cause),那麼最好還是記錄下。

  如果惡意登錄,那系統內部會出現太多WARN,從而讓管理員誤以為是代碼錯誤。可以反饋用戶以錯誤,但是不要記錄用戶錯誤的行為,除非想達到控制的目的。

  啟示

  日誌改不改記?記成什麼級別?如何記?這些都是問題,一定要根據具體情況,需要考慮:

  是用戶行為錯誤還是代碼錯誤?

  記錄下來的日誌,能否能給別人在不造成過多的干擾前提下提供有用的信息以快速定位問題。

  10.忘設初始容量

  現象

  在JAVA中,我們常用Collection中的Map做Cache,但是我們經常會遺忘設置初始容量。

  雙擊代碼全選

  1

  cache=newLRULinkedHashMap(maxCapacity);

  解決

  初始容量的影響有多大?拿LinkedHashMap來說,初始容量如果不設置默認是16,超過16×LOAD_FACTOR,會resize(2*table.length),擴大2倍:採用Entry[]newTable=newEntry[newCapacity];transfer(newTable),即整個數組Copy,那麼對於一個需要做大容量CACHE來說,從16變成一個很大的數量,需要做多少次數組複製可想而知。如果初始容量就設置很大,自然會減少resize,不過可能會擔心,初始容量設置很大時,沒有Cache內容仍然會佔用過大體積。其實可以參考以下表格簡單計算下,初始時還沒有cache內容,每個對象僅僅是4位元組引用而已。

  以上就是關於「Java編碼易疏忽哪些問題」的內容,希望對大家有用。更多資訊請關注學步園。學步園,您學習IT技術的優質平台!

抱歉!評論已關閉.