Java script for loop Parse Promise

Question

I am trying to read a file on parse.com and using a for loop iterate over all the records present in it. On each record, I need to perform 4 operations, each dependent on the other. Can someone please guide how I can do that so that each record is processed in the for loop.

Parse.Cloud.httpRequest({
            url: urlValue
        }).then(function(fileResponse) {
            console.log("processUploadFile:httpRequest:response:" + JSON.stringify(fileResponse.buffer.length));
            // console.log("processUploadFile:Text:" + fileResponse.text);

            var records = fileResponse.text.split("\r");

            for (var i = 0; i < records.length; ++i) {
                // console.log("Record:" + i + " detail:" + records[i] + "\n\n");
                var record = records[i];


                        console.log("processUploadFile:adding patient");
                        Parse.Cloud.run("addPatient", {
                            record:record
                        }, {
                            success: function(objectId) {
                                console.log("Created objectId:" + JSON.stringify(objectId));
                                Parse.Cloud.run("addProvider", {
                                    record:record
                                }, {
                                    success: function(objectId) {
                                        console.log("Created objectId:" + JSON.stringify(objectId));

                                        Parse.Cloud.run("addLocation", {
                                            record:record
                                        }, {
                                            success: function(objectId) {
                                                console.log("objectId:" + JSON.stringify(objectId));
                                            },
                                            error: function(error) {
                                                console.log(JSON.stringify(error));
                                            }
                                        });
                                    },
                                    error: function(error) {
                                        console.log(JSON.stringify(error));
                                    }
                                });
                            },
                            error: function(error) {
                                console.log(JSON.stringify(error));
                            }
                        });
                    };
                }
            }


            response.success();
        });

This sort of nested-event madness is why I created www.taskrunnerjs.com - although there are lots of other alternatives. IMO those styles of chaining are much easier to read and comprehend than this. (This is just an opinion, not so much an answer. Honestly it's not entirely clear what you're asking.) — bvaughn
– bvaughn, Commented Mar 4, 2015 at 17:49
Do those subsequent steps depend on each other, in other words, e.g. must addPatient be completed before before addProvider can work, or may these be performed in any order? — danh
– danh, Commented Mar 4, 2015 at 18:03
@danh... yes ... addPatient is dependent on completion of addProvider — i_raqz
– i_raqz, Commented Mar 4, 2015 at 18:24
@brianvaughn ... www.taskrunnerjs.com looks great... but I don't think I can use that on parse.com. Can I use promises? — i_raqz
– i_raqz, Commented Mar 4, 2015 at 18:26
you can (and should) use promises with parse.com on both client and server. @brianvaughn's library looks to be well thought out, though I've never used it. — danh
– danh, Commented Mar 4, 2015 at 18:30

danh · Accepted Answer · 2015-03-04 19:26:05Z

1

The right right answer depends on the semantics of those operations, whether they depend on each other in any way. The other part of a right right answer accounts for transaction rate limits and timeouts imposed by parse.com. That also depends on what happens in the cloud operations and how much data is being processed.

But the right answer (as opposed to right right) is to perform operations serially by chaining promises' then(), and to perform groups of operations concurrently (or in arbitrary order) with Parse.Promise.when().

One such ordering would look like this:

var patientQs = [];
var providerQs = [];
var locationQs = [];
var records;

Parse.Cloud.httpRequest({url: urlValue}).then(function(fileResponse) {
    console.log("processUploadFile:httpRequest:response:" + JSON.stringify(fileResponse.buffer.length));
    records = fileResponse.text.split("\r");
    for (var i = 0; i < records.length; ++i) {
        // console.log("Record:" + i + " detail:" + records[i] + "\n\n");
        var record = records[i];
        patientQs.push(Parse.Cloud.run("addPatient", {record:record}));
        providerQs.push(Parse.Cloud.run("addProvider", {record:record}));
        locationQs.push(Parse.Cloud.run("addLocation", {record:record}));
    }
    return Parse.Promise.when(patientQs);
}).then(function() {
    // since the result of addPatient is an objectId, arguments
    // will be the corresponding objectIds for each run 
    for (var i=0; i<arguments.length; i++) {
        console.log(arguments[i] + " is the object id for input record " + JSON.stringify(records[i]));
    }
    return Parse.Promise.when(providerQs);
}).then(function() {
    return Parse.Promise.when(locationQs);
}).then(function() {
    response.success();
}, function(error) {
    response.error(error);
});

This says, "go thru the http-retrieved records, and first add all of the patients for those records, then add all of the providers, and so on".

Or, you could group the operations by input record, like this:

Parse.Cloud.httpRequest({url: urlValue}).then(function(fileResponse) {
    console.log("processUploadFile:httpRequest:response:" + JSON.stringify(fileResponse.buffer.length));
    var records = fileResponse.text.split("\r");
    var recordQs = [];

    for (var i = 0; i < records.length; ++i) {
        // console.log("Record:" + i + " detail:" + records[i] + "\n\n");
        var record = records[i];
        recordQs.push(processARecord(record));
    }
    return Parse.Promise.when(recordQs);
}).then(function() {
    response.success(arguments);
}, function(error) {
    response.error(error);
});

function processARecord(record) {
    var result = {};
    return Parse.Cloud.run("addPatient", {record:record}).then(function(objectId) {
        console.log(objectId + " is the object id for input record " + JSON.stringify(record));
        result.patientId = objectId;
        return Parse.Cloud.run("addProvider", {record:record});
    }).then(function (providerId) {
        result.providerId = providerId;
        return Parse.Cloud.run("addLocation", {record:record});
    }).then(function(locationId) {
        result.locationId = locationId;
        return result;
    });
}

edited Mar 4, 2015 at 19:26

answered Mar 4, 2015 at 18:25

danh

62.7k10 gold badges100 silver badges139 bronze badges

Sign up to request clarification or add additional context in comments.

9 Comments

i_raqz Over a year ago

this is something close to what I was looking for. I didnt know, we could add those many "then" blocks... How do we a return a value from the block though. after the addPatient completes, I need the objectId of the currently added patient.

i_raqz Over a year ago

also, can't we perform addpatient, then addProvider then addLocation for the same record then move on to the next record?

danh Over a year ago

@i_raqz - re the first comment: see edit. var args to the then() will contain objects corresponding to each fulfilled promise. re your second comment: yes, this is easily done by making a chain of promises for each record, rather than a group for each function type.

danh Over a year ago

@i_raqz - regrouped to illustrate point about grouping by input rather than by function. The bigger point is that chaining then()'s gives you sequence and performing when()'s gives you semi-concurrence.

i_raqz Over a year ago

this is exactly what i was looking for..you gave a me a little tutorial about using Promise as well async calls.

|

Collectives™ on Stack Overflow

Java script for loop Parse Promise

1 Answer 1

9 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

9 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related