我在Android应用程序中使用RxJava进行Retrofit进行通信,并且必须处理从似乎正常的HTTP响应(状态200代码)解析响应时出错。
我还使用retryWhen
运算符实现了一种处理错误的方法,该运算符连接到用户的输入以决定是否重试。这可以通过重新订阅原始Observable
来实现。
我尝试过的第一种方法就是这样:
services.getSomething()
.map(response -> {
if (checkBadResponse(response)) {
throw new RuntimeException("Error on service");
} else {
return parseResponse(response);
}
}).retryWhen(this::shouldRetry);
这样就不会再次调用该服务。 retryWhen
运算符似乎无法重新订阅服务的Observable
。
最终工作结果是实现了另一个不向前发送onCompleted
的运营商,并将其与lift
一起使用,如下所示:
public class CheckResponseStatus<T> implements Observable.Operator<ResponsePayload<T>, ResponsePayload<T>> {
@Override
public Subscriber<? super ResponsePayload<T>> call(Subscriber<? super ResponsePayload<T>> subscriber) {
return new Subscriber<ResponsePayload<T>>() {
private boolean hasError = false;
@Override
public void onCompleted() {
if (!hasError)
subscriber.onCompleted();
}
@Override
public void onError(Throwable e) {
hasError = true;
subscriber.onError(e);
}
@Override
public void onNext(ResponsePayload<T> response) {
if (response.isOk()) {
subscriber.onNext(response);
} else {
hasError = true;
subscriber.onError(new RuntimeException(response.getMessage()));
}
}
};
}
}
使用它像:
services.getSomething()
.lift(new CheckResponseStatus())
.map(response -> parseResponse(response))
.retryWhen(this::shouldRetry);
这是处理它的正确方法还是有更简单,更好的方法?
答案 0 :(得分:4)
它看起来像rx-java
实施中的错误。无论如何,从map
函数抛出异常是一件坏事,因为函数应该是纯粹的(例如没有副作用)。在您的情况下,您应该使用flatMap
运算符:
services.getSomething()
.flatMap(response -> {
if (checkBadResponse(response)) {
return Observable.<ResponseType>error(new RuntimeException("Error on service"));
} else {
return Observable.<ResponseType>just(parseResponse(response);
}
}).retryWhen(this::shouldRetry);
上面的代码按预期工作,如果发生错误,实际上会重试请求。