7-18这章节，关于随机播放

Question

黄sir课程里，是做完随机播放功能之前，并没有在shuffle函数里创建arr的副本。
效果：随机播放功能完成，返回歌曲列表并点击，被打乱了，不能正确播放。
然后又去做了selectPlay里如果是random。这层逻辑。
还是行不通，最后再去写的shuffle里 _arr=arr.slice()这个逻辑。
最后效果：success。

我呢，做完了歌手详情页随机播放按钮这个功能，返回歌曲列表页，顺序并没有被打乱。(在此之前我做了shuffle函数里，arr的副本,arr.slice())
大胆猜测：selectPlay里，这层逻辑似乎多余了。如下图：

Answer 1

高傲的小猫 2020-04-05 18:03:08

我也觉得这个代码是多余的，在 selectPlay 没加判断之前，点了随机播放全部， 然后在点击滚动里面的歌曲出现bug，我觉得是在 shuffle 这个函数里面 没添加 slice（）这个方法导致的。如果不加这个方法话，相当于把sequenceList里面的数组给改变了。也即是传值与传址的问题了。

Answer 2

ustbhuangyi 2019-08-20 10:41:37

为什么是多余的呢？

• 提问者 qq_是时候换个脑子了_0 #1

  因为，我没有写这层逻辑，只是在shuffle函数里，多写了let _arr=arr.slice()，没有问题的。刚才又试了一遍，不写arr.slice()这层逻辑，歌曲列表乱了。

• ustbhuangyi 回复 提问者 qq_是时候换个脑子了_0 #2

  slice 是肯定需要的，课程源码也是加了这个的，findIndex 的目的是修正切换模式后播放的索引。

• 提问者 qq_是时候换个脑子了_0 回复 ustbhuangyi #3

  黄sir，我想表达的意思是：
  我在selectPlay这个action中没有去写：判断随机播放这层逻辑，
  只是在shuffle工具函数中和你视频中保持一样，slice建arr副本，
  但是我的业务还是正常的，返回歌曲列表页，点击歌曲还是对应的，没有被打乱。
  
  所以，源码里，我想反驳这层逻辑多余。

点开查看后面16条评论