`審核人：白占寧` `被審核代碼負責人：尹幫會 & 歐陽德才` `代碼地址：https://192.168.1.240:8443/svn/repos/yunwei/src` ### 存在的問題 #### 1、緩存多次使用到的變量 ``` dealData: function(data) { faultProfile.init(data.result.faultProfile) faultType.init(data.result.faultType) newFault.init(data.result.newFault) detachFault.init(data.result.detachFault) } ``` 解決方案：緩存`data.result`，如下所示： ``` dealData: function(data) { var result = data.result faultProfile.init(result.faultProfile) ... } ``` #### 2、避免定義不必要的變量 ``` // 接口參數 var startTime = "" var endTime = "" // 接口參數 var param = { startTime: startTime, endTime: endTime } ``` 解決方案：刪除startTime和endTime。 ``` // 接口參數 var param = { startTime: '', endTime: '' } ``` #### 3、使用var定義變量 ``` // 渲染數據 tpl = require('../../components/troubleTable/newFault.tpl') template = Handlebars.compile(tpl); ``` 解決方案：不使用var，直接定義變量，將導致該變量成為window的一個屬性，即window.tpl和tpl是同一個變量，容易造成全局污染。 #### 4、JS中建議使用單引號而不是雙引號 ``` $(".frontEndPercent").html('50%') $(".systemPercent").html('50%') ``` #### 5、注釋與內容不符 ``` // 前端分頁 var pageArray = [] ``` 解決方案：注釋內容應明確其功能及作用。 #### 6、初始加載的代碼放到初始化方法中 ``` //時間軸選擇界面 date.initPicker('month') $('.layui-form-pane').find('.submit-btn').on('click', function() { $('.layui-form-pane').find('.close-btn').trigger('click') }) ``` 解決方案：將以上代碼放到init方法中。 #### 7、調用不必要的方法 ``` $('.environment-params').empty().html(template(data)) ``` 解決方案：不用調用empty方法，因為調用html方法都會先將原來內容清空的。 #### 8、不要出現硬編碼，且要說明其含義 ``` if (d.value > 22.8) { return "url(#alarmGradient)" } else { return "url(#normalGradient)" } ``` 解決方案：定義為常量，并且說明其代表的含義。 #### 9、畫圖表的代碼不要和業務代碼混到一起，單獨封裝到一個文件中。 #### 10、可以優化的方法調用 ``` $('.date-container').empty().append(dateHTML) ``` 解決方案：`$('.date-container').html(dateHTML)`。 #### 11、@德才，參考其他模塊代碼，先重構下代碼結構。