如何在ViewModel中正确调用QueryUtils.fetchData并解决数据未显示问题

你的推测完全正确——问题就出在异步数据还没获取完成，LiveData就已经把空/未初始化的数据推给了界面，而且当数据真正获取到后，没有通知LiveData去更新UI。我来帮你修正代码逻辑：

问题根源拆解 #

1. 你在ViewModel构造时启动了异步线程去拉取数据，但线程是后台执行的，此时earthquakes变量还没被赋值
2. fetchEarthquakes()方法直接把这个空的earthquakes设置给earthquakesData并返回，导致界面Observer先收到了空数据
3. 当线程完成数据获取后，你没有主动触发LiveData的更新通知，所以界面永远不知道新数据已经来了

修改后的ViewModel代码 #

public class EarthquakeViewModel extends AndroidViewModel {
    private MutableLiveData<List<Earthquake>> earthquakesData;

    public EarthquakeViewModel(@NonNull Application application) {
        super(application);
        Log.i("EarthquakeViewModel", "EarthquakeViewModel constructor entered");
        earthquakesData = new MutableLiveData<>();
        // 启动异步数据获取
        startEarthquakeFetch();
        Log.i("EarthquakeViewModel", "EarthquakeViewModel constructor finished");
    }

    // 给UI层暴露LiveData，不需要手动设置值，数据更新后自动通知
    public LiveData<List<Earthquake>> getEarthquakes() {
        return earthquakesData;
    }

    private void startEarthquakeFetch() {
        new Thread(() -> {
            // 后台线程调用QueryUtils获取数据
            List<Earthquake> fetchedData = QueryUtils.fetchData(REQUEST_URL);
            // 注意：子线程更新LiveData必须用postValue()，setValue()只能在主线程用
            earthquakesData.postValue(fetchedData);
        }).start();
    }
}

修改后的MainActivity代码 #

public class MainActivity extends AppCompatActivity {
    private EarthquakeAdapter mEarthquakeAdapter;
    private EarthquakeViewModel aEarthquakeViewModel;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        
        // 这里假设你已经完成了RecyclerView和Adapter的初始化（比如绑定RecyclerView、设置LayoutManager等）
        
        aEarthquakeViewModel = ViewModelProviders.of(this).get(EarthquakeViewModel.class);
        aEarthquakeViewModel.getEarthquakes().observe(this, earthquakes -> {
            if (earthquakes != null && !earthquakes.isEmpty()) {
                mEarthquakeAdapter.clear();
                mEarthquakeAdapter.addAll(earthquakes);
            }
        });
    }
}

关键修改点说明 #

1. 正确触发LiveData更新：在后台线程获取到数据后，调用postValue()通知LiveData更新，这样界面的Observer会收到最新的数据集，进而刷新Adapter
2. 规范命名与职责划分：把原来的fetchEarthquakes()改成getEarthquakes()，ViewModel只负责暴露数据观察接口，数据获取逻辑内部处理，符合MVVM的职责分离原则
3. 面向接口编程：用List<Earthquake>代替ArrayList<Earthquake>，让代码更灵活，后续如果换其他List实现也不需要改动UI层代码
4. 增强空数据判断：在Observer中增加!earthquakes.isEmpty()的判断，避免空列表时做无用的Adapter操作

进阶优化建议 #

如果你的项目可以使用更现代的异步处理方案，推荐用Kotlin协程（如果已经迁移到Kotlin）或者ExecutorService（Java中比直接用Thread更规范），示例如下（Java版ExecutorService）：

private void startEarthquakeFetch() {
    ExecutorService executor = Executors.newSingleThreadExecutor();
    Handler mainHandler = new Handler(Looper.getMainLooper());

    executor.execute(() -> {
        List<Earthquake> fetchedData = QueryUtils.fetchData(REQUEST_URL);
        // 切换回主线程更新LiveData
        mainHandler.post(() -> earthquakesData.setValue(fetchedData));
    });
}

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