update to fix logic around next music switch

danilko · Aug 15, 2019 · d4b367c · d4b367c
1 parent d3dbb8e
commit d4b367c
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/src/main/java/com/musicutility/component/MusicPlayerComponent.java b/src/main/java/com/musicutility/component/MusicPlayerComponent.java
@@ -34,8 +34,6 @@ public void run() {
                     if(musicPlayerSetting.getCurrentMusicFile() != null && musicPlayerSetting.getCurrentMusicList() != null && musicPlayerSetting.getCurrentMusicMixer() != null) {
                         MusicPlayerComponent musicPlayerComponent = new MusicPlayerComponent(musicPlayerRepository);
                         musicPlayerComponent.play();
-
-                        musicPlayerRepository.nextMusicFile();
                     }
                     else
                     {
@@ -109,17 +107,25 @@ void play() {
                 musicPlayerState.setPlaying(true);
                 musicPlayerState.setFramePosition(0);
 
+                boolean nextMusic = true;
+
                 while ((byteRead = audioInputStream.read(byteBuffer)) != -1) {
                     sourceDataLine.write(byteBuffer, 0, byteRead);
                     musicPlayerState.setFramePosition( musicPlayerState.getFramePosition() + (byteRead / musicPlayerState.getFrameSize()));
                     musicPlayerRepository.setMusicPlayerState(musicPlayerState);
 
                     if ( musicPlayerRepository.getMusicPlayerSetting().getPlay() == false) {
 
+                        // As it is sent by stop command, stop at current music, not move to next one
+                        nextMusic = false;
                         break;
                     }
                 }
 
+                // Only switch if it is a complete music file play, not due to early stop
+                if(nextMusic) {
+                    musicPlayerRepository.nextMusicFile();
+                }
 
             } else {
                 musicPlayerState.setErrorMessage("cannot find target mixer");