假设我有以下RxJava代码(访问数据库,但确切的用例无关紧要):
public Observable<List<DbPlaceDto>> getPlaceByStringId(final List<String> stringIds) {
return Observable.create(new Observable.OnSubscribe<List<DbPlaceDto>>() {
@Override
public void call(Subscriber<? super List<DbPlaceDto>> subscriber) {
try {
Cursor c = getPlacseDb(stringIds);
List<DbPlaceDto> dbPlaceDtoList = new ArrayList<>();
while (c.moveToNext()) {
dbPlaceDtoList.add(getDbPlaceDto(c));
}
c.close();
if (!subscriber.isUnsubscribed()) {
subscriber.onNext(dbPlaceDtoList);
subscriber.onCompleted();
}
} catch (Exception e) {
if (!subscriber.isUnsubscribed()) {
subscriber.onError(e);
}
}
}
});
}
鉴于此代码,我有以下问题:
如果有人从此方法返回的observable取消订阅(在之前的订阅之后),该操作是否是线程安全的?那么我的'isUnsubscribed()'检查在这个意义上是否正确,无论安排如何?
是否有更简洁的方法用更少的样板代码检查未订阅状态,而不是我在这里使用的?我在框架中找不到任何东西。我认为SafeSubscriber解决了订阅者取消订阅时不转发事件的问题,但显然没有。
答案 0 :(得分:8)
该操作是否是线程安全的?
是。您正在接收一个rx.Subscriber,它(最终)检查在订阅者的订阅被取消订阅时设置为true的volatile布尔值。
更简洁的方法,用更少的样板代码来检查未订阅状态
为此用例创建了SyncOnSubscribe
和AsyncOnSubscribe
(可用作版本1.0.15的@Experimental
api)。它们是调用Observable.create
的安全替代方案。这是同步案例的一个(人为的)例子。
public static class FooState {
public Integer next() {
return 1;
}
public void shutdown() {
}
public FooState nextState() {
return new FooState();
}
}
public static void main(String[] args) {
OnSubscribe<Integer> sos = SyncOnSubscribe.createStateful(FooState::new,
(state, o) -> {
o.onNext(state.next());
return state.nextState();
},
state -> state.shutdown() );
Observable<Integer> obs = Observable.create(sos);
}
请注意,SyncOnSubscribe
next函数不允许每次迭代多次调用observer.onNext
,也不能同时调用该观察者。以下是指向SyncOnSubscribe
分支头部的1.x
implementation和tests的几个链接。它的主要用途是简化编写同步和onNext下游迭代或解析数据的observable,但在支持背压的框架中执行此操作,并检查是否取消订阅。基本上你会创建一个next
函数,每当下游运算符需要一个新的数据元素onNexted时就会调用它。你的下一个函数可以调用onNext 0或1次。
AsyncOnSubscribe
旨在为异步操作的可观察源(例如外线呼叫)提供良好的背压。您的下一个函数的参数包括请求计数,您提供的observable应该提供一个observable,它可以满足所请求数量的数据。此行为的一个示例是来自外部数据源的分页查询。
以前,将OnSubscribe
转换为Iterable
并使用Observable.from(Iterable)
是一种安全的做法。此实现获取一个迭代器并为您检查subscriber.isUnsubscribed()
。