最近在做一个测试题,前后可以切换题目,点击按钮选择答案,选择答案的同时改变按钮的背景色,如下图所示:
初始代码
我用了vue和swiper。所有的题目是一个对象数组,通过v-for渲染:
<swiper ref="mySwiper" :options="swiperOptions"> <swiper-slide v-for="(item, index) in listData" :key="index"> <div class="question-box"> <div class="idx">- 第{{ index+1 }}题 -</div> <div class="question">{{ item.question }}</div> </div> <button @click="goNext(index, 'a')" :class="item.answer=='a' ">是</button> <button @click="goNext(index, 'b')" :class="item.answer=='b' ">否</button> </swiper-slide> </swiper>
一开始我把每道题目的answer存放在对象里面,点击的按钮时候切换answer的值,button的动态class监听到值改变后会引发背景色的改变。js部分:
goNext(index, answer) { this.$set(this.listData[index], 'answer', answer) this.swiper.slideNext(100) },
发现问题
测试发现这样把点击事件和动态样式互相依赖,会造成大概几百毫秒的延迟才执行slideNext(),是可以直观感受到的延迟。通过调试,发现造成延迟有两方面的原因:
this.$set
更改数组执行完点击事件
到动态class监听到数据的改变
中间也有延迟。
于是我换了一个思路,不把每个题目的answer放在对像数组里面,而是在data里面定义一个answerMap,这样解决了问题1。为了解决问题2,我选择把动态样式 :class
去掉,改成用原生js在点击事件里面直接修改class。这样两步下来,确实看不到延迟了。
优化后的代码
html部分
<button @click="goNext($event, index, 'a')">是</button> <button @click="goNext($event, index, 'b')">否</button>
js部分
goNext(e, index, answer) { const element = e.target const bro = element.parentNode.children for (let i = 0; i < bro.length; i++) { if (bro[i] !== element) { bro[i].classList.remove("active") } } element.classList.add('active') this.answerMap[this.listData[index].question] = answer this.swiper.slideNext(100) },
测试结果:可喜可贺,slideNext()不再有肉眼可见的延迟了。
有的时候为了少写点代码,不知不觉牺牲了性能。这次实践,虽然原生JS使我增加了好几行代码,但是带来的性能提升也是非常明显的。经过这次优化,我觉得如果对Vue源码理解透彻的人,大概是能马上知道优化点。我虽然看过一遍源码,但还是理解不够深入。需要学习的还有很多呀~