`審查人:白占寧`
`被審核代碼負責人:舒曼`
`代碼地址:https://192.168.1.240:8443/svn/repos/source-analysis`
### 存在的問題
#### 1、使用傳參數的形式代替全局變量(已修改)
```
/**
* 公用變量
*/
var comChoiceName = '' //傳過來的名稱
var comStartTime = 0 //傳過來的開始時間
var comEndTime = 0 //傳過來的結束時間
var comElement = '' //要素名稱
var comOption = 0 //選擇庫
var comPagSize = 10 //每頁展示數量
var comCurrentNum = 1 //表的當前頁數
dosageTotal: function() {
var self = this
request.sendAjax(baseConfig.dosageTotal+'/'+comStartTime+'/'+comEndTime+'/'+0+'/'+1, function(data) {
var rankData = [data.cjlTop, data.sylTop]
drawing.rankingDraw(rankData)
self.scrollUp('scrollUp','66',60,0)
})
}
```
解決方案:盡量減少全局變量的定義,各個方法需要的數據通過參數傳遞進去。
#### 2、不符合基本的代碼規范(已修改)
```
if(r!=null)return decodeURIComponent(r[2])
```
解決方案:參考《JavaScript編碼規范》。
#### 3、注釋不完整(已修改)
```
/**
* [獲取傳參]
*/
parameters: function(name){
var search = window.location.search
var reg = new RegExp("(^|&)"+ name +"=([^&]*)(&|$)");
var r = search.substr(1).match(reg);
if(r!=null)return decodeURIComponent(r[2])
return null
}
```
解決方案:完整的注釋應該包括方法描述、參數說明和返回值說明。
#### 4、使用傳遞的參數前,先判斷類型是否符合預期(未修改)
```
/**
* [設置列表]
*/
setCustom: function(data, setStatus){
var setListHtml = Handlebars.compile(setListTpl)
var allData = []//全部值
$('.pop-page-main ul').empty()
data.forEach(function(d, i){
// ...
```
解決方案:如果參數data是`undefined`或`null`或`string`類型,則程序就會報錯,應該先判斷參數類型是否符合預期,然后再使用參數值。
#### 5、對空字符串的判斷方式(已修改)
```
if($(this).val() == '')
```
解決方案:`if(!$(this).val())`
#### 6、第三方類庫的位置(已修改)
`jquery.shapeshift.js`屬于第三方類庫,應該放到`sea-modules`目錄下面。
- 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-謝洋