`审核人：张红桥` `被审核代码负责人：李伟` `代码地址：https://192.168.1.240:8443/svn/repos/wanda/script/official.js` ### 存在的问题 #### 1、单双引号混用 ``` renderPunish:function(data) { var punishHtml = Handlebars.compile(punishTpl) data.forEach(function(d){ d.value = d.value.toLocaleString() }) $(".offi-punish-list").html(punishHtml(data)) }, ``` 解决方案：javaScript中统一使用单引号 #### 2、方法中传多个字符或变量时逗号后面未保留一个空格隔开,该文件中每个方法都存在此问题 ``` renderAcceptType:function(data) { console.log(data) var config = { width: 730, height: 260 } var renderData = this.getAcceptTypeData(data) waterBall.drawCharts("#offiAcceptChart",renderData,config) }, ``` 解决方案：方法中传多个字符或变量时逗号后面使用保留一个空格 ``` renderAcceptType:function(data) { console.log(data) var config = { width: 730, height: 260 } var renderData = this.getAcceptTypeData(data) waterBall.drawCharts("#offiAcceptChart", renderData, config) }, ``` #### 3、waterBall变量只在该方法中使用了一次，没必要定义为全局变（变量gradientAreaChart，punishTpl都存相同的问题） ``` renderAcceptType:function(data) { var config = { width: 730, height: 260 } var renderData = this.getAcceptTypeData(data) waterBall.drawCharts("#offiAcceptChart",renderData,config) }, ``` 解决方案：建议将waterBall变量的组件引入定义局部变量，减少全局变量的定义。