今天修改一个bug,其实是个很简单的需求,但是花了很多的时间,改到后面越来越无奈,因为如果经验丰富或者有水平的话,马上就可以改出来。
需求很简单:要求返回刷新。按照一般图方便思路都是,如果setState不同,让界面重新渲染。
componentWillReceiveProps(nextProps){
this.setState({
...nextProps
})
}
但是我这个bug的问题在于,原代码使用的是不相关组件(A跳转到B,B返回A)的形式,使用dva作为状态管理。
问题一:不清楚为什么一开始没有使用到componentWillReceiveProps,还是对生命周期的掌握度不够高,
问题二:因为懒,直接放入nextProps,没有对比参数
问题三,两个兄弟组件之间,如果用event事件总线的话,会导致监听太多,导致内存溢出。所以在已经有dva的基础上,使用dva。A跳转到B时,定义一个公共状态标志,调完接口,界面显示则设为false,从B回到A则设为true。因为一直没怎么实际使用,总是觉得有些难。还想到了sessionStorage,不清楚怎么想的,总之要好好加油,
总结:兄弟组件可以使用状态管理器
总之,解决一个很简单的bug,也没有喜悦感,因为这个bug不该花这么多事件,不过有学到很多东西,也是经验,要多实践,总之要加油