`審查人：白占寧` `被審核代碼負責人：舒曼` `代碼地址：https://192.168.1.240:8443/svn/repos/source-analysis` ### 比較好的地方 * 注釋很完善，并且很規范。 ### 存在的問題 #### 1、Handlebars中的helpers定義到調用的方法外面 ``` Handlebars.registerHelper('addKey', function(index){ return index + 1 }) ``` 解決方案：將以上定義定義到初始化方法中或單獨放到一個helper.js文件中，避免重復定義。 #### 2、避免拼接字符串 ``` paging: function(totalPage, currentPage){ var pagingHtml = '' for(var i = 0; i < totalPage; i++){ pagingHtml += '<span><b></b></span>' } $('.ball-page p').html(pagingHtml) $('.ball-page span').eq(currentPage-1).addClass('active') } ``` 解決方案：使用模板代替，不要拼接字符串，該問題的修改可延后。 #### 3、避免無意思的方法聲明 ``` 'cx': function(d, i){ return 0 }, 'cy': function(d, i){ return svgHs / 2 - 30 }, ``` 解決方案：固定值時不用寫function，如下： ``` 'cx': 0, 'cy': svgHs / 2 - 30 ``` #### 4、每個變量都單獨使用`var`聲明 ``` var width = 0, height = 0, cfg = {}, svg = ''; ``` #### 5、方法定義位置不對 ``` function start(){ svg.selectAll('*').remove() pageBox.forEach(function(data, i){ var svgWs = 0; var svgHs = 0; if(i == (g-1)){ svgWs = svgW1; svgHs = svgW1; }else{ svgWs = svgW; svgHs = svgH; } var cirbox = svg.append('g') .attr({ 'transform': function(){ if(i == (g-1)){ // ...... ``` 解決方案：`start()`方法不要定義到`init()`方法內部，另外`forEach`循環體中操作的內容過多。 #### 6、data方法綁定數據問題 ``` var updataMiddleCir = updataCirSvg.selectAll('.middleCir') .data(function(d){ return [data] }) ``` 解決方案：使用如下的方式： ``` var updataMiddleCir = updataCirSvg.selectAll('.middleCir') .data([data]) ```