`審核人：白占寧` `被審核代碼負責人：李偉 & 舒曼` `代碼地址：https://192.168.1.240:8443/svn/repos/wanda` ### 存在的問題 #### 1、關于null的判斷 ``` if(name == null){ groupData = [xsData, xzData] nameDat = ['xingshi', 'xingzheng'] } ``` 解決方案：使用`!`運算符，如下： ``` if(!name){ groupData = [xsData, xzData] nameDat = ['xingshi', 'xingzheng'] } ``` #### 2、代碼可讀性差 ``` $('.nav li:eq('+index+')').siblings().removeClass('active').end().addClass('active') ``` 解決方案：添加必要的空格，必要的時候換行，如下： ``` $('.nav li:eq(' + index + ')') .siblings() .removeClass('active') .end() .addClass('active') ``` #### 3、代碼縮進問題 ``` switch(index) { case 0: //將地圖數據顯示為刑事、行政數據 // all.renderMap(mapData, null) $('#allMap .xingshi, #allMap .xingzheng').stop().fadeIn(); $('.all-map').removeClass('all-map-big') $('.all-wrap').stop().show(function(){ $('.all-wrap').addClass('act-animated') }) $('.all-map .cutline').stop().fadeIn(); break; ``` 解決方案：請參考《JavaScript編碼規范》進行正確的縮進，如下： ``` switch(index) { case 0: //將地圖數據顯示為刑事、行政數據 // all.renderMap(mapData, null) $('#allMap .xingshi, #allMap .xingzheng').stop().fadeIn(); $('.all-map').removeClass('all-map-big') $('.all-wrap').stop().show(function(){ $('.all-wrap').addClass('act-animated') }) $('.all-map .cutline').stop().fadeIn(); break; ``` #### 4、尾部分號不統一 ``` $('#allMap .xingshi').stop().fadeOut(); $('#allMap .xingzheng').stop().fadeIn(); $('.all-map').addClass('all-map-big') ``` 解決方案：要不統一全部加，要不全都不加分號，建議統一不加尾部分號。 #### 5、代碼注釋與下一個方法之前留一個空行 ``` /** * 左上角刑事和行政總數 */ renderCaseTotal:function(data) { var xsTotal = data.xstotal var xzTotal = data.xztotal $('.all-case-cri .all-case-value').html(xsTotal); $('.all-case-offi .all-case-value').html(xzTotal); var maxValue = xsTotal - xzTotal > 0 ? xsTotal : xzTotal; var xsxzArr = [xsTotal,xzTotal]; xsxzArr.map(function(value,index) { $('.all-num .all-progress').eq(index).css('width',Math.ceil(100 * xsxzArr[index] / maxValue) + '%') }) }, /** * 案件類型 */ renderCaseType:function(data) { var config = { width: 460, height: 260 } gradientAreaChart.drawCharts('#allTypeChart', data, config) } ``` 解決方案：在第一個方法結束后，留一個空行。 #### 6、data如何是數組，則循環方式有問題 ``` for(var d in data){ NewData.push({'name': nameData[i], 'value': data[d]}) i++ } ``` 解決方案：`for in`用于循環對象，不適用循環數組。可以使用傳統的`for循環，map或forEach`等。 #### 7、方法注釋不規范 ``` /** * 刑事案件強制措施 */ renderCriminalMeasure:function(data) { var measureHtml = Handlebars.compile(measureTpl) $("#criMeasure").html(measureHtml(data)) }, ``` 解決方案：使用編輯器插件，配置統一的注釋，至少包括方法描述、參數說明、返回值這三部分，最好能夠再添加上參數示例，如下： ``` /** * @description 將原始數據轉換成D3圖表組件 waterBall能直接用于渲染的數據 * @params {array} data 參數說明 * @return {array} 計算后的數組 * 原始數據例子：[ * { * name: "治安", * value: 55 * } * ] * 轉換后：[[[55,45],['治安']]] * 注：第一個數組兩個值合起來是100，第一個值是D3渲染值 */ getAcceptTypeData:function(data) { var renderData = []; data.map(function(item,index) { renderData.push([ [item.value,100 - item.value], [item.name] ]) }) return renderData; } ```