Parse javascript for loop, creating multiple objects without duplicates

Question

I am having trouble with Parse in Javascript. I am just trying to loop through my list of objects that I have in an array. If the imageURL matches one in Parse, then do nothing. If their is no imageURL from the array, in Parse, then add the row to Parse.

I think my issue is synchronicity. Its is currently saving the same row (same name, imageURL, and fbURL) many times to Parse. Basically saving one row, and its duplicated 70 times.

 for (var i = 0; i < listobjects.length; i++) {

      objName = listobjects[i]["name"]
      objImageURL = listobjects[i]["imageURL"]
      objfbURL = listobjects[i]["fbURL"]

      console.log("NAME: " + listobjects[i]["name"])

      // Check if record is in Parse
      var query = new Parse.Query(Recipe);
      query.equalTo("imageURL", listobjects[i]["imageURL"]);
      query.find({
        success: function(results) {
          if (results.length > 0) {
            // Do nothing. We have a result. Or maybe check if videoURL is present
            console.log("Object is already in DB.")
          } else {


            // Save the new object.
            var recipeObject = new Recipe();
            recipeObject.save({name: objName, imageURL:objImageURL, fbURL: objfbURL}, {
              success: function(object) {
                console.log("Object Saved!")
              },
              error: function(model, error) {
                console.log("Error saving object!")
              }
            });


          }
        },
        error: function(error) {
          alert("Error: " + error.code + " " + error.message);
        }
      });

    }

You've not given enough here to reproduce the issue.

Jeremy J Starcher
– Jeremy J Starcher

2016-03-07 22:51:24 +00:00
Commented Mar 7, 2016 at 22:51 — Jeremy J Starcher
– Jeremy J Starcher, Commented Mar 7, 2016 at 22:51

Anatol - user3173842 · Accepted Answer · 2016-03-07 23:33:09Z

1

Change

for (var i = 0; i < listobjects.length; i++) {
  objName = listobjects[i]["name"]
  // ...
}

to

listobjects.forEach(function(obj) {
  // access obj.name, obj.imageURL
  // ...
});

to create a scope and the right object will be used within the find callback.

answered Mar 7, 2016 at 23:33

Anatol - user3173842

1,4251 gold badge13 silver badges18 bronze badges

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

Collectives™ on Stack Overflow

Parse javascript for loop, creating multiple objects without duplicates

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related