map array to another array in TypeScript

Question

Can I improve my code and replace for-loop by array.map?

I searched and I think I could do that but I didn't find how could I apply that.

This is what I found :

var result = arr.map(person => ({ value: person.id, text: person.name }));

And my code is here:

  public getFlights(): Observable<RowItem[]> {
    return this.http
      .get(this.apiHostFlights)
      .map((res: any) => {
        return <LocationModelItem[]>res.json();
      })
      .map((items: LocationModelItem[]) => {
        var rowItems: RowItem[]=[];
        var cachedLenght = items.length;
        for (var i = 0; i < cachedLenght; i++) {
          rowItems.push(
            new RowItem(i, items[i].name, items[i].img, items[i].category)
          );
        }
        return rowItems;
      })
      .catch((error: any) => {
        return Observable.throw(error.statusText);
      });
  }

Thomas · Accepted Answer · 2018-03-06 10:13:09Z

11

 public getFlights(): Observable<RowItem[]> {
    return this.http
      .get(this.apiHostFlights)
      .map((res: any) => {
        return <LocationModelItem[]>res.json();
      })
      .map((items: LocationModelItem[]) => {
        return items.map((item, index) => ({
           index: index,
           name: item.name,
           img: item.img,
           category: item.category)
        }))

      })
      .catch((error: any) => {
        return Observable.throw(error.statusText);
      });
  }

This should do it hopefully

edited Mar 6, 2018 at 10:13

answered Mar 6, 2018 at 10:05

Thomas

2,5461 gold badge19 silver badges23 bronze badges

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

7 Comments

amorenew Over a year ago

can I you made it like this answer var result = arr.map(person => ({ value: person.id, text: person.name }));

Thomas Over a year ago

That means you want to get a list of objects instead of a list of RowItems?

amorenew Over a year ago

I just want a better performance instead of using for loop or 3 maps but rowitems

Thomas Over a year ago

Because you have to. I see no logical possibility to avoid that.

zerkms Over a year ago

@amorenew and "3 maps" is a problem because... ?

|

Collectives™ on Stack Overflow

map array to another array in TypeScript

1 Answer 1

7 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related