`審核人：尹幫會` `被審核代碼負責人：歐陽德才` `代碼地址：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`?時須省略單位。 (也只有長度單位可省)