最近在項目中接到一個任務,我負責后臺開發,另一開發人員負責前臺開發。任務非常簡單,請看下面的類圖。

A和B有一個單向的關聯關系,現在要為A增加一個屬性boolean resident,該屬性值有如下簡單的業務邏輯決定(偽代碼):
if(a.x == a.b.x)
resident = true;
else
resident = false;
前臺的查看頁面需要顯示這個值,當用戶修改a.x或者a.b.x時,要觸發一個事件,實時的在前臺更新resident,后臺需要以這個resident來
做某些業務邏輯的判斷。
我們知道,由于resident完全可以由a.x與a.b.x決定,屬于一個冗余數據,不需要顯式的增加該屬性。很快,前臺的那位開發人員就輕松的
告訴我,這個任務已經寫完了,原來他是在頁面寫了一個JS腳本,當a.x或a.b.x發生改變時,觸發一個onChange()事件,里面就實現了上述
的邏輯。我一看,感覺問題來了,然后我告訴他,我已經在后臺的服務提供了一個方法,用于判斷resident的值,他應該在事件觸發的時候
,通過異步方式調用后臺方法來獲取reisdent的值。他一聽覺得非常奇怪,這么簡單的邏輯,為什么需要使用異步方式去調用后臺這么麻煩
,于是我就從設計的角度跟他解釋,resident由 a.x與a.b.x決定,這是一個業務邏輯,從職責劃分的設計原則分析,前臺只負責顯示邏輯,
業務邏輯屬于后臺的職責。接著,他又提出了疑問,說這里其實只有一行代碼,沒有必要分得那么清,這樣調用不但麻煩,而且性能相對較
低。
這個例子,反映出了很多開發人員的通病,沒有真正從事物的本質考慮問題,只從代碼的數量去考慮。
上述的做法存在兩個問題:
1.開發人員沒有從職責劃分的角度考慮問題,而只是貪圖一時的方便,或者覺得簡單的一兩行代碼不需要進行設計,但實際上,上面的問題
必然導致重復代碼。上面的例子,即使只有一行代碼,但在系統的前后臺卻出現了兩份相同的邏輯,假設以后某天業務邏輯發生了變化,那
么,必須在兩個地方(甚至更多的地方)進行修改、測試,如果漏掉了一個地方,就會導致Bug的出現。作為一個開發人員,如果不注意這些
細節,隨意的拷貝和重復代碼,長期下來,一個系統就會遍布無數的重復代碼,系統越往后期就越難維護,最終陷于崩潰。
2.如果作為后臺開發人員,看到前臺已經實現了這個邏輯,而在前臺往后臺傳遞數據的時候,把resident也傳遞進來,從而認為后臺就不需
要重復這段邏輯,直接拿著前臺傳過來的這個resident來進行業務判斷,那么就可能會給系統帶來致命的漏洞,因為數據在傳輸過程中,很
可能被有意或無意的修改,不在后臺進行業務規則校驗這個錯誤是常見的,但也是致命的。
有一個原則開發人員特別容易犯,也一定要切記:
在Web應用中,相信客戶端提交的數據是正確的,不在業務層進行校驗,這是致命的錯誤。
筆者會在下一篇博文中,詳細說明某國內著名的第三方支付平臺,因為犯了這個低級的錯誤,而導致系統出現致命的漏洞,敬請關注。