为什么 LiveData setValue 或 PostValue 在视图中只触发一次 onChange？

Question

LiveData setValue 应该触发了 Activity 中的 onChanged 方法，但是它仅在第一次调用，在我尝试进行分页之后，它会中断并且不再调用 onChanged，尽管我的响应确实成功并且我在日志。setValue/postValue 有什么问题？这是一个错误吗？我应该自己实现观察者模式吗？那么使用 LiveData 有什么意义呢？我的寻呼不仅仅适用于这已经 2-3 天了.....

1. 主活动类

    public class MainActivity extends AppCompatActivity 
    private MutableLiveData<List<Photo>> mLivePhotos;
    // some code...
   
    @Override
    protected void onCreate(Bundle savedInstanceState) {
       mLivePhotos = loadData();
       mLivePhotos.observe(this, photos -> {
           Log.d(TAG, "onChanged!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
           mProgressBar.setVisibility(View.GONE);
           mPhotos = photos;
           if (mIsInitialCall) {
               initiateAdapter();
               mIsInitialCall = false;
           } else {
               mAdapter.updateList(mPhotos.subList(mPageNumber, mPageNumber + 10));
           }
       });
   
       mRecyclerView.addOnScrollListener(new RecyclerView.OnScrollListener() {
           @Override
           public void onScrolled(RecyclerView recyclerView, int dx, int dy) {
               int lastPosition = 
   mLayoutManager.findLastCompletelyVisibleItemPosition();
               Log.d(TAG, "onScrolled - lastPosition: " + lastPosition);
               if (lastPosition == mLayoutManager.getItemCount() - 1) {
                   Log.d(TAG, "onScrolled - End of list?");
                   loadData();
               }
           }
       });
   }
   
   private MutableLiveData<List<Photo>> loadData() {
       Log.d(TAG, "loadData");
       if (mArticleViewModel == null) return null;
       mPageNumber += 10;
       mProgressBar.setVisibility(View.VISIBLE);
       return mArticleViewModel.loadPhotos();
   }
   

2. 视图模型

   public class ArticleViewModel extends ViewModel {
       private MutableLiveData<List<Photo>> photos;
       private ArticleRepository articleRepository;
   
       public MutableLiveData<List<Photo>> loadPhotos() {
           Log.d(TAG, "getArticleList");
           //TODO; add Dagger 2
           articleRepository = new ArticleRepository();
           photos = articleRepository.getPhotos();
           return photos;
       }
   

3. 存储库

   public class ArticleRepository {
   
       public MutableLiveData<List<Photo>> getPhotos() {
           final MutableLiveData<List<Photo>> result = new MutableLiveData<>();
           Log.d(TAG, "getResults");
           ApiService.getService().getPhotos().enqueue(new Callback<List<Photo>>() {
           @Override
           public void onResponse(Call<List<Photo>> call, Response<List<Photo>> response) {
               Log.d(TAG, "onResponse");
               if (response.isSuccessful()) {
                   Log.d(TAG, "isSuccessful");
                   result.postValue(response.body());
               }
           }
   
           @Override
           public void onFailure(Call<List<Photo>> call, Throwable t) {
               Log.d(TAG, "onFailure: " + t.getMessage() + "\n" +  t.getStackTrace());
           }
       });
       return result;
   }
   

Answer 1

Activity 不应该有任何 MutablieLiveData 成员变量，它应该在 ViewModel 中。

它只在第一次工作的原因是因为你第一次观察到它通知改变的东西，但是因为你的安排不正确，它永远不会再次更新。也就是说，因为使用一组新的 MutableLiveData 在您的 ViewModel 中再次重新创建了 ArticleRepository，所以您订阅的前一个不再相关 - 您只订阅一次onCreate()。

您应该将绑定与异步任务分开，因为loadData()它们不是一回事。绑定是您在开始收集 MutableLiveData 时所做的事情（您在 loadData 中所做的事情），但是一旦完成，您就不应该再这样做了。

我还注意到您实际上在模型中有 LiveData，不建议这样做，因为它打破了模式并可能带来其他问题。应该是 ViewModel 准备演示文稿，而不是 Repository。由于您目前已经配置了一些东西，因此您的存储库也可以称为 ViewModel。相反，您应该做的是使用 observables 通知 ViewModel 一个新批次，以发布或处理可能发生的错误。

研究这个例子：https : //developer.android.com/topic/libraries/architecture/viewmodel

请注意，loadUsers()在getUsers()调用时完成一次。这就是将 Activity 绑定到 ViewModel 的原因。但是loadUsers()可以稍后再做，并且应该将更改发布到 ViewModel 中的 LiveData。

Answer 2

您不应该new MutableLiveData()每次加载更多照片时，而是调用postValue单个MutableLiveData对象。示例代码如下：

视图模型：

public class ArticleViewModel extends ViewModel {
    private MutableLiveData<List<Photo>> photos;

    public MutableLiveData<List<Photo>> getPhotoList() {
        Log.d(TAG, "loadData");
        if (photos == null) {
            photos = new MutableLiveData<>();
            loadPhotos();
        }
        return photos;
    }

    public void loadPhotos() {
        Log.d(TAG, "getArticleList");

        ApiService.getService().getPhotos().enqueue(new Callback<List<Photo>>() {
            @Override
            public void onResponse(Call<List<Photo>> call, Response<List<Photo>> response) {
                Log.d(TAG, "onResponse");
                if (response.isSuccessful()) {
                    Log.d(TAG, "isSuccessful");
                    List<Photo> morePhotos = response.body();
                    List<Photo> allPhotos = new ArrayList<>();
                    allPhotos.addAll(photos.getValue());
                    allPhotos.addAll(morePhotos);
                    photos.postValue(allPhotos);   // <--- post change here
                }
            }

            @Override
            public void onFailure(Call<List<Photo>> call, Throwable t) {
                Log.d(TAG, "onFailure: " + t.getMessage() + "\n" +  t.getStackTrace());
            }
        });
    }
}

主要活动：

public class MainActivity extends AppCompatActivity 
    private ArticleViewModel mArticalViewModel;

    // some code...

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        mArticalViewModel = ViewModelProviders.of(this).get(ArticleViewModel.class);
        mArticleViewModel.getPhotoList().observe(this, photos -> {
            Log.d(TAG, "onChanged!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!");
        mProgressBar.setVisibility(View.GONE);
        mPhotos = photos;
        if (mIsInitialCall) {
            initiateAdapter();
            mIsInitialCall = false;
        } else {
            mAdapter.updateList(mPhotos.subList(mPageNumber, mPageNumber + 10));
        }
    });

    mRecyclerView.addOnScrollListener(new RecyclerView.OnScrollListener() {
        @Override
        public void onScrolled(RecyclerView recyclerView, int dx, int dy) {
            int lastPosition = 
mLayoutManager.findLastCompletelyVisibleItemPosition();
            Log.d(TAG, "onScrolled - lastPosition: " + lastPosition);
            if (lastPosition == mLayoutManager.getItemCount() - 1) {
                Log.d(TAG, "onScrolled - End of list?");
                mArticleViewModel.loadPhotos(); // <--- changed here
            }
        }
    });
}