`审核人：尹帮会` `被审核代码负责人：欧阳德才` `代码地址：https://192.168.1.240:8443/svn/repos/yunwei/src` ### 存在的问题 #### 1、 `var` 声明多个变量。 ``` var svg = null, y = null, yAxis = null ``` 解决方案：一个 `var` 声明多个变量，容易导致较长的行长度，并且在修改时容易造成逗号和分号的混淆。 ``` var svg = null var y = null var yAxis = null ``` #### 2、模块缺少文件注释 ``` define(function(require) { require('d3') ... }） ``` 解决方案：文件顶部必须包含文件注释，用 `@file` 标识文件说明。 #### 3、函数无注释或者注释不规范 ``` function init(data) { ... } /** * 设置内部旋转圈 */ function renderInnerCircle(center) { ... } ``` 解决方案： 函数/方法注释必须包含函数说明，有参数和返回值时必须使用注释标识。 #### 4、单行注释不规范 ``` //加柱状 svg.append('g').attr('class', 'rects').selectAll("rect") ``` 解决方案：必须独占一行。`//` 后跟一个空格，缩进与下一行被注释说明的代码一致。 #### 6、注释不规范 ``` /************后台数据请求及绘图 begin *************/ getCircleData(center) ``` 解决方案：采用规范的注释方法。 #### 5、部分切图不合理 解决方案：切图不能将涉及页面可能修改的文字等元素同背景图片一起切下来，后期可能会有修改，不利于后期维护。 #### 6、`>`、`+`、`~` 选择器的两边未保留一个空格。 ``` .environment-params>div { position: relative; float: left; width: 400px; height: 370px; } ``` 解决方案：`>`、`+`、`~` 选择器的两边各保留一个空格。 #### 7、长度为 `0` 时未省略单位。 ``` body { transform-origin: 0px 0px 0px; -ms-transform-origin: 0 0; overflow: hidden; position: relative; } ``` 解决方案：长度为 `0` 时须省略单位。 (也只有长度单位可省)