17370845950

在android开发中，若需从非activity类（如工具类、管理器类）更新ui组件（如textview），必须确保操作发生在主线程；否则需通过runonuithread等机制切换线程，否则将抛出calledfromwrongthreadexception异常。

在你的代码中，Navigation 类持有了 MapActivity 的强引用，并尝试直接访问其成员变量 mTravelTime（注：实际Activity中声明的是 mDistance，此处应为笔误，需统一命名）。这种方式语法上可行，但存在两个关键风险：

1. 线程安全性问题：updateText() 方法若在子线程（如网络回调、位置监听器的后台处理、定时任务等）中被调用，直接调用 setText() 会触发 android.view.ViewRootImpl$CalledFromWrongThreadException —— 因为 Android 的 UI 操作严格限制在主线程（UI线程）执行。

2. 内存泄漏隐患：Navigation 持有 MapActivity 的强引用，若该实例生命周期长于 Activity（例如作为单例或静态持有），会导致 Activity 无法被 GC 回收，引发内存泄漏。

✅ 正确做法：始终通过主线程安全方式更新 UI。推荐以下两种方案：

方案一：使用 Activity 的 runOnUiThread（推荐，简洁明确）

public class Navigation {
    private final Context mContext;
    private final MapActivity mMapActivity;

    public Navigation(Context context, MapActivity mapActivity) {
        this.mContext = context;
        this.mMapActivity = mapActivity;
    }

    public void updateText(String text) {
        // 确保在主线程执行
        if (mMapActivity != null && !mMapActivity.isFinishing() && !mMapActivity.isDestroyed()) {
            mMapActivity.runOnUiThread(() -> {
                if (mMapActivity.mDistance != null) {
                    mMapActivity.mDistance.setText(text);
                }
            });
        }
    }
}

方案二：使用 Handler + Looper.getMainLooper()（适用于无Activity上下文场景）

private final Handler mainHandler = new Handler(Looper.getMainLooper());

public void updateText(String text) {
    mainHandler.post(() -> {
        if (mMapActivity != null && mMapActivity.mDistance != null) {
            mMapActivity.mDistance.setText(text);
        }
    });
}

⚠️ 注意事项：

• 始终检查 Activity 是否已销毁（isFinishing() / isDestroyed()），避免空指针或已销毁Activity上更新UI；
• 避免在 Navigation 中直接暴露 MapActivity 的成员变量（如 mDistance），建议改为提供 setTextToDistance(String) 等受控方法，增强封装性；
• 更现代的替代方案：考虑使用 LiveData + ViewModel 或 StateFlow 实现跨组件状态通信，彻底解耦 UI 更新逻辑与业务类。

总结：不能无条件地直接调用 mapActivity.mDistance.setText()；必须确认调用线程——不确定时，默认按子线程处理，并强制切回主线程。这是 Android UI 编程的基本铁律。