kunminx/jetpack-musicplayer

Json 数据驱动，音乐播放组件

jetpack-musicplayer's Issues

【优化】切换歌曲后第一时间的播放进度会被上一首数据污染

现在当前播放的数据是只有一个实例：

private PlayingMusic mCurrentPlay = new PlayingMusic("00:00", "00:00");

但是当切换歌曲时，只是更换了部分播放数据：

   public void setChangingPlayingMusic(boolean changingPlayingMusic) {
        mIsChangingPlayingMusic = changingPlayingMusic;
        if (mIsChangingPlayingMusic) {
            ...
            mCurrentPlay.setBaseInfo(mPlayingInfoManager.getMusicAlbum(), getCurrentPlayingMusic());
           ....
        }
    }

    public void setBaseInfo(B musicAlbum, M music) {
        //要用当前实际播放的列表，因为不同模式存在不同的播放列表
        this.title = music.getTitle();
        this.summary = musicAlbum.getSummary();
        this.albumId = musicAlbum.getAlbumId();
        this.musicId = music.getMusicId();
        this.img = music.getCoverImg();
        this.artist = (A) music.getArtist();
        this.music = music;
    }

这个时候的播放进度和时间是没有被重置的，分发当前播放音乐消息时会把上一首的播放数据也带过去，造成数据污染，
建议在更换歌曲时重置一下播放进度和时间：

   public void setChangingPlayingMusic(boolean changingPlayingMusic) {
        mIsChangingPlayingMusic = changingPlayingMusic;
        if (mIsChangingPlayingMusic) {
             // 重置播放时间和进度数据
            mCurrentPlay.setNowTime("00:00");
            mCurrentPlay.setAllTime("00:00");
            mCurrentPlay.setPlayerPosition(0);
            mCurrentPlay.setDuration(0);
            mCurrentPlay.setBaseInfo(mPlayingInfoManager.getMusicAlbum(), getCurrentPlayingMusic());
           ....
        }
    }

有apk体验包吗？

有demo吗

如题

丢失文件 import com.kunminx.puremusic.ui.helper.DrawerCoordinateHelper

在"SearchFragment"类里面引用“DrawerCoordinateHelper”,找不到这个文件

is:issue is:open 学习了，但是为什么少了一些java文件，AdapterPlayItemBinding，FragmentPlayerBinding等这些文件不在里面

请问再重复 播放 再调用暂停的方法 会导致 程序会变得很卡的情况 logcat上一直报这个错误

MediaPlayerNative: error (-38, 0)
MediaPlayerNative: [notify] : [1170] callback app listenerNotNull=1, send=1
MediaPlayer: [HSM] stayAwake false uid: 10759, pid: 28843
MediaPlayerNative: stop called in state 0, mPlayer(0xc0cbf8e0)
MediaPlayerNative: Message: MEDIA_ERROR(100), ext1=Unknown MediaErrorType(-38), ext2=0x0
MediaPlayerNative: error (-38, 0)
MediaPlayerNative: [notify] : [1170] callback app listenerNotNull=1, send=1
。。。
一直重复的报这个 是什么原因额

代码中就写了获取当前播放音频getCurrentPlayingMusic() 的url 一致就调用pauseAudio() 方法

一直点击列表卡顿之后崩溃切换音乐崩溃

java.lang.RuntimeException: Unable to start service com.kunminx.puremusic.player.notification.PlayerService@b6f7781 with Intent { cmp=com.kunminx.puremusic.debug/com.kunminx.puremusic.player.notification.PlayerService }: java.lang.IllegalStateException: Pid 2142 has exceeded the number of permissibleregistered listeners. Ignoring request to add.
at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:4186)
at android.app.ActivityThread.access$1900(ActivityThread.java:229)
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1933)
at android.os.Handler.dispatchMessage(Handler.java:107)
at android.os.Looper.loop(Looper.java:226)
at android.app.ActivityThread.main(ActivityThread.java:7592)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:539)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950)
Caused by: java.lang.IllegalStateException: Pid 2142 has exceeded the number of permissibleregistered listeners. Ignoring request to add.
at android.os.Parcel.createException(Parcel.java:2082)
at android.os.Parcel.readException(Parcel.java:2042)
at android.os.Parcel.readException(Parcel.java:1990)
at com.android.internal.telephony.ITelephonyRegistry$Stub$Proxy.listenForSubscriber(ITelephonyRegistry.java:1056)
at android.telephony.TelephonyManager.listen(TelephonyManager.java:5087)
at com.kunminx.puremusic.player.helper.PlayerCallHelper.bindCallListener(PlayerCallHelper.java:80)
at com.kunminx.puremusic.player.notification.PlayerService.onStartCommand(PlayerService.java:96)
at android.app.ActivityThread.handleServiceArgs(ActivityThread.java:4165)
at android.app.ActivityThread.access$1900(ActivityThread.java:229) 
at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1933) 
at android.os.Handler.dispatchMessage(Handler.java:107) 
at android.os.Looper.loop(Looper.java:226) 
at android.app.ActivityThread.main(ActivityThread.java:7592) 
at java.lang.reflect.Method.invoke(Native Method) 
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:539) 
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:950) 
Caused by: android.os.RemoteException: Remote stack trace:
at com.android.server.TelephonyRegistry.add(TelephonyRegistry.java:957)
at com.android.server.TelephonyRegistry.listen(TelephonyRegistry.java:671)
at com.android.server.TelephonyRegistry.listenForSubscriber(TelephonyRegistry.java:640)
at com.android.internal.telephony.ITelephonyRegistry$Stub.onTransact(ITelephonyRegistry.java:428)
at android.os.Binder.execTransactInternal(Binder.java:1024)

minSdkVersion 23 这没法用到真实项目中吧

现在minSdkVersion 一般是16 19，这23有点太高了吧

可以增加单曲播放的模式切换嘛？

横屏按返回键无法退出应用

what to pass in second arguement in efaultPlayerManager.getInstance().init(this,??);

what to pass in second arguement in efaultPlayerManager.getInstance().init(this,??);

【 提问须知 】

如有 bug，请另外 new 一个 issue ⚠️⚠️⚠️

本项目开 issue 规范：

1. 有任何 bug 都欢迎及时开 issue，我看到后予以处理。
2. 如有使用上的疑问，请先认真阅读 Readme 和源码 sample，在没有找到答案后，另外开 issue。
3. 如开 issue 是为了发表个人见解，请务必 客观、具体、严谨；严禁草率、乱入、带节奏：

务必注明观点所对应的场景，并附上完整可复现的代码，

不然缺乏一致的前提依据来有效交流！

任何缺乏实证依据和因果逻辑的泛泛而谈，都可能对其他使用者造成困扰。

在发表个人见解前，请先确保自己认真阅读过源码。这是对自己、对作者、对其他读者最起码的尊重。

写得真不错，学习了，来一波商业互推

我也撸了一个，不知道能不能入大佬法眼
StarrySky

【建议】关于PlayerController里的几个事件分发将setValue更改为postValue

这里的几个事件：

  private final MutableLiveData<ChangeMusic> changeMusicLiveData = new MutableLiveData<>();
  private final MutableLiveData<PlayingMusic> playingMusicLiveData = new MutableLiveData<>();
  private final MutableLiveData<Boolean> pauseLiveData = new MutableLiveData<>();
  private final MutableLiveData<Enum> playModeLiveData = new MutableLiveData<>();

在分发时都是使用的setValue，但是我在项目里的音频资源来自网络，在进行PlayerManager.getInstance().loadAlbum(album, 0)时会耗一些时间造成界面会卡一下，网络不好时甚至更会出现ANR，所以我将此方法放入了子线程，把里面的事件分发更改为postValue()，这样是不是更好一点，这样无论在子线程还是主线程能能使用。

测试音频

https://ws.stream.qqmusic.qq.com/C400003nGa8A3ysktw.m4a?guid=3710892&vkey=D9C0E0E81420C49A1B5346C3F6BF0A832DBE61A32F6E990C4C95A0EB0F17D91B58550AE7D018DF8EEBA47DAAE11CCE398531632F3A77D8AD&uin=626567678&fromtag=143

将移动网络调为3G测试网络慢的情况。

MediaPlayerHelper里去掉对格式的判断

  public boolean play(final String localPathOrURL) {
   //if (!checkAvalable(localPathOrURL)) {
   //   return false;
   //}
   ...
  }

MainFragment对资源进行替换：

mState.musicRequest.getFreeMusicsLiveData().observe(getViewLifecycleOwner(), dataResult -> {
      if (!dataResult.getResponseStatus().isSuccess()) return;
      TestAlbum musicAlbum = dataResult.getResult();
      for(TestAlbum.TestMusic music:musicAlbum.getMusics()){
        music.setUrl(URL);
      }
      .....
    });

如果这样，那么是否需要对格式进行判断，或者向外曝露一个额外的判断接口，是不是更好呢。