很多人在寫程式的時候都只關心編譯後能不能執行好進行測試,因為程式一旦有錯誤就會中斷編譯程序、阻擋工作的進行,要等到錯誤的程式碼被修正後才能繼績工作。而警告的訊息並不會阻擋工作的進行,就會變得無所謂、大多數的人都會選擇不予理會。
這並不是一個好的開發習慣,Android Studio 進行的程式碼分析就像健康檢查一樣,用來顯示程式碼的健康程度。當健康檢查的報告中有數值超出標準值,雖然沒有達到疾病的程度,但代表身體已經有一定的問題。如果繼續忽視不理,很有可能就會轉變成疾病,甚至危害生命。同樣的道理,當分析的結果出現警告的訊息代表程式的寫法有一定程度出錯的風險。所以一但有警告訊息出現,應該要和錯誤一樣認真地看待並謹慎地處理。
不過,有的時候特定的習慣性寫法或是團隊的 Coding Convetion 可能會和工具預設的檢查規則有衝突,但其實並不影響程式的執行結果。舉例來說,以下的程式碼是一般判斷布林值的慣用寫法:
但是這樣的寫法由於「!」的寬度不寬,和判斷布林值是 true 的程式碼差別沒有很大,很容易因為看漏而導致修改程式時誤判而發生改錯程式碼的情況。再者,我們的母語並不是英文,所以在命名變數時常常詞不達意,甚至在過去的工作經驗中遇到有人命名變數的語意與用途相反的情況,更大大地增加了這樣的情況出現的機率。
為了有效地提高防呆的機率,我都習慣性地使用以下的方式來撰寫判斷式:
雖然會多打好幾個字,但是卻可以換來表達意思明確的效果。閱讀程式碼的人容易判斷區塊中到底是要執行 true 還是 false 的邏輯,不會被變數的名稱誤導形成邏輯上的混亂。
但 Android Studio 預設的檢查規則在遇到以上的寫法卻會顯示以下的警告訊息:
flag == false can be simplified to !flag
Reports pointless or pointlessly complicated boolean expressions. Such expressions include anding with true, oring with false, equality comparison with a boolean literal, or negation of a boolean literal. Such expressions may be the result of automated refactorings not completely followed through to completion, and in any case are unlikely to be what the developer intended to do.
Reports pointless or pointlessly complicated boolean expressions. Such expressions include anding with true, oring with false, equality comparison with a boolean literal, or negation of a boolean literal. Such expressions may be the result of automated refactorings not completely followed through to completion, and in any case are unlikely to be what the developer intended to do.
當這樣的訊息與其他的警告混在一起就會造成在寫程式時的困擾,因為沒有辦法在第一時間就利用編輯窗格右上角的綠色勾勾,了解程式是否已經通過其他的檢查規則。當看到窗格的右上角呈現黃色的方塊,如果要確認還要將滑鼠游標移到有問題的位置,才能夠判斷是不是因為以上的程式碼寫法所產生的警告訊息,或是還有其它的問題待修正!
這樣的動作如果在寫程式時頻繁的重複,會讓工作的效率下降很多,而且也會因為久了出現怠惰的心態,進而忽略警告訊息造成程式出問題的風險增加。
所幸 Android Studio 的檢查規則是可以調整的,在 Settings 的對話視窗中選擇「Editor->Inspections」項目,就可開啟或關閉特定的檢查規則。以文章中的示範程式碼寫法為例,可以切換到「Java->Control flow issues->Pointless boolean expression」項目。如下圖所示取消勾選該項目,Android Studio 就不會再針對這項寫法進行檢查。
有更動過的選項會以藍色的字體顯示,同時所屬的逐級向上分類項目也會顯示藍色,以方便識別。
0 意見:
張貼留言