Attempting to push into array gives error in angular6

Question

I want to sort the array and to store the fetched values into a new array for print. I tried with push() but there is an error is occurring and shows that" Cannot read property 'push' of undefined".

`

this.dataService.userprofile().subscribe((data:any) => {
      let profile:any[]=data.profiles;
      for(let index=0;index<profile.length;index++) {
        const element = profile[index];  
        const today: number = Date.now();
        var b = Math.abs(today - Date.parse(element["dob"]));
        element["dob"] = Math.floor(b / (1000 * 3600 * 24 * 365.25));
        if (element["active"]=="N"){
          this.row.push(element); 
        }}
      this.rowData = this.row;//ag-grid row
      console.log(this.rowData)
    })

`

Is this.row initialized in any way in the context of your function ? — Paqman
– Paqman, Commented Jan 10, 2019 at 2:39
@Paqman i think row is a local variable to store the array of elements and populate the initialized this.rowData.. In my answer i suggest to work directly with this.rowData assigning the result of Array.prototype.reduce() — Yosvel Quintero
– Yosvel Quintero, Commented Jan 10, 2019 at 4:03

Yosvel Quintero · Accepted Answer · 2019-01-10 03:06:49Z

2

Instead of using for loop that it is hard to read and maintain you better use Array.prototype.reduce() and using the dot notation

Also notice that in TypeScript you should avoid using type any

Code:

this.dataService.userprofile().subscribe((data: any) => {
  this.rowData = data.profiles.reduce((acc, element) => {
    const b = Math.abs(today - Date.parse(element.dob));
    element.dob = Math.floor(b / (1000 * 3600 * 24 * 365.25));
    return element.active == 'N' ? [...acc, element] : acc;
  }, []);

  console.log(this.rowData)
});

edited Jan 10, 2019 at 3:06

answered Jan 10, 2019 at 2:55

Yosvel Quintero

19.1k5 gold badges39 silver badges47 bronze badges

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

Comments

Roman Lytvynov · Accepted Answer · 2019-01-10 02:42:47Z

0

Declare row variable before for loop.

this.dataService.userprofile().subscribe((data:any) => {
      let profile:any[]=data.profiles;
      let row = [];
      for(let index=0;index<profile.length;index++) {
        const element = profile[index];  
        const today: number = Date.now();
        var b = Math.abs(today - Date.parse(element["dob"]));
        element["dob"] = Math.floor(b / (1000 * 3600 * 24 * 365.25));
        if (element["active"]=="N"){
          row.push(element); 
        }}
      this.rowData = row;//ag-grid row
      console.log(this.rowData)
    })

answered Jan 10, 2019 at 2:42

Roman Lytvynov

5132 silver badges13 bronze badges

Collectives™ on Stack Overflow

Attempting to push into array gives error in angular6

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related