`審核人：歐陽德才` `被審核代碼負責人：謝洋` `代碼地址：https://192.168.1.240:8443/svn/repos/qingbao-wall` ### 存在的問題 #### 1、用ES6方法代替lodash方法（未修改） 之前合并配置之處用的是lodash的merge方法，可以用es6的Object.assign方法實現 ```javascript this.config = _.merge({}, defaultSetting, opt) ``` 解決方案： ```javascript this.config = Object.assign({}, defaultSetting, opt) ``` #### 2、d3屬性設置優化 在設置d3對象屬性時，建議把所有的屬性設置包含在一個對象當中，不用分開設置 ```javascript text.attr('fill', (d, i) => `${type === 1 ? '#fff' : self.getColor(i)}`) .attr('text-anchor', (d) => self.midAngel(d) < Math.PI ? 'start' : 'end') .attr('font-size', 14) .attr('class', `${type === 1 ? 'name-text' : 'value-text'}`) .attr('transform', (d) => { let pos = self.lEndArc.centroid(d) let radius = outerRadius + 40 pos[0] = radius * (self.midAngel(d) < Math.PI ? 1.1 : -1.1) return `translate(${pos})` }) .attr('dy', `${type === 1 ? '1.2em' : '-0.5em'}`) ``` 解決方案： ```javascript text.attr({ 'fill':(d, i) => `${type === 1 ? '#fff' : self.getColor(i)}`, 'text-anchor': (d) => self.midAngel(d) < Math.PI ? 'start' : 'end', 'font-size': 14, 'class': `${type === 1 ? 'name-text' : 'value-text'}`, 'transform':(d) => { let pos = self.lEndArc.centroid(d) let radius = outerRadius + 40 pos[0] = radius * (self.midAngel(d) < Math.PI ? 1.1 : -1.1) return `translate(${pos})` }, 'dy': `${type === 1 ? '1.2em' : '-0.5em'}` }) ```