asynchronous chrome method inside for loop

Question

for every variable i, below code should traverse thro' each bookmark node and compares the url, whether it exist or not.

for(i=0;i<arg1;i++){
    chrome.bookmarks.getChildren(Traverse[i], function(child){       //to fetch the child nodes
        Loc =child.length;
        alert(Loc);   // This message to appear first
        if(Loc != 0){
            child.forEach(function(book) {
                if (book.url == urL){
                    alert("Bookmark already exist");
                    element = "init";
                }   
            }); 
        }
    });
alert("message to be printed last");
}

since the method is asynchronous, i'm getting the last message and bookmark traversing doesn't happens. Any help would be much appreciated.

Thanks !!

score 0 · Accepted Answer · answered Mar 24 '13 at 12:01

You probably need a closure:

for(i=0;i<arg1;i++){
    (function(my_i) {
        chrome.bookmarks.getChildren(Traverse[my_i], function(child){
            Loc =child.length;
            alert(Loc);
            if(Loc != 0){
                child.forEach(function(book) {
                    if (book.url == urL){
                        alert("Bookmark already exist");
                        element = "init";
                    }   
                }); 
            }
        });
    })(i);
    alert("message to be printed last");
}

I'm guessing you're aware that you are overwriting both the Loc and the element variables on each iteration inside the loop ?

Thanks Adeneo ! above code works good. I'm testing it. One more query, how to exit when i hit a bookmark exist scenario, i.e. how to exit the loop, as break statement didn't work. — Gomathi Sankar, Mar 24 '13 at 13:08

asynchronous chrome method inside for loop

1 Answers1