Как правильно передать информацию об ошибки из ViewModel в Activity?

Question

[Alexey_soloviev]

Подскажите как правильно передать информацию об ошибки из ViewModel в Activity? Передать callBack в ViewModel, это правильное решение?

public class MainViewModel extends ViewModel {

    private final MutableLiveData<ArrayList<UserObject>> liveData = new MutableLiveData<>();

    public LiveData<ArrayList<UserObject>> getLiveData() {
        return liveData;
    }

    public void getUserList() {
        new UserRepository().getUserList(new RepositoryCallBack.UserGetList() {
            @Override
            public void onResponse(ArrayList<UserObject> userObject) {
                liveData.setValue(userObject);
            }

            @Override
            public void onError(int statusCode) {
                //Информациб об ошибки необходимо передать в активити (View)
            }
        });
    }

}

Answer 1

[Денис Загаевский]

Я бы сказал, точно так же, как и информацию об успехе.
Был бы нормальный котлин, я бы посоветовал завернуть этот самый UserObject в sealed класс:

sealed class Result {
    data class Ok(val data: UserObject): Result()
    data class Error(val statusCode: Int): Result()
}

С джавой будет многословно, но тоже примерно это можно изобразить.

Comments

[Alexey_soloviev]

Правильно ли я понял, что репозиторий и liveData должен возвращать допустим не ArrayList, а некий класс в котором допустим будет состояние загрузки (загружается, загруженно, ошибка), объект ну и так далее в зависимости от потребностей? С точке зрения архитектуры, это правильное решение?

[Денис Загаевский]

Alexey_soloviev, да, так можно сделать.

[Alexey_soloviev]

Денис Загаевский, спасибо большое за ответ, но мне кажется должно быть решение лучше этого.

[Денис Загаевский]

Alexey_soloviev, ну как найдёшь, скидывай.

[Alexey_soloviev]

Денис Загаевский, ещё не подскажешь, если придерживаться такого решения и возвращать в liveData pojo объект, у меня получается такой код. Это плохой код?

private void initData() {
        LiveData<UserListResponse> liveData = mainViewModel.getLiveData();
        liveData.observe(this, this::rendering);
        mainViewModel.getUserList();
    }

    private void rendering(UserListResponse userListResponse) {
        switch (userListResponse.getStatus()) {
            //Запрос к API выполнен успешно
            case 1:
                object.addAll(userListResponse.getObject());
                mProgressBar.setVisibility(View.GONE);
                mAdapter.notifyDataSetChanged();
                break;

            //При обращение к API произошла ошибка
            case 2:
                mProgressBar.setVisibility(View.GONE);
                //Отобразить информацию об ошибке
                break;

            //Ожидания ответа от сервера
            default:
                mProgressBar.setVisibility(View.VISIBLE);
        }
    }

[Денис Загаевский]

Alexey_soloviev, слушай, ну нет, мне это не нравится. Я за подход, основанный на типах.

[Alexey_soloviev]

Денис Загаевский, спасибо, мне тоже не нравится. Просто не понимаю как сделать лучше.

[Денис Загаевский]

Alexey_soloviev, делаешь двух финальных наследников абстрактного класса, у которого package-private конструктор. Это будет максимально близко к запечатанной иерархии котлина. В месте использования сравниваешь по instanceOf, и кастуешь.

[Alexey_soloviev]

Денис Загаевский, что-то наподобие этого должно получиться или это все равно плохо?

private void initData() {
        LiveData<Response> liveData = mainViewModel.getLiveData();
        liveData.observe(this, this::rendering);
        mainViewModel.getUserList();
    }

    private void rendering(Response response) {
        if(response instanceof ResultOk)
            resultOk(response);
        else
            resultError(response);
    }

    private void resultOk(Response response) {
        object.clear();
        object.addAll(((ResultOk) response).getResponseObject());
        mProgressBar.setVisibility(View.GONE);
        mAdapter.notifyDataSetChanged();
    }

    private void resultError(Response response) {
        mProgressBar.setVisibility(View.GONE);
    }

[Денис Загаевский]

Alexey_soloviev, да, типа такого. Только ещё лучше, чтобы resultOk/Error принимали сразу ResultOk/ResultError.

[Alexey_soloviev]

Денис Загаевский, так или я неправильно Вас понял?

private void rendering(Response response) {
        if(response instanceof ResultOk)
            resultOk(((ResultOk) response));
        else
            resultError(((ResultError) response));
    }

[Денис Загаевский]

Alexey_soloviev, да