`审核人：白占宁` `被审核代码负责人：谢洋` `代码地址：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、在部分电脑上页面效果有问题（未修改）