`审核人：白占宁` `被审核代码负责人：李伟 & 舒曼` `代码地址：https://192.168.1.240:8443/svn/repos/wanda` ### 存在的问题 #### 1、新增一个doc目录，将接口文档放在该文件夹下 #### 2、页面时间从后台获取 ``` // charts/showTime.js setInterval(function() { var data = showTime.getData() var showTimeHtml = Handlebars.compile(showTimeTpl) $(".show-time").html(showTimeHtml(data)) },1000) ``` 解决方案：将以上实现方式改成通过websocket接口，从后台获取时间。 #### 3、JavaScript中使用单引号 ``` switch (day) { case 1: weekDay = "星期一"; break; case 2: weekDay = "星期二"; break; ``` #### 4、Mock数据引用方式优化 ``` // scripts/index.js /** * 三个页面公用的mocke数据 */ require('../mockData/allNum.js') require('../mockData/map.js') /** * 总览页面mocke数据 */ require('../mockData/allCaseType.js') require('../mockData/allCaseRatio.js') require('../mockData/allAreaTop.js') require('../mockData/allMeasure.js') ``` 解决方案：在mockData下，新建一个index.js文件，index.js中require各个mock的文件，并且在index.js中require('mock')。 ``` // mockData/index.js define(function(require) { /**     * 三个页面公用的mocke数据     */       require('../mockData/allNum.js')       require('../mockData/map.js')           /**     * 总览页面mocke数据     */      require('../mockData/allCaseType.js')   require('../mockData/allCaseRatio.js')     require('../mockData/allAreaTop.js')   require('../mockData/allMeasure.js') }) // scripts/index.js require('../mockData/index') // 可进一步优化，在seajs.config.js中配置mockData的path // scripts/index.js require('mockData') ``` #### 5、charts目录下文件的引用方式 ``` var showTime = require('../charts/showTime.js') ``` 解决方案：在`seajs.config.js`中配置charts的path，然后通过require('showTime')这种方式引用。 #### 6、模板文件可读性 ``` {{#each this}} <li><span>{{name}}</span><span>{{value}}</span></li> {{/each}} ``` 解决方案：通过必要的换行提高代码可读性。 ``` {{#each this}} <li> <span>{{name}}</span> <span>{{value}}</span> </li> {{/each}} ```