计时器在歌曲播放结束后未停止，播放进度条和时间标签交互不正常 about beslyric-for-x HOT 5 CLOSED

尝试解决定时器问题：
https://github.com/Beslyric-for-X/Beslyric-for-X/blob/de490e3d6cfe238097ac2c40cffff05807b6fa5b/Entities/MusicPlayer/musicPlayer.cpp#L761-L775

from beslyric-for-x.

行，算是临时修改吧。

有个问题，这里不需要判断 m_positionUpdateTimer.isActive() 了吗？

如果这里不需要，其他地方对其的判断是多余吗？

from beslyric-for-x.

我写的时候也有稍微想到这一点，应该也算是比较懒，直接认为如果定时器已经停止，再调用一次不会做任何动作。

不过你这么一问我倒是仔细注意了这个问题，不知道对已经停止的定时器调用stop会是什么结果？

from beslyric-for-x.

另外有一个问题。运行 de490e3 版本，歌曲播放结束时会输出：

QObject::killTimer: Timers cannot be stopped from another thread

测试了下， 收到 audioFinish 信号后执行的匿名函数是在 PlayThread 线程中执行的，所以会有这个警告。尽管 m_positionUpdateTimer 的确是停止了，但这或许会有隐患？

不过换用 QMetaObject::Connection QObject::connect(const QObject *sender, const char *signal, const QObject *receiver, const char *method, Qt::ConnectionType type = Qt::AutoConnection) 方法能确保代码在 receiver 中执行，就像这样：

connect(playThread, SIGNAL(audioFinish(bool)), this, SLOT(theSlot(bool)));

void MusicPlayer::theSlot(bool isEndByForce) {
	m_positionUpdateTimer.stop();
}

from beslyric-for-x.

不过你这么一问我倒是仔细注意了这个问题，不知道对已经停止的定时器调用stop会是什么结果？

我在网上找了，没看到相关的说法，可以猜测定时器会继续保持停止状态。

另外， void QTimer::start() 倒是会使已开始工作的定时器停止并重新开始。

void QTimer::start() # QTimer Class | Qt Core 5.12.3

from beslyric-for-x.