这是在 Android 应用程序上重试 API 调用的按钮的正确方法吗?

Question

我正在从事的项目从中间没有空间的API获取用户列表(仅限网络架构)。我实际上让代码按照我希望的方式工作，但我想知道这是否是正确的方法。下面我标记了我有疑问的确切位置。

我的 UI 是由 ListAdapter 支持的回收器视图。我有一个视图模型(mainViewModel)，它正在观察存储库类提供的用户列表，还有一个布尔值来显示或不显示"重试"按钮。

在主活动中.java我有

// Observing the boolean that represents if the api call failed
mainViewModel.getCallFailure().observe(this, new Observer<Boolean>() {
@Override
public void onChanged(Boolean aBoolean) {
if (aBoolean) {
tryAgainButton.setVisibility(View.VISIBLE);
} else {
tryAgainButton.setVisibility(View.GONE);
}
}
});
// Setting up onClickListener
tryAgainButton.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View v) {
mainViewModel.retryCall(); // HERE IS WHERE I'M HAVING DOUBTS
}
});

MainViewModel.java具有

public class MainViewModel extends ViewModel {
...
public LiveData<Boolean> getCallFailure(){
return userRepo.getCallFailure();
}
// THIS IS THE METHOD I'M CALLING FROM THE UI
public void retryCall(){
userRepo.getRetryCallback();
}
}

最后，存储库.java

public class Repository {
private Call<UsersResponse> retryCall;
private Callback<UsersResponse> callback;
private final MutableLiveData<Boolean> callFailure = new MutableLiveData<>();
public LiveData<List<UserModel>> getListUsers(){
final MutableLiveData<List<UserModel>> users = new MutableLiveData<>();
callFailure.setValue(false);
(Api.getClient().getUsersList()).enqueue(new Callback<UsersResponse>() {
@Override
public void onResponse(Call<UsersResponse> call, Response<UsersResponse> response) {
users.setValue(response.body().getUsersList());
callFailure.setValue(false);
}
@Override
public void onFailure(Call<UsersResponse> call, Throwable t) {
Log.d("Response GET", t.toString());
callFailure.setValue(true);
setCallback(this);
retryCall = call.clone();
}
});
return users;
}
private void setCallback(Callback<UsersResponse> usersResponseCallback) {
callback = usersResponseCallback;
}
public LiveData<Boolean> getCallFailure(){
return callFailure;
}
public void getRetryCallback(){
retryCall.enqueue(callback);
}
}

我试图只提及我认为相关的代码，我管理 ProgressDialog 的方式与我在存储库中处理布尔callFailure的方式相同，它正在创造奇迹。但老实说，在 MainActivity.java 文件中进行mainViewModel.retryCall();调用感觉有些不对劲，我真的很感激任何建议！

Answer 1

在我使用 MVVM 制作的项目中，我的活动和片段会像您公开一样通知我的 ViewModel(通过调用充当触发器的方法)并通过 LiveData 观察结果。

对我来说很奇怪的一件事是您的存储库存储了状态的一部分(重试和失败)。在我的项目中，我把这个逻辑放在 ViewModel 中，我的存储库在这里执行简单的操作(如网络调用)。但这更多的是个人选择，而不是其他任何事情。

---

编辑：这是我如何根据您的代码执行此操作(也许存在更好的方法)。

基本上，我更喜欢在 ViewModel 中管理 LiveData 对象而不是在存储库中，因为在我看来，存储库层只是应用程序和数据(在网络中或在数据库中)之间的桥梁，并且 UI 的状态(如重试按钮可见性)必须由 ViewModel 管理。

存储库.java：

public class Repository {
public void getListUsers(Callback<UserResponse> callback) {
(Api.getClient().getUsersList()).enqueue(callback);
}
}

主视图模型.java：

public class MainViewModel extends ViewModel {
...
private final MutableLiveData<Boolean> callFailure = new MutableLiveData<>();
private final MutableLiveData<List<UserModel>> users = new MutableLiveData<>();
public LiveData<Boolean> getCallFailure() {
return callFailure;
}
public LiveData<Boolean> getUsers() {
return users;
}
// THIS IS THE METHOD I'M CALLING FROM THE UI
public void retryGetUsers() {
callFailure.setValue(false);
getUsers();
}
private void getUsers() {
userRepo.getListUsers(new Callback<UserResponse>(){
@Override
public void onResponse(Call<UsersResponse> call, Response<UsersResponse> response) {
users.setValue(response.body().getUserList());
}
@Override
public void onFailure(Call<UsersResponse> call, Throwable t) {
callFailure.setValue(true);
}
});
}
}