讀Clean Code有感：例外處理程式重構篇

May 6 22:24~23:16

 

前一陣子在讀Clean Code這本書，讀著、讀著突然想到一件事，幾年前Teddy還在唸書的時候，曾經提出一個在Java/C#語言中使用try statement的用法，就是：

One method, one try statement. （一個method裡面只能有一個try敘述）

這個想法是Teddy從Eiffel語言的例外處理做法得到的靈感，Eiffel的例外處理是透過rescue、retry來達成，細節就不管它了，總之一個method裡面只能有一個rescue敘述（可以想成Java的catch(Throwable e)）。

***

當年Teddy只是直覺認為在Java裡面如果也可以遵守一個method裡面只有一個try statement，應該可以讓method正常邏輯與例外處理邏輯變得更簡單且易懂。有一點像是single responsibility principle一樣的道理，一個method如果需要一個以上的try statement，可能隱含這個method負擔太多的責任。但是當時Teddy並沒有特別花時間去把這個想法做更進一步的整理。

最近讀了Clean Code之後，突然想到把之前寫的例外處理範例程式拿出來整理一下。請看最原始的版本，這個method並沒有做例外處理，而是把IOException直接往外丟。

***

經過分析之後，發現這個method做了兩件事，這兩件事都可能發生例外。首先，這個method從一個DataInputStream讀出一個整數，這個整數代表某個資料封包的長度。第52行的程式aIS.ReadInt()可能發生IOException，如果發生例外，則表示讀取資料長度錯誤，因此丟出一個InvalidPacketException例外，並且註明「Read data length error.」。

 

第二件事就是連續讀取N個byte，把資料封包的內容讀出來。程式61行可能會發生兩個例外，分別是EOFException與IOException。前者表示資料長度不夠，後者表示讀取資料內容發生錯誤。所以這個method一共有三個狀況會發生例外，雖然對外都是丟出InvalidPacketException，但是錯誤的原因各不相同。

***

套用One method, one try statement這個設計原則，以及簡單的extract method這個refactoring，再把上面這個程式改成：

 

不知鄉民們有沒有覺得改過的程式不但變得比較易懂，對於例外處理的邏輯也清楚很多。

***

友藏內心獨白：有人說，例外處理的文章都看不懂啊…Orz。