JS/TS - function return empty array

Asked Mar 20 '17 at 14:23

Active Mar 20 '17 at 16:50

Viewed 33 times

I have this function

populateGenerals() {

    var generals = [];
    this.productsService.getGenerals()
        .subscribe(data => {
            data.forEach(element => {
                generals.push({ id: element._id, text: element._id });
            });
            console.log(generals);
        });

    console.log(generals);
    return generals;
}

inner console gives me:

[{"id":"1","text":"red"},{"id":"2","text":"blue"}]

while outer console gives me:

[]

How can I return the same array as inner console when returning generals?

UPDATE / SOLUTION

populateGenerals() {
    this.loading = true;
    var generals = [];


    this.productsService.getGenerals()
        .subscribe(data => {
            this.loading = false;
            data.forEach(element => {
                generals.push({ id: element._id, text: element._id });
            });

            this.populateCallback(generals);
        });

    return this.generals;
}

populateCallback(array) {
    this.generals =  array;
}

edited Mar 20 '17 at 16:50

asked Mar 20 '17 at 14:23

Kavoos

Use `map` instead of `forEach` and return that? – evolutionxbox Mar 20 '17 at 14:27
@evolutionxbox, map doesn't work. I think because I don't actually map them – Kavoos Mar 20 '17 at 14:32
Ah. Is subscribe an asynchronous method? – evolutionxbox Mar 20 '17 at 14:33
@evolutionxbox, no Idea. The reason I said that map is not working, is that I want to assign the `element._id` to both `id` and `name` – Kavoos Mar 20 '17 at 14:36
@epascarello, Thanks, but still I really can't get how return the right array using the answer you provided :/ – Kavoos Mar 20 '17 at 15:15
1

You can't return it... You need to use a call back or a promise. – epascarello Mar 20 '17 at 15:15

JS/TS - function return empty array

0 Answers0