Android MediaPlayer未按预期释放致音频播放异常求助

咱们先梳理你代码里几个关键的问题点，这些正是导致多次点击后音频停止播放的核心原因：

1. Fragment点击事件里的MediaPlayer逻辑冗余且存在资源泄漏风险 #

看你Fragment中的点击处理代码：

listView.setOnItemClickListener(new AdapterView.OnItemClickListener() { 
    @Override 
    public void onItemClick(AdapterView<?> adapterView, View view, int i, long l) { 
        if(MainActivity.mediaPlayer != null && MainActivity.mediaPlayer.isPlaying()){ 
            MainActivity.mediaPlayer.release(); 
            MainActivity.mediaPlayer = new MediaPlayer(); 
        } 
        MainActivity.mediaPlayer = MediaPlayer.create(getContext(), MainActivity.hashMap.get(adapterView.getItemAtPosition(i))); 
        MainActivity.mediaPlayer.start(); 
    } 
});

这里有两个明显问题：

• 你在release()后手动new了一个MediaPlayer，但紧接着就被MediaPlayer.create()的返回值覆盖，这个手动创建的实例完全无用，还会造成不必要的内存占用。
• MediaPlayer.create()方法本身会自动创建并初始化一个新的MediaPlayer实例，不需要提前手动实例化。而且如果之前的实例已经被释放但未置为null，后续的非空判断会失效。

2. MainActivity生命周期方法里的MediaPlayer处理不严谨 #

在onPause()中你调用了stop()和release()，但没有将mediaPlayer置为null。这会导致后续判断mediaPlayer != null时，实际上它已经是一个被释放的无效实例，调用isPlaying()等方法会直接抛出异常。

另外，onResume()里直接new MediaPlayer的操作也没必要，应该等到需要播放时再创建实例，避免提前占用资源。

3. 未处理MediaPlayer的播放完成事件 #

你的猜测是对的——如果音频播放完成后没有及时释放MediaPlayer资源，这些闲置实例会持续占用系统资源，多次积累后就会导致后续无法正常创建新实例，最终出现停止播放的问题。

修复后的代码示例 #

第一步：修正MainActivity里的MediaPlayer生命周期处理 #

//... 
public static MediaPlayer mediaPlayer; 
//... 

@Override 
protected void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    //... Layout stuff 
    // 初始化时先置为null，避免后续判断出错
    mediaPlayer = null; 
    //Setting up fragment stuff 
    //... 
} 

@Override 
public void onPause(){ 
    super.onPause(); 
    // 统一封装释放逻辑，避免重复代码
    releaseMediaPlayer(); 
} 

@Override 
public void onResume(){ 
    super.onResume(); 
    // 无需提前实例化，播放时再创建
    mediaPlayer = null; 
}

// 新增统一的MediaPlayer释放方法
public static void releaseMediaPlayer() {
    if (mediaPlayer != null) {
        if (mediaPlayer.isPlaying()) {
            mediaPlayer.stop();
        }
        mediaPlayer.release();
        mediaPlayer = null; // 必须置为null，避免后续判断无效实例
    }
}

第二步：修正Fragment里的点击事件逻辑 #

listView.setOnItemClickListener(new AdapterView.OnItemClickListener() { 
    @Override 
    public void onItemClick(AdapterView<?> adapterView, View view, int i, long l) { 
        // 先释放之前的MediaPlayer资源
        MainActivity.releaseMediaPlayer(); 
        // 创建新实例并准备播放
        MainActivity.mediaPlayer = MediaPlayer.create(getContext(), MainActivity.hashMap.get(adapterView.getItemAtPosition(i))); 
        if (MainActivity.mediaPlayer != null) {
            // 设置播放完成监听，播放结束自动释放资源
            MainActivity.mediaPlayer.setOnCompletionListener(new MediaPlayer.OnCompletionListener() {
                @Override
                public void onCompletion(MediaPlayer mp) {
                    MainActivity.releaseMediaPlayer();
                }
            });
            MainActivity.mediaPlayer.start(); 
        }
    } 
});

额外优化建议 #

• 尽量不要使用static的MediaPlayer实例，静态实例会持有Activity引用，容易引发内存泄漏。更好的方式是在Fragment中持有实例，或者用ViewModel来管理它的生命周期。
• 增加容错处理，比如MediaPlayer.create()可能返回null（如资源不存在），此时要避免空指针异常。
• 如果需要频繁切换音频或处理复杂播放场景，建议用ExoPlayer替代MediaPlayer，它的生命周期管理和资源处理更健壮。

内容的提问来源于stack exchange，提问作者DChalo123