`審核人：白占寧` `被審核代碼負責人：孫雨珩` `代碼地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-sunyuheng` ### 值得分享的地方 * 各個組件文件中代碼的組織形式； * `moment.js`的使用； * `conic-gradient`使用場景及能實現的效果分享。 ### 存在的問題 #### 1、項目目錄結構 目錄結構與`gitlab`上的不一致，所有項目的目錄結構應該保持一致。 #### 2、變量命名 ``` var $el = me.$el = $(tpl) var $count = me.$count = $el.find('.J-count') var $title = me.$title = $el.find('.J-title') ``` 解決方案：聲明變量時不建議以`$`開頭。 #### 3、項目中`_`默認為表示`lodash.js` ``` var _ = require('common/util') ``` 解決方案：替換變量名稱，不要使用`_`命名。 #### 4、選擇元素時原生JS與jQuery混用 ``` var containerEl = document.querySelector(container); if (containerEl) { containerEl.appendChild( me.$el.node() ) } ``` 解決方案：項目中引入了jQuery，涉及到DOM相關操作就都使用jQuery來完成。 #### 5、提供的API方法的參數 提供的繪圖API的方法參數固定為三個：容器，數據和配置項，其中容器和數據必選，配置項可選。 ``` /** * 繪制圓形類型圖，對外提供的API接口 * * @param {string} container 容器 * @param {array} data 數據源 * @param {object} options 配置項 */ function drawRadarCircle(container, data, options) { var cfg = _.merge({}, defaultSetting, options) // ...... ``` #### 6、對外提供的API中不需要創建SVG元素 ``` var $el = me.$el = d3.select( document.createElementNS('http://www.w3.org/2000/svg', 'svg') ) .classed('donut-map-chart', true) ``` 解決方案：不建議在對外提供的API中創建SVG元素。 #### 7、定義的變量從未使用 ``` var timer = new Timer({ el: '#J-timer-container', }) ``` 解決方案：定義了`timer`變量，在后面的代碼中一直沒有使用，就不要定義該變量。 #### 8、關于假值的判斷 ``` floorNum: function(num, n) { if (n == null) { n = 2 } var scale = Util.calcNumScale(n) return Math.floor(num * scale) / scale } ``` 解決方案：n有默認值，因此是可選的，所以這里判斷時候建議使用`!n`代替`n == null` ，或者使用如下方式： ``` if(n == null) { n = 2 } // 替換為下面這種方式 n = n || 2 ```