2012/11/03

[FEEL] 重構



終於看不下去 開始動手幫之前的code重構了

一個 for 迴圈組字串之後送 xml 給廠商的系統 原本的程式竟然可以寫成跑完一次迴圈就送一次

明明對方一次不只能接收一組訊息 (還能同時在同一次 XML 中 IO 都上 XD)

結果造成對方在看 log 幫我們找問題的時候 log 大到滑鼠要拼命滾 ....

還有明明只是要要個 DOMNode

可以寫成一個 function 之後 每次要一個 node 就要先抓回整個 tree

讓我自己在寫的時候根本不想去 call 那個 function


前天跟昨天處理掉另外一段整合的優化

原本的作法是我們要撈對方某個節點的時候去call一個function去撈

但那個 func 裡面有一個廠商提供的指令是去撈回他們整個 domTree

然後就這樣要一個節點,撈回整個 Dom

整個流程檢查了幾10個地方 也就重複撈幾10次同樣的東西

重點是這個撈回來就可以當變數存下來重複利用了

導致對方的 api log 一個流程下來我們對他們 IO接近100多次 (單一步驟20~30不等)

兩個多月以前我發現問題的時候原本以為整個翻過要大約一週以上

但沒想到一個下午就把介接的主要程式部分改完了 剩下部分有用到整合的功能區還沒修

兩天下來處理過後 現在整個流程走完只要 call 12 次而已

其中只有三次是拉資料 其他都是修改資料的 api

整個速度提昇大概有 4x 左右



--

寫給自己爽而已 做這個也只想證明自己也沒這麼糟糕而已

--
※ 發信站: 批踢踢兔(ptt2.cc)
◆ From: 1.160.25.240

沒有留言:

張貼留言