feeluown / FeelUOwn

trying to be a robust, user-friendly and hackable music player
http://feeluown.readthedocs.io
GNU General Public License v3.0
3.52k stars 596 forks source link

[Bug] 私人 FM 极其偶然可能出现重复播放列表的问题 #367

Closed BruceZhang1993 closed 4 years ago

BruceZhang1993 commented 4 years ago

描述 私人 FM 极其偶然可能出现重复播放列表的问题

重现方法

  1. 进入私人 FM 播放
  2. 向播放列表添加此前已播放歌曲/私人FM接口提供重复的歌曲
  3. 当前歌曲播放结束后未获取 FM 歌曲而是选择列表中的下一首

预期行为 当前歌曲播放结束后获取下一首 FM 歌曲

版本信息

cosven commented 4 years ago

复现了。当一首歌 url 不正常(bad song)的时候,播放器就不会去获取新歌曲了。 workaround:将播放列表的最后一首歌删除(最后一首是不正常),然后继续播放就好了。

BruceZhang1993 commented 4 years ago

复现了。当一首歌 url 不正常(bad song)的时候,播放器就不会去获取新歌曲了。 workaround:将播放列表的最后一首歌删除(最后一首是不正常),然后继续播放就好了。

我遇到的好像并不是这样的情况?我是播放到了 FM 模式的历史播放歌曲.. URL 是有效的

cosven commented 4 years ago

我又仔细看了几遍代码,感觉「播放到了 FM 模式的历史播放歌曲」这个逻辑应该是可以正确处理的...有空的时候,我手动构造一下有重复歌曲的这种场景,看看问题是什么。

https://github.com/feeluown/FeelUOwn/commit/fd89f333143d8bb601f9c293d5a0bccbd3db0d25 我先修复了我上面说的那种情况。

cosven commented 4 years ago

我先把这个 close 了... 等复现的时候再打开吧 ~

strit12 commented 4 years ago

Sukaa