`審核人：白占寧` `被審核代碼負責人：謝洋` `代碼地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-xieyang` ## 技偵全息指揮系統代碼審查 #### 1、文件命名問題（已修改）  解決方案：所有文件，包括HTML、js、css或模板，首字母都應該以小寫字母開始。 #### 2、`seajs.config.js`中的配置項命名問題（已修改）  解決方案：首字母小寫。 #### 3、請求數據的代碼的位置（已修改） ``` // 初始化js中的方法 /** * @description 按需加載 * @param filed 時間段參數 */ load: function (filed) { map.init() category.init() totalCase.init() bar.init() sample.init() timeLine.init(filed) } // 組件中的方法 init: function () { var self = this; request.sendAjax(baseConfig.bar, function (data) { self.sort(data.case, 'value') self.sort(data.tCase, 'tValue') self.render(data) }) } ``` 解決方案：請求數據的代碼應該統一放到初始化js中，加載到數據后，將所需的數據通過參數的形式傳給各個組件的初始化方法。 #### 4、尾部分號使用不統一（已修改） ``` require('handlebars') require('jquery') var request = require('request') var tpl = require('tpl/totalCase.tpl') var baseConfig = require('baseConfig') require('tiData'); var util = require('../common/util.js') ``` 解決方案：要么都統一使用尾部分號，要么都不使用，建議不使用分號。 #### 5、每個變量都單獨使用`var`來聲明（已修改） ``` var request = require('request'), tpl = require('tpl/sample.tpl'), baseConfig = require('baseConfig'), util = require('../common/util.js') ``` 解決方案：如下所示 ``` var request = xxx var tpl = xxx ``` #### 6、不恰當的換行（已修改） ``` .style( { 'left': cp.properties.proPeking[0] - $(this).width() / 2 + 'px', 'top': cp.properties.proPeking[1] - $(this).height() / 2 + 'px' } ) ``` 解決方案：如下 ``` .style({ 'left': cp.properties.proPeking[0] - $(this).width() / 2 + 'px', 'top': cp.properties.proPeking[1] - $(this).height() / 2 + 'px' }) ``` #### 7、上次code Review的結果沒有改完（已修改）  #### 8、在部分電腦上頁面效果有問題（未修改）