`审核人：白占宁` `被审核代码负责人：赵家红` `代码地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-zhaojiahong-es6` #### 1、命名问题（已修改） ``` .li-i{ margin-top:10px; margin-left:70px; font-size:24px; } .left-l{ background: url(./images/bg_1.png); } ``` 命名不仅仅指JS中变量、方法的命名，也包括CSS中class、id的命名及图片命名等，命名要有一定的语义化。 #### 2、CSS中不规范的用法（已修改） ``` .crews:{ width:1750px; height: 50px; } ``` CSS中类名后面跟冒号 ？ #### 3、(已修改） ``` var activityData = $('.cinfo-activity') var actData = $('.cinfo-activity') for (let i = 0, length = activityData.length; i < length; i++) { var context = activityData.eq(i).html() if (context === '不活跃') { actData.eq(i).css('color','#ff002a') actData.eq(i).append('<p class="colorp">(共预警10次)</p>') } ``` #### 4、jQuery与原生JS混用（已修改） ``` document.querySelector('#add').innerHTML = adds(data.info) ``` 既然引入了jQuery，就用jQuery吧。 说明：`proinfo.js`中未修改。 同一问题，要修改时候全部都修改了，不要只修改几个剩余的就不管了。 #### 5、删除无用的代码（已修改） ``` constructor() { console.log('aaa') } ``` 说明：如果该方法完全没用，则全部删除掉即可。 #### 6、注释里面的描述要有意义（已修改） ``` /** * @Author: zhaojiahong * @DateTime: 2017-11-13 17:55:32 * @Description: Description * @Last Modified By: zhaojiahong * @Last Modified Time: */ ``` #### 7、方法注释不全（已修改） ``` // 活跃度上色 color() { ... ``` 参考《JavaScript编码规范》。 #### 8、建议使用解构赋值简化操作（已修改） ``` import MyInfo from './scripts/index' import loader from '@/loader/loader' // 实例化 const temp = new MyInfo.ProInfos() const search = new MyInfo.Search() const add = new MyInfo.Add() ``` 修改为： ``` import { ProInfos, Search, Add } from './scripts/index' import loader from '@/loader/loader' // 实例化 const temp = new ProInfos() const search = new Search() const add = new Add() ```