我有一个在页面中返回数据的服务。对一个页面的响应包含有关如何查询下一页的详细信息。
我的方法是返回响应数据,然后如果有更多页面可用,则立即将延迟调用连接到相同的可观察序列。
function getPageFromServer(index) {
// return dummy data for testcase
return {nextpage:index+1, data:[1,2,3]};
}
function getPagedItems(index) {
return Observable.return(getPageFromServer(index))
.flatMap(function(response) {
if (response.nextpage !== null) {
return Observable.fromArray(response.data)
.concat(Observable.defer(function() {return getPagedItems(response.nextpage);}));
}
return Observable.fromArray(response.data);
});
}
getPagedItems(0).subscribe(
function(item) {
console.log(new Date(), item);
},
function(error) {
console.log(error);
}
)
这一定是错误的做法,因为你会在2秒内得到:
throw e;
^
RangeError: Maximum call stack size exceeded
at CompositeDisposablePrototype.dispose (/Users/me/node_modules/rx/dist/rx.all.js:654:51)
分页的正确方法是什么?
答案 0 :(得分:7)
查看OP代码,它确实是正确的方法。只需要使您的模拟服务异步以模拟真实条件。这是一个避免堆栈耗尽的解决方案(我还使getPageFromServer
实际上返回一个冷可观察对象,而不是要求调用者包装它。)
请注意,如果您真的希望您的服务请求在实际应用程序中同步完成,那么您需要确保在发生这种情况时代码不会耗尽堆栈,只需让getPagedItems()
调用currentThread scheduler 。 currentThread
调度程序使用trampoline调度任务以防止递归调用(和堆栈耗尽)。请参阅getPagedItems
function getPageFromServer(index) {
// return dummy data asynchronously for testcase
// use timeout scheduler to force the result to be asynchronous like
// it would be for a real service request
return Rx.Observable.return({nextpage: index + 1, data: [1,2,3]}, Rx.Scheduler.timeout);
// for real request, if you are using jQuery, just use rxjs-jquery and return:
//return Rx.Observable.defer(function () { return $.ajaxAsObservable(...); });
}
function getPagedItems(index) {
var result = getPageFromServer(index)
.retry(3) // retry the call if it fails
.flatMap(function (response) {
var result = Rx.Observable.fromArray(response.data);
if (response.nextpage !== null) {
result = result.concat(getPagedItems(response.nextpage));
}
return result;
});
// If you think you will really satisfy requests synchronously, then instead
// of using the Rx.Scheduler.timeout in getPageFromServer(), you can
// use the currentThreadScheduler here to prevent the stack exhaustion...
// return result.observeOn(Rx.Scheduler.currentThread)
return result;
}
答案 1 :(得分:4)
编辑啊!我看到你面临的问题。一些尾部调用优化应该可以帮到你:
function mockGetPageAjaxCall(index) {
// return dummy data for testcase
return Promise.resolve({nextpage:index+1, data:[1,2,3]});
}
function getPageFromServer(index) {
return Observable.create(function(obs) {
mockGetPageAjaxCall(index).then(function(page) {
obs.onNext(page);
}).catch(function(err) {
obs.onError(err)
}).finally(function() {
obs.onCompleted();
});
});
}
function getPagedItems(index) {
return Observable.create(function(obs) {
// create a delegate to do the work
var disposable = new SerialDisposable();
var recur = function(index) {
disposable.setDisposable(getPageFromServer(index).retry().subscribe(function(page) {
obs.onNext(page.items);
if(page.nextpage === null) {
obs.onCompleted();
}
// call the delegate recursively
recur(page.nextpage);
}));
};
// call the delegate to start it
recur(0);
return disposable;
});
}
getPagedItems(0).subscribe(
function(item) {
console.log(new Date(), item);
},
function(error) {
console.log(error);
}
)
答案 2 :(得分:3)
这是一个更简洁的&恕我直言干净的答案,没有任何递归。 它使用ogen(~46 loc)将任何生成器转换为可观察的。
它有一个自定义构建的下一个函数,只要你的函数产生某些东西就会发出数据。
nb:original article值得一读
function getPagedItems({offset=0, limit=4}) {
paginatedQueryGenerator = function*(someParams offset, limit) {
let hasMore = true
while(hasMore) {
const results = yield YOUR_PROMISE_BASED_REQUEST(someParams, limit, offset)
hasMore = results && results.nextpage !== null
offset += limit
}
}
return ogen(paginatedQueryGenerator)(someParams, offset, limit)
}
答案 3 :(得分:0)
另一种解决方案是使用retryWhen
getAllData() {
let page = 0;
let result = [];
const getOnePage = () => {
return of(0).pipe(mergeMap(() => getPaginatedData(page++)));
};
return getOnePage()
.pipe(
map(items => {
result = result.concat(items);
if (templates.length === PAGE_SIZE) {
throw 'get next page';
}
}),
retryWhen(e => e.pipe(
takeWhile(er => er === 'get next page'))
),
map(e => result)
)
.subscribe(items => {
console.log('here is all data', items);
});
}