有些新手寫程序,當須要定義某個變量名(也多是函數名、類名、包名等)時,隨意地一敲鍵盤,名字就起好了......若干星期後,碰到某 bug,再來看本身寫的代碼時,心中暗自嘀咕:「這代碼是我寫的嗎?咋都看不懂捏?」
因此我常跟新來的菜鳥說,命名不規範害死人啊!鑑於該問題至關廣泛,我整理了幾種典型的做爲反面教材,具體以下:使用單字母命名變量;使用一些沒太大意義的變量名(例如 s一、s二、s3);對同一個業務概念使用不一樣的術語/縮寫(容易讓讀代碼的人神經分裂);使用拼音命名(若是你團隊中有港臺人士或者老外,就慘了)。
html
這是一個很廣泛的問題。不少新手寫代碼的時候,若是發現要寫的某個函數和前幾天寫的某個函數差很少,就把原來的那個函數貼過來,而後稍微改幾下,心中還暗喜:「又快速搞定了一個功能」......
同窗,若是你也喜歡這麼幹,可要注意了。這種作法是代碼臭味(借用《重構 - 改善既有代碼的設計》的提法)的主要來源,致使代碼可維護性大大降低。當你未來須要增長功能或修改 bug 的時候,要同時改動多個地方,而那時你估計已經想不起來這砣代碼有幾個克隆了。java
若是你沒有據說過「Magic Number」,先看「這裏」瞭解一下。
爲了說明Magic Number的問題,咱找個例子來講事兒:假設有個業務邏輯中須要進行10秒的超時等待,你會怎麼寫這個sleep語句?我估計大部分人不外乎下面三種寫法:
一、直接寫上 sleep(10*1000);
了事
二、定義一個常量 TIMEOUT_XXX = 10*1000;
而後 sleep(TIMEOUT_XXX);
三、在配製文件中加入一個超時項,而後程序讀取配製文件得到超時值,而後調用 sleep。(此處提到的「配置文件」是廣義的,泛指各類可用於存儲配置信息的機制,如:xml 文件、ini 文件、數據庫 ...)
若是你的作法相似於寫法1,你多半喜歡隨手硬編碼。硬編碼不光缺少可讀性,並且具備和「代碼拷貝粘貼」相似的代碼臭味(可能會存在多個 Magic Number 克隆),不利於往後維護。
至於寫法2,比寫法1稍好(至少可讀性好了)。可是,未來一旦發生需求變動,要求在【運行時】調整超時間隔(甚至要求讓用戶來配製超時間隔),則寫法2的缺點立馬暴露無遺。程序員
每當說到 MVC 或者設計模式,幾乎每一個 Java 開發人員都能說得頭頭是道?可是說歸說,真正寫代碼的時候,鮮有人寫出的代碼是層次清楚的。至於說到代碼耦合分別由哪些狀況引發?什麼是正交的設計?(關於耦合與正交設計,我後面會專門討論一下)能徹底搞明白的人就更少了。
因此不少 Java 新手的代碼耦合度大也就不足爲奇了。我曾經抽查過試用期員工的代碼,各類業務邏輯糾纏在一塊兒,代碼臭味都要薰死人。想重構都無從下手,只好讓他推倒重寫。面試
因爲 Java 在語言層面提供了內存的垃圾回收機制,程序員只管申請內存,不須要再關心釋放的問題。所以不少新手養成了壞習慣,對於其它資源(好比數據庫鏈接)也只申請不釋放(有些人甚至天真地覺得 JVM 會幫你搞定資源回收)。
還有些人雖然知道資源須要釋放,可是經常忘記(好比寫了打開數據庫鏈接和相關代碼,【即將】寫關閉數據庫鏈接時,忽然有人叫你去吃中飯,回來後就把這茬給忘了)。
這個壞習慣會致使資源的泄露,而資源泄露每每比內存泄露更要命。若是你寫的程序是長時間運行的(好比運行在 Web Server 上),時間長了會因爲資源耗盡而致使整個進程出問題。算法
犯這種錯誤的人比較少,通常發生在剛學會 Java 或者剛參加工做不久的人身上。
所謂「空 catch 語句塊」就是在 catch 語句塊中沒有對異常做任何處理(好比記錯誤日誌),致使異常信息被丟棄/忽略。一旦程序不能正確運行,因爲查不到任何 log 信息,只好從頭看代碼,靠肉眼找 bug。數據庫
不少人在 catch 語句以後不使用 finally 語句。因爲在 try 語句中可能會涉及資源的申請和釋放。若是在資源申請以後、資源釋放以前拋出異常,就會發生資源泄露。
(資源泄露的嚴重性,上面已經聊過了)編程
有些人爲了省事,只在本身模塊的最外層代碼包一個 try 語句塊,而後 catch(Exception)。無論捕獲到什麼異常,都做統一 log 了事。這種作法比「空 catch 語句塊」稍好,但因爲不能對具體的異常進行具體處理,對一些可恢復的異常(下面會提到),喪失了恢復的機會。並且也可能致使上述提到的資源泄露的問題。小程序
有些人放着 Java 的異常機制不用,而用函數返回值來表示成功/失敗(好比:返回 true 表示成功、false 表示失敗),簡直是「捧着金碗要飯」。我的感受,從 C 轉到 Java 的人比較容易有此毛病。這種作法會致使以下幾個問題:
1. 返回值通常用整數值或布爾值表示,傳遞的信息過於簡陋;
2. 一旦調用者忽略了錯誤返回碼,就會致使和「空 catch 語句塊」相似的問題;
2. 對同一個函數的多處調用,都須要對返回值進行重複判斷,致使代碼冗餘(代碼冗餘的壞處,上面也已經聊過了)。設計模式
這個現象比較廣泛,俺發現不少2年以上 Java 工做經驗的人還沒有徹底搞明白二者的區別。看來這個問題得詳細說一下。
當初Java的設計者有意區分這兩種異常,是別有深意的。其中「Checked Exception」用於表示可恢復的異常(也就是你必須檢查的異常);而「Runtime Exception」表示不可恢復的異常(也就是運行時異常,主要是程序 bug 和致命錯誤,你【不須要】檢查)。不過這種作法引來了不少爭議(包括不少 Java 大牛),鑑於本帖子主要針對新手,之後再專門來聊這個爭議的話題。
爲了便於理解,下面我舉一個例子來講明。假設你要寫一個 Download 函數,根據傳入的 URL(String 參數)返回對應網頁的內容文本。這時候有兩種狀況你須要處理:
1. 若是傳入的 URL 參數是 null,這代表該函數的調用者出 bug 了,而程序自己的 bug 是很難在運行時自我恢復的。這時候 Download 函數必須拋出 Runtime Exception。而且 Download 函數的調用者【不該該】嘗試去處理這個異常,必須讓它【儘早】暴露出來(好比讓 JVM 本身終止運行)。
2. 若是傳入的 URL 參數非 null,可是它包含的字符串不是一個合法的URL格式(可能因爲用戶輸入錯誤致使)。這時候 Download 函數必須拋出 Checked Exception。而且 Download 函數的調用者必須捕獲該異常並進行相應的處理(好比提示用戶從新輸入 URL)。數組
不少新手不理解Java的基本類型和引用類型在本質上有什麼區別。請看以下的問題:
◇這兩種類型在內存存儲上有什麼區別?
◇這兩種類型在性能上有什麼區別?
◇這兩種類型對於 GC 有什麼區別?
關於前兩個問題,請看以前的帖子「Java性能優化[1]:基本類型 vs 引用類型」。
不少新手不理解 GC 的實現機制。請看以下的問題:
◇GC 是如何判斷哪些對象已經失效?
◇GC 對性能會有哪些影響?
◇如何經過 JVM 的參數調優 GC 的性能?
關於 GC 的問題,能夠參見以前的帖子「Java性能優化[3]:關於垃圾回收(GC)」。
對於 Java 提供的 String 和 StringBuilder,想必不少人都知道:String 用於常量字符串,StringBuilder 用於可變字符串。那 Java 當初爲何要這樣設計捏?爲啥不用一個類來統一搞定捏?
從 JDK 1.5開始,Java 引入了一個重量級的語法:範型。不過捏,不少新手僅僅知道範型的皮毛,而對於不少本質的東東,不甚瞭解。
◇GP 是在編譯時實現的仍是在運行時實現的?爲何要這麼實現?
◇GP 的類型擦除機制是咋回事?有啥優勢/缺點?
◇使用範型容器(相對於傳統容器)在性能上有啥影響?爲何?
另外,多線程也是大部分 Java 新手的短板。因此俺最後再來提幾個關於多線程的問題。◇synchronized 關鍵字是怎麼起做用滴?◇synchronized 的顆粒度(或者說做用域)如何?是針對某個類仍是針對某個類對象實例?◇synchronized 對性能有沒有影響?爲何?◇volatile 關鍵字又是派啥用滴?啥時候須要用這個關鍵字捏?