`审核人：白占宁` `被审核代码负责人：覃英琪` `代码地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-qinyingqi` #### 1、文件命名不规范（已改） `scripts/test.js` 解决方案：`test.js`文件重命名，命名需要和具体业务相关联。 #### 2、Mock数据引入问题（已改） `mockData/` 业务代码里面，不需要`require`多个mock文件。 解决方案：在`mockData/`下新建一个`index.js`文件，该文件中`require`其他所有的mock文件。 #### 3、缺失必要的注释（已改） 完善JS文件中的注释，不仅包括各个方法及文件需要注释，关键的代码处也需要明确的注释。 #### 4、代码格式混乱（已改）  解决方案：严格遵守《JavaScript编码规范》中的要求，该缩进的地方就缩进，注意代码的可读性。 #### 5、无用的`else`分支（已改） ``` .attr('fill', function(d, i) { if(i == 3) { return 'none' }else{ return '#0c1746' } }) ``` 解决方案：类似这种`if`分支中已经return了，则不再需要`else`分支，直接return即可，如下： ``` .attr('fill', function(d, i) { if(i == 3) { return 'none' } return '#0c1746' }) ``` #### 6、图表组件封装不合理（已改） `charts`目录下所有图表组件封装不合理，参考开发框架中的示例代码。 #### 7、CSS中class命名不规范（已改） `total_car_frame`。 解决方案：参考《CSS编码规范》，修改为`total-car-frame`。 #### 8、CSS书写不规范（已改） ``` .total_car_quantity{ font-family: digifacewide; font-size:35px; color:#fff; float:right; margin-right:5px; line-height:45px; display:inline-block; background:url(../images/bac_total_car_numberbac.png); width:31px; height:45px; vertical-align:middle; text-align: center; } ``` class命名、图片命名及属性顺序都不正确。 解决方案：参考《CSS编码规范》。 #### 9、JS中变量命名不规范（已改） ``` var svg_width = 530 var svg_height = 355 ``` 解决方案：参考《JavaScript编码规范》。 #### 10、空`if`语句（已改） ``` if(a[i].length == 7) { //调试 }else{ a[i].unshift('') if(a[i].length == 7) { // 调试 }else{ a[i].unshift('') } } ``` 禁止出现空的if语句。 解决方案：参考《JavaScript编码规范》。