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

[Theoneee]

这里的几个事件：

  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);
      }
      .....
    });

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

[KunMinX]

感谢你的反馈

格式可以手动通过 getFormatList 方法在外部添加

[Theoneee]

格式可以手动通过 getFormatList 方法在外部添加

/**

• 检查是否可以播放 */ private boolean checkAvalable(String path) { boolean surport = false; for (int i = 0; i < ext.length; i++) { if (path.toLowerCase().endsWith(ext[i])) { surport = true; } } if (!surport) { callBack(CallBackState.FORMATE_NOT_SURPORT, uiHolder.player); return false; } return true; }

现在是用后缀名去判断，但是有些[播放地址](https://ws.stream.qqmusic.qq.com/C400000v4WXU3vOqOt.m4a?guid=24928479&vkey=6F167DF476118C85FA803A77B590353B7A8E69E114270AA617D84C491993E49EA20BB6AED6E0EFADBD18953DAAF624360D6998E92A97F795&uin=&fromtag=143)不是以那个结尾的，所以在

if (!surport) {
   return  onCustomCheckAvailable(String url)
}

这个地方，是不是可以再一次的自行去判断。

[KunMinX]

感觉这个建议不错。加个

[Theoneee]

感觉这个建议不错。加个

感谢~

[Theoneee]

用这个框架写了个APP：

HifiNi