Android Fragment中用Thread定时执行代码触发空指针崩溃问题

为什么会出现这个NullPointerException？ #

你的错误日志已经把问题点得明明白白：Attempt to invoke virtual method 'void android.support.v4.app.FragmentActivity.runOnUiThread(java.lang.Runnable)' on a null object reference——简单说就是你调用getActivity()时，它返回了null，然后你试图用这个空对象去调用runOnUiThread方法。

至于为什么getActivity()会变成null，核心原因很清晰：

• 你在onCreateView里启动了一个独立的Thread，这个线程会每隔120秒执行一次循环。但Thread是完全脱离Fragment生命周期的——哪怕你的Fragment已经从Activity上分离（比如用户切换了页面、Activity被销毁/重建），这个线程依然会自顾自地跑下去。
• 虽然你启动线程前判断了getActivity() != null，但这只是启动瞬间的状态。等到120秒后线程再次执行时，Fragment可能已经和Activity解绑了，此时getActivity()自然就返回null了。

解决方案：绑定生命周期+安全校验 #

我们需要从两个维度修复问题：让任务随Fragment生命周期停止，每次调用getActivity()前做安全校验。下面给你两种可行的修改方案：

方案1：改进现有Thread的实现

这种方案基于你现有的代码做最小改动：

1. 把Thread改成成员变量，方便在生命周期方法中控制它：

public class MainReminderFragment extends Fragment {
    // ... 其他成员变量
    private Thread updateThread; // 新增线程成员变量
    // ...
}

2. 修改线程的循环逻辑，用while(isRunning)代替固定次数的for循环，并且每次调用runOnUiThread前做双重校验：

// 替换你原来的线程启动代码
updateThread = new Thread() {
    @Override
    public void run() {
        while (isRunning) { // 只要isRunning为true就持续循环
            try {
                // 双重校验：先判断Fragment是否还附着在Activity上，再判断getActivity不为null
                if (isAdded() && getActivity() != null) {
                    getActivity().runOnUiThread(() -> getData());
                }
                Thread.sleep(120000); // 等待120秒
            } catch (InterruptedException e) {
                e.printStackTrace();
                break; // 线程被中断后退出循环
            }
        }
    }
};
updateThread.start();

3. 在Fragment销毁时停止线程，避免内存泄漏和空指针：

@Override
public void onDestroyView() {
    super.onDestroyView();
    isRunning = false; // 标记停止循环
    if (updateThread != null && updateThread.isAlive()) {
        updateThread.interrupt(); // 中断线程，让sleep抛出异常退出循环
    }
}

方案2：用Handler+Runnable替代Thread（更推荐）

这种方式更贴合Android的生命周期，也更安全，避免手动管理线程的麻烦：

1. 添加Handler和Runnable成员变量：

public class MainReminderFragment extends Fragment {
    // ... 其他成员变量
    private Handler updateHandler = new Handler();
    private Runnable updateRunnable = new Runnable() {
        @Override
        public void run() {
            // 同样做安全校验
            if (isAdded() && getActivity() != null) {
                getData();
            }
            // 如果还在运行，就延迟120秒再次执行
            if (isRunning) {
                updateHandler.postDelayed(this, 120000);
            }
        }
    };
    // ...
}

2. 在Fragment可见时启动任务，不可见时停止：

@Override
public void onStart() {
    super.onStart();
    isRunning = true;
    updateHandler.post(updateRunnable); // 立即执行一次，然后每隔120秒重复
}

@Override
public void onStop() {
    super.onStop();
    isRunning = false;
    updateHandler.removeCallbacks(updateRunnable); // 移除所有待执行的任务
}

额外优化建议 #

• 你的getData()方法里有个空的catch (Exception e)，建议至少添加e.printStackTrace()，否则出现API调用异常时你完全看不到错误信息，排查问题会非常困难。
• 避免在onCreateView里启动重复任务——如果Fragment因为屏幕旋转重建，onCreateView会被多次调用，可能启动多个线程/任务。方案2里用onStart()/onStop()绑定生命周期就不会有这个问题。

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