fix: #151 #152

go-musicfox · Jun 13, 2024 · 5e53d75 · 5e53d75
1 parent fc9999f
commit 5e53d75
Show file tree

Hide file tree

Showing 4 changed files with 38 additions and 31 deletions.
diff --git a/pkg/state_handler/play_controller.go b/pkg/state_handler/play_controller.go
@@ -11,5 +11,4 @@ type Controller interface {
 	CtrlPrevious()
 	CtrlSeek(duration time.Duration)
 	CtrlSetVolume(volume int)
-	PlayingInfo() PlayingInfo
 }
diff --git a/pkg/state_handler/state_handler_darwin.go b/pkg/state_handler/state_handler_darwin.go
@@ -35,7 +35,7 @@ const (
 	MediaTypeVedio
 )
 
-func NewHandler(p Controller) *Handler {
+func NewHandler(p Controller, _ PlayingInfo) *Handler {
 	playingCenter := mediaplayer.MPNowPlayingInfoCenter_defaultCenter()
 	commandCenter := mediaplayer.MPRemoteCommandCenter_sharedCommandCenter()
 	commandHandler := remoteCommandHandler_new(p)

diff --git a/pkg/ui/player.go b/pkg/ui/player.go
@@ -95,7 +95,7 @@ func NewPlayer(model *NeteaseModel) *Player {
 	ctx, p.cancel = context.WithCancel(context.Background())
 
 	p.Player = player.NewPlayerFromConfig()
-	p.stateHandler = state_handler.NewHandler(p)
+	p.stateHandler = state_handler.NewHandler(p, p.PlayingInfo())
 
 	// remote control
 	go func() {
@@ -547,6 +547,14 @@ func (p *Player) PreviousSong(isManual bool) {
 	_ = p.PlaySong(song, DurationPrev)
 }
 
+func (p *Player) Seek(duration time.Duration) {
+	p.Player.Seek(duration)
+	if p.lrcTimer != nil {
+		p.lrcTimer.Rewind()
+	}
+	p.stateHandler.SetPlayingInfo(p.PlayingInfo())
+}
+
 // SetPlayMode 播放模式切换
 func (p *Player) SetPlayMode(playMode player.Mode) {
 	if playMode > 0 {
@@ -738,12 +746,24 @@ func (p *Player) handleControlSignal(signal CtrlSignal) {
 	case CtrlNext:
 		p.NextSong(true)
 	case CtrlSeek:
-		p.Player.Seek(signal.Duration)
-		if p.lrcTimer != nil {
-			p.lrcTimer.Rewind()
-		}
-		p.stateHandler.SetPlayingInfo(p.PlayingInfo())
+		p.Seek(signal.Duration)
 	case CtrlRerender:
 		p.model.Rerender(false)
 	}
 }
+
+func (p *Player) PlayingInfo() state_handler.PlayingInfo {
+	music := p.curSong
+	return state_handler.PlayingInfo{
+		TotalDuration:  music.Duration,
+		PassedDuration: p.PassedTime(),
+		State:          p.State(),
+		Volume:         p.Volume(),
+		TrackID:        music.Id,
+		PicUrl:         music.PicUrl,
+		Name:           music.Name,
+		Album:          music.Album.Name,
+		Artist:         music.ArtistName(),
+		AlbumArtist:    music.Album.ArtistName(),
+	}
+}
diff --git a/pkg/ui/player_controller.go b/pkg/ui/player_controller.go
@@ -2,64 +2,52 @@ package ui
 
 import (
 	"time"
-
-	"github.com/go-musicfox/go-musicfox/pkg/state_handler"
 )
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlPaused() {
 	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.ctrl <- CtrlSignal{Type: CtrlPaused}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlResume() {
 	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.ctrl <- CtrlSignal{Type: CtrlResume}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlStop() {
 	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.ctrl <- CtrlSignal{Type: CtrlStop}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlToggle() {
 	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.ctrl <- CtrlSignal{Type: CtrlToggle}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlNext() {
 	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.ctrl <- CtrlSignal{Type: CtrlNext}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlPrevious() {
 	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.ctrl <- CtrlSignal{Type: CtrlPrevious}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlSeek(duration time.Duration) {
-	p.ctrl <- CtrlSignal{
-		Type:     CtrlSeek,
-		Duration: duration,
-	}
+	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
+	p.ctrl <- CtrlSignal{Type: CtrlSeek, Duration: duration}
 }
 
+// Deprecated: Only state_handler.Handler can call this method, others please use Player instead.
 func (p *Player) CtrlSetVolume(volume int) {
-	// 不更新playingInfo
+	// NOTICE: 提供给state_handler调用，因为有GC panic问题，这里使用chan传递
 	p.Player.SetVolume(volume)
 }
-
-func (p *Player) PlayingInfo() state_handler.PlayingInfo {
-	music := p.curSong
-	return state_handler.PlayingInfo{
-		TotalDuration:  music.Duration,
-		PassedDuration: p.PassedTime(),
-		State:          p.State(),
-		Volume:         p.Volume(),
-		TrackID:        music.Id,
-		PicUrl:         music.PicUrl,
-		Name:           music.Name,
-		Album:          music.Album.Name,
-		Artist:         music.ArtistName(),
-		AlbumArtist:    music.Album.ArtistName(),
-	}
-}