`审查人：白占宁` `被审核代码负责人：谢洋` `代码地址：http://10.0.91.5/opt/svn/DataCommandScheduling/project/bjDataVisualization` #### 1、`sagas/index.js`文件优化（已修改） 参考贵州块数据指挥调度系统代码，优化以下代码： ``` yield all([ /* ---------------------------------------- 总览页 ------------------------------------------ */ // 发案态势 fork(fetchCaseToday), fork(fetchCaseType), fork(fetchCaseDistribution), // 破案态势 fork(fetchCaseResolvedToday), fork(fetchCaseResolvedType), fork(fetchCaseResolvedDistribution) ``` #### 2、CSS属性中不用加引号（未修改） ``` background: url("law-enforcement-list-item-bg.png") no-repeat center / 632px 97px; ``` CSS中属性值不用加引号，如下： ``` background: url(law-enforcement-list-item-bg.png) no-repeat center / 632px 97px; ``` #### 3、注释不完整（未修改） ``` /** * 渲染 * @returns {*} */ render() { ``` #### 4、`redux`与`react`组件结合使用问题（未修改） 使用`reudx`中的`connect`方法连接`React`组件时，尽量只在顶层组件中使用，避免每个子组件中都使用`connect`，在顶层组件中获取到数据，在`mapStateToProps`方法中筛选需要的数据，然后通过`props`传递给各个子组件即可。  上面的代码中，应该在`index.js`中获取数据，筛选需要的数据后，分别传入到各个子组件中。 #### 5、缓存多次使用到变量（未修改） ``` componentDidMount() { this.initMap() this.props.dispatch(caseTodayListRequest()) this.props.dispatch(solvedCaseTodayListRequest()) this.props.dispatch(vehicleListRequest()) this.props.dispatch(policeListRequest()) } ``` 建议修改如下： ``` const { dispatch } = this.props dispatch(caseTodayListRequest()) ``` #### 6、关键逻辑缺少说明注释（未修改） `map`组件中关键逻辑部分缺少相关的说明注释，建议在关键地方添加注释说明，便于后续项目维护。