Problemet her er, at i findOne
tilbagekald - dit beerId
vil altid være indstillet til den sidste øl i beerObjects
, fordi løkken afsluttes, før du når dit første tilbagekald - velkommen til asynkront javascript.
Et middel til dette er at indpakke din findOne
kode i en IFFE (Immediately Invoked Function Expression). Denne kode vil fuldføre, før du går videre til den næste øl fra beerObject.
Her er lidt mere info om IFFE
Jeg gik hurtigt igennem koden, jeg tror, det burde virke, men du skal muligvis foretage nogle justeringer med den interne kode...
for(var i = 0; i < beerObjects.length; i++) {
console.log(beerObjects[i].beerId);
//var currentBeer = beerObjects[i]; dont need this now
(function (currentBeer) {
Beer.findOne({ beerId: currentBeer},
function(err, beer) {
if(!err && !beer) {
var newBeer = new Beer();
newBeer.beerId = currentBeer.beerId;
newBeer.name = currentBeer.name;
newBeer.description = currentBeer.description;
newBeer.abv = currentBeer.abv;
newBeer.image = currentBeer.image;
newBeer.save(function(err) {
// log your error here...
});
} else if(!err) {
console.log("Beer is in the system");
} else {
console.log("ERROR: " + err);
}
}
);
})(beerObjects[i].beerId);
}