Android TV无限轮播Fragment应用因内存异常持续崩溃问题求助

嘿，我看你遇到了个挺棘手的问题——测试环境（模拟器和几台设备）都正常，但Android TV真机跑久了就因内存异常崩溃。结合你的代码，我整理了几个可能的原因和对应的修复方案，应该能帮到你：

1. 修复ViewPager的Fragment回收逻辑 #

你的SliderAdapter里getItemPosition返回POSITION_UNCHANGED，这会让ViewPager认为所有Fragment都不需要更新或回收，轮播次数多了之后，堆里会攒一堆没销毁的Fragment实例。Android TV的内存管理比手机严格得多，时间一长就容易触发内存异常。另外直接引用DispolyActivity的静态数据也暗藏风险：

@Override
public int getItemPosition(@NonNull Object object) {
    // 改成POSITION_NONE，让ViewPager在页面切换或数据更新时正确回收旧Fragment
    return POSITION_NONE;
}

@NonNull
@Override
public Fragment getItem(int position) {
    // 别直接用Activity的静态列表，用适配器自己持有的数据更安全
    PromotionalBanner promotionalBanner = promotionalBannerArrayList.get(position);
    Log.e("positiondata", "dataPost: "+position );
    return BannerFragment.getInstantFragment(promotionalBanner, position, promotionalBannerArrayList);
}

2. 完善ExoPlayer的生命周期管理 #

Android TV的生命周期触发逻辑和手机/模拟器不太一样，比如应用切后台时可能直接触发onStop而非onPause。你现在只在onPause和slideNextItem里释放Player，得补上onStop的处理，确保Player被彻底释放：

在BannerFragment中添加：

@Override
public void onStop() {
    super.onStop();
    if (simpleExoPlayer != null) {
        simpleExoPlayer.release();
        simpleExoPlayer = null;
    }
}

另外要注意：所有触发页面切换的场景（比如视频播放结束、图片加载失败），都要检查Player是否存在并释放，别漏了任何一处。

3. 取消Glide未完成的加载请求 #

当Fragment销毁时，如果Glide还有没完成的图片加载请求，会持有Fragment的上下文引用，导致内存无法正常释放。在BannerFragment的onDestroyView里加一行代码取消请求：

@Override
public void onDestroyView() {
    super.onDestroyView();
    // 取消Glide的未完成加载，避免持有上下文引用
    Glide.with(this).clear(vb.imageView);
    // 确保ViewBinding被置空，切断视图引用
    vb = null;
}

4. 彻底清理CountDownTimer的引用 #

你现在在onPause里取消了timer，但在onDestroy里也要确保timer被清空，避免它持有Fragment的引用：

@Override
public void onDestroy() {
    super.onDestroy();
    Log.e("Destory", "onDestroy: " );
    if (timer != null) {
        timer.cancel();
        timer = null;
    }
    if (playerListener != null) {
        playerListener = null;
    }
    if (requestListener != null) {
        requestListener = null;
    }
}

5. 别直接引用Activity的静态视图 #

你的goToNextItem方法直接调用DispolyActivity.vb.viewPager，这是个强引用，会导致Activity就算被销毁也没法被GC回收（因为Fragment还拿着它的引用）。换成接口回调的方式更安全：

首先在BannerFragment里定义回调接口：

public interface OnBannerSlideListener {
    void onSlideToNext(int nextPosition);
}

private OnBannerSlideListener slideListener;

@Override
public void onAttach(@NonNull Context context) {
    super.onAttach(context);
    // 要求宿主Activity实现这个接口
    try {
        slideListener = (OnBannerSlideListener) context;
    } catch (ClassCastException e) {
        throw new ClassCastException(context.toString() + " 必须实现OnBannerSlideListener接口");
    }
}

@Override
public void onDetach() {
    super.onDetach();
    slideListener = null;
}

然后修改goToNextItem方法：

private void goToNextItem(int nextPosition) {
    if (slideListener != null) {
        slideListener.onSlideToNext(nextPosition);
    }
}

最后让你的DispolyActivity实现这个接口：

public class DispolyActivity extends AppCompatActivity implements BannerFragment.OnBannerSlideListener {
    // ... 其他代码

    @Override
    public void onSlideToNext(int nextPosition) {
        vb.viewPager.setCurrentItem(nextPosition);
    }
}

6. 补全轮播循环的边界处理 #

你现在的轮播逻辑里，当nextPosition超过列表长度时没有处理，会直接引发索引越界异常，这也可能是崩溃的诱因之一。添加上循环逻辑：

private void goToNextItem(int nextPosition) {
    // 取模实现无限循环
    int actualNextPosition = nextPosition % promotionalBannerArrayList.size();
    if (slideListener != null) {
        slideListener.onSlideToNext(actualNextPosition);
    }
}

额外排查建议 #

• 拿Android TV真机用Android Studio Profiler的内存快照功能，每次轮播几次就拍一张快照，对比BannerFragment、SimpleExoPlayer的实例数量，看它们是否被正确回收。
• 检查PromotionalBanner这个Parcelable对象，别让它持有大对象或者上下文引用，尽量保持轻量级。

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