今天再次Review了代碼,但思路卻因此而開始混亂。
從名字上解這兩個對象:
XmlPullParser當然是以“拉”的方式從流中獲取信息。
SocketReader單從字面上理解,功能自然是從Socket上獲取字節流。
“單一職責原則”,在幾年前就在我腦子里打下了烙印。The Simpler The Easier,既是我做人的原則,也是我做程序的原則。常理上講,我應該盡力維護這個原則,讓上述兩個對象都盡可能的簡單。 在wildfire中,也有SocketReader,然而它的SocketReader卻不是那么簡單,功能遠遠超出了字面意義。大部分業務都要靠這個來控制,分配。
受它影響(之前我通讀了wildfire的所有源碼),在tim中也給SocketReader的子類ClientSocketReader等加上了重擔。因為它掌握了太多的信息,應該說大多數信息都暴露在這個地方,Session,Socket,SocketConnection,我實在找不出理由不讓它參與進業務。也許,這是OO的一種失敗,但我一時也找不到新對象來管理這一系列的相關信息。
XmlPullParser則和SocketReader息息相關,因為Socket中Read出來的東西首先就要經過Parser,才能從字符流形成有用的東西。
在原先的設計中,XmlPullParser被SocketReader所包含,并提供了get方法暴露給外界。在很多事件分配的地方,都要XmlPullParser提供信息,之前,都是通過SocketReader間接獲取xpp,高層真的需要直接使用xpp嗎?我覺得不然,高層需要的信息完全可以通過SocketReader來提供。
那么該怎么設計兩者的關系呢?是包含,還是父子?我傾向于包含,但懶惰促使我選擇了父子。目前看起來,父子關系并沒有帶來什么壞的影響,如果有必要,今后再重構吧。
現在結構似乎更為清晰了,SocketReader的子類(ClientSocketReader。。。)會負責解析流,并根據解析出的內容進行第一層處理,如選擇下級處理器或者是直接計算業務,或者是進行轉發。。。根據不同的子類,表現不同的業務族(Client,Server,Agent。。。)