在android开发中，若需从非activity类（如工具类、管理器类）更新ui组件（如textview），必须确保操作发生在主线程；否则需通过runonuithread等机制切换线程，否则将抛出calledfromwrongthreadexception异常。
在你的代码中，Navigation 类持有了 MapActivity 的强引用，并尝试直接访问其成员变量 mTravelTime（注：实际Activity中声明的是 mDistance，此处应为笔误，需统一命名）。这种方式语法上可行，但存在两个关键风险：
线程安全性问题：updateText() 方法若在子线程（如网络回调、位置监听器的后台处理、定时任务等）中被调用，直接调用 setText() 会触发 android.view.ViewRootImpl$CalledFromWrongThreadException —— 因为 Android 的 UI 操作严格限制在主线程（UI线程）执行。
内存泄漏隐患：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 编程的基本铁律。
