Retrofit 2 и RXJava как правильно обработать исключения?

Question

[Дмитрий Серков]

Здравствуйте.

Есть сайт, где авторизация происходит путем Post запроса с передачей логина и пароля.
Правильно ли так обрабатывать исключения в связке Retrofit и RXJava:
apiService.login возвращает Observable ResponseBody

apiService.login(login, pass)
                        .map(responseBody -> {
                            try {
                                return responseBody.string();
                            } catch (IOException e) {
                                e.printStackTrace();
                            }
                            return "";
                        })
                        .subscribeOn(Schedulers.newThread())
                        .observeOn(AndroidSchedulers.mainThread())
                        .subscribe(s -> System.out.println(s));

интерфейс

@FormUrlEncoded
    @POST("/login.php")
    Observable<ResponseBody> login(
            @Field("login") String login,
            @Field("pass") String pass);

public SiteService(Context context) {

        cookieJar = new PersistentCookieJar(new SetCookieCache(), new SharedPrefsCookiePersistor(context));

        okHttpClient = new OkHttpClient.Builder()
                .cookieJar(cookieJar)
                .build();

        client = new Retrofit.Builder()
                .baseUrl(BASE_URL)
                .client(okHttpClient)
                .addCallAdapterFactory(RxJavaCallAdapterFactory.create())
                .build();


        apiService = client.create(SiteApi.class);
    }
    public Observable<ResponseBody> login(String login, String pass){

        return apiService.login(login, pass);
    }

Спасибо.

Answer 1

[Юрий]

неа, неправильно. Добавь onError() после вызова login. Почитать можно тут

Comments

[Дмитрий Серков]

Интересует не login, а именно responseBody.string(). Если убрать блок try catch, подсвечивается красным, добавлял после .onErrorReturn()
подсветка остается.

[Юрий]

в IDEA подсветка? а что говорит подсказка? если проверка на null - так сделаю ее. Смотри IOException тут ничего не словит, сам апи колл происходит в другом месте...

[Дмитрий Серков]

Юрий: unhandled exception: java.io.IOException в IDEA подсвтка.
Вот и хочется узнать есть ли возможность избавиться от try catch средствами rxjava.

[Юрий]

Давай сигнатуру метода, где ты определил функцию login(). Стойкое ощущение, что ты в методе написал throws IOException;

[Дмитрий Серков]

Юрий: добавил в начале вопроса

[Юрий]

блин, чудеса :) может стринг кидает эксепшн... залей на гитхаб?

[Дмитрий Серков]

Юрий: как я понял ResponseBody это поток, поэтому при преобразовании в string используем try catch. И по видимому в rxjava без try catch не обработать.

[Юрий]

нет, это не поток. хочешь посмотреть что это - ctrl+клик на класс. Это просто десериализованный в дженерик ответ с сервера.

[Юрий]

давай гитхаб, посмотрю как так.

[Дмитрий Серков]

Юрий: public abstract class ResponseBody implements Closeable {
/** Multiple calls to {@link #charStream()} must return the same instance. */

[Юрий]

ну да,public final String string() throws IOException {
return new String(bytes(), charset().name());
}.
Это не rx поток - и рхжава с ним сама по себе ничего сделать не может. Только писать враппер на ResponseBody.

[Дмитрий Серков]

Юрий: уже пишу) спасибо.

Answer 2

[davidnum95]

Подписчику ошибка возвращается в метод OnError(Exception throwable), соотвественно:

apiService.login(login, pass)
                        .map(responseBody -> {
                            try {
                                return responseBody.string();
                            } catch (IOException e) {
                                e.printStackTrace();
                            }
                            return "";
                        })
                        .subscribeOn(Schedulers.newThread())
                        .observeOn(AndroidSchedulers.mainThread())
                        .subscribe(s -> System.out.println(s), throwable -> {
                                        // тут обработка ошибок
});

Comments

[Дмитрий Серков]

С самим login все ок, и там обработка ошибок это понятно. Меня интересует обработка map ответа от сервера, а именно responseBody.string(), она и выдает исключение, которое помещаем в try catch, в связи с этим и интересуюсь, есть ли в rxjava обработка этого исключения без использования try catch. Если в map просто 1/0, что нельзя (деление на ноль), то можно использовать onErrorReturn. А вот как с try catch быть?

[davidnum95]

Дмитрий: а зачем в map это обрабатывать? Все исключения которые будут в цепочки преобразований попадут к подписчику в onError

[Дмитрий Серков]

davidnum95: если убрать try catch, в IDEA идет подсветка красным и соответственно не компилируется