CODEWITHSUNDEEP
javascriptnode.jsrecursionpromise
Scott G.
Scott G.

Reputation: 323

What is the best way to approach pagination with Promises?

My friend and I are working with promises, and we are making sure that we are getting all pages of data before returning back to our initial call. Is there a more trivial way to approach this?

function getDocuments(startIndex, result, entries) {

    startIndex = typeof startIndex !== 'undefined' ? startIndex : 0;
    result = typeof result !== 'undefined' ? result : {};
    entries = typeof entries !== 'undefined' ? entries : [];

    // build our entries set with the result parameter

    for(var i in result.items) {
        try
        {
            var id = result.items[i].id;
            var name = result.items[i].name;
            var content = result.items[i].content;
            var entry = { "id": id, "name": name, "content": content };
            entries.push(entry);
        }
        catch(e) {
        }
    }

    // return a promise that fulfills a promise that then returns either a promise or a result.
    return new Promise(function(fulfill, reject) {
        // fulfill the promise and resolve the value, we pass a recursive promise as the value.
        fulfill(documentClient.getDocuments({ "startIndex": startIndex }).then(function(result) { // once our request is made, let's check the page count.
            var startIndex = result.startIndex;
            var pageSize   = result.pageSize;
            var totalCount = result.totalCount;
            if (startIndex + pageSize <= totalCount) { // if our current position is not at the end of the pages, return a promise with our current data and our current entries.
                return getDocuments(startIndex + pageSize, result, entries);
            }
            return entries; // otherwise our entries will bubble back up the stack and be resolved into the initial fulfill value.
        }));
    });
}

getDocuments().then(function(d) { console.log(d.length); });

My adjustments:

function getDocuments(startIndex, result, entries) {

    startIndex = typeof startIndex !== 'undefined' ? startIndex : 0;
    result = typeof result !== 'undefined' ? result : {};
    entries = typeof entries !== 'undefined' ? entries : [];

    // build our entries set with the result parameter

    // ...

    // return a promise that fulfills a promise that then returns either a promise or a result.
    return documentClient.getDocuments({ "startIndex": startIndex }).then(function(result) { // once our request is made, let's check the page count.
        var startIndex = result.startIndex;
        var pageSize   = result.pageSize;
        var totalCount = result.totalCount;
        if (startIndex + pageSize <= totalCount) { // if our current position is not at the end of the pages, return a promise with our current data and our current entries.
            return getDocuments(startIndex + pageSize, result, entries);
        }
        return entries; // otherwise our entries will bubble back up the stack and be resolved into the initial fulfill value.
    });
}

getDocuments().then(function(d) { console.log(d.length); });

Upvotes: 1

Views: 1714

Answers (2)

Paul S.
Paul S.

Reputation: 66364

Have a code path in getDocuments which does not end in a new promise

Here is a simple example

function defer(fn) {
    setTimeout(fn, 100);
}
function promiseChain(i, msg) {
    if (i <= 0) return msg; // end code path
    return new Promise((res, rej) => {
        defer(e => res(promiseChain(i - 1, msg)));
    });
}
promiseChain(10, "Hello World!").then(data => console.log(data));
// "Hello World!" logged after ten 100ms timeouts

Upvotes: 0

Linus Oleander
Linus Oleander

Reputation: 18137

Yes, you can chain the promises like this as documentClient.getDocuments returns a promise.

function getDocuments(startIndex, result, entries) {
    // ...
    return documentClient.getDocuments({ "startIndex": startIndex }).then(function(result) {
        // ...
        return entries; // otherwise our entries will bubble back up the stack and be resolved into the initial fulfill value.
    });
}

getDocuments().then(function(d) { console.log(d.length); });

Upvotes: 1

Related Questions