`審核人:孫雨珩`
`被審核代碼負責人:謝洋`
`代碼地址:http://192.168.1.170/hyfe/demo-project/tree/master/demo-xieyang`
# 技偵全息指揮系統代碼審查
#### 1、組件拆分
頁面上使用的條形圖,地圖,甚至查詢條件都應該組件化,而不是放在同一個js里進行初始化。這種做法不利于組件的重復利用,以及項目的維護
解決方案:對頁面上使用到的元素組件化,在頁面入口只負責對各個組件初始化。查詢條件組件的條件發生變化時重新拉取數據,并把數據更新到組件中去
#### 2、地圖應使用d3實現,區域的id應使用地圖數據中的id而不是自己定義
使用ajax拉取地圖數據后,應用d3將地圖畫出來。現在這種方式可操作性太低,如果需要渲染其他地圖還要重新制作svg
解決方案:將地圖數據看成是組件外部數據進行設置,組件本身根據數據畫制svg。地圖數據可以到我做的項目里復制一份。
#### 3、查詢條件發生變化時,頁面上顯示的時間間隔也應發生變化
解決方案:將查詢條件組件化,并設置一個狀態屬性(本月,本周,今日),當狀態發生變化時展示的時間間隔也應發生變化
#### 4、進度條的數量應與數據數組的長度進行關聯
現在的設計下,如果數據數組的長度不足7條時會發生問題
解決方案:引入d3進行操作,進度條的數量根據數據數組長度的變化進行添加/刪除
#### 5、條形圖中柱狀體的數量應與數據數組的長度進行關聯
現在的設計下,如果數據數組的長度不足5條時會發生問題
解決方案:引入d3進行操作,柱狀體的數量根據數據數組長度的變化進行添加/刪除
#### 6、每次獲得數據后,地圖上的餅圖每一個都要重新創建dom
現在的設計下,會在渲染餅圖時會清空畫板上的內容,應對已有的餅圖進行復用
解決方案:引入d3進行操作,餅圖的數量根據數據數組的長度進行添加/刪除,然后再根據數據更新餅圖的位置以及顯示的數值
#### 7、右側警令發布,要情發布,每日值班未實現數據綁定
右側這三個面板的數據也應根據請求回來的數據重新渲染
解決方案:實現對應的組件,并暴露數據更新的接口。警令發布和要情發布功能類似,所以還應實現兩個具有數據更新功能的組件。
#### 8、 頁面上各個組件的模板直接寫在了html中
這會導致當需要復用組件時,還需要在html中粘貼一份組件模板,如果組件模板需要更改時,所有用到這個組件的html都要進行更改
解決方案:將初始化模板的邏輯放入組件中
- 1、執法項目
- 1.1 20170523-張紅橋
- 1.2 20170523-李偉
- 2、運維項目
- 2.1 20170527-歐陽德才
- 2.2 20170613-尹幫會&歐陽德才
- 2.3 20170706-歐陽德才
- 2.4 20170706-尹幫會
- 3、指揮大屏項目
- 3.1 20170712-歐陽德才
- 4、PC端項目
- 4.1 20170516-張紅橋
- 5、萬達項目
- 5.1 20170619-李偉&舒曼
- 5.2 20170623-李偉&舒曼
- 5.3 20170803-李偉(未修改)
- 5.4 20170811-李偉(未修改)
- 6、可視化組件
- 6.1 20170718-張紅橋
- 7、UI組件
- 7.1 20170522-舒曼
- 8、新員工練習項目
- 8.1 20170401-歐陽德才
- 8.2 20170401-舒曼
- 8.3 20170503-舒曼
- 8.4 20170609-劉黎
- 8.5 20170620-劉黎
- 8.6 20170703-謝洋
- 8.7 20170706-孫雨珩
- 8.8 20170706-謝洋
- 8.9 20170711-謝洋
- 8.10 20170724-謝洋
- 8.11 20170908-譚俊
- 8.12 20171026-廖爽
- 8.13 20171027-廖爽
- 8.14 20171027-廖爽
- 8.15 20171031-廖爽
- 8.16 20171103-覃英琪
- 8.17 20171103-01-廖爽
- 8.18 20171103-02-廖爽
- 8.19 20171110-趙家紅
- 8.20 20171110-覃英琪
- 8.21 20171110-02-趙家紅
- 8.22 20171115-01-廖爽
- 8.23 20171127-趙家紅
- 8.24 20171122-李祁
- 8.25 20180528-吳泓江
- 9、資源監控大屏
- 9.1 20170705-舒曼
- 9.2 20170711-舒曼
- 9.3 20170724-舒曼
- 10、CQ指揮艙
- 10.1 20170906-張紅橋&李偉&謝洋
- 11、昆明
- 11.1 20171009-李偉&舒曼
- 12、情報墻
- 12.1 20171025-謝洋
- 13、動態管控
- 14、迪愛斯區縣
- 14.1 20171128-歐陽德才
- 15、貴州塊數據指揮調度平臺
- 16、北京東城區可視化項目
- 16.1 20180425-歐陽德才&謝洋
- 16.2 20180510-謝洋