Create an array inside for loop javascript

Question

I am trying to create a dataset pattern like this from the rest service in my angular application.

Example:

$scope.chartSeries = [
  {"name": "Google", "data": [100, 200, 400, 700, 300]},
  {"name": "Yahoo", "data": [300, 100, null, 500, 200]},
  {"name": "Facebook", "data": [500, 200, 200, 300, 500] },
  {"name": "Microsoft", "data": [100, 100, 200, 300, 200] }
];

Right now this is my code to build a similar pattern ,

 if (datai){ 
    $rootScope.DashboardData =[];
    $rootScope.DashboardData = datai;
    var _fieldData = [];
    widget.chartConfig.series  = [];
    widget.chartConfig.series  = 
    $rootScope.DashboardData.map(function(elm) {
        return { 
            name: elm[widget.seriesname], 
            data:_fieldData.push(elm[widget.dataname])
        };
    });
 }

Problem is data field is not creating an array. This is the final output which the above code generates.

 [{"name": "Google", "data": 1}];

Can you explain little more details about your problem? Where is your for loop? — Ramesh Rajendran
– Ramesh Rajendran, Commented Mar 16, 2015 at 5:39
should be like $rootScope.DashboardData.map(function(elm) { _fieldData.push(elm[widget.dataname]) return { name: elm[widget.seriesname], data: _fieldData }; }); } — Pankaj Parkar
– Pankaj Parkar, Commented Mar 16, 2015 at 5:41
sorry guys, i did it in c# way. now got it! thanks @pankajparkar — Sajeetharan
– Sajeetharan, Commented Mar 16, 2015 at 5:48

Will Reese · Accepted Answer · 2015-03-16 05:53:16Z

3

push() returns the new array's length

$rootScope.DashboardData.map(function(elm) 
{
    var _fieldData = [];
    _fieldData.push(elm[widget.dataname]);
    return { name: elm[widget.seriesname], data: _fieldData };
});

or you can use concat()

return { name: elm[widget.seriesname], data: _fieldData.concat(elm[widget.dataname])};

edited Mar 16, 2015 at 5:53

Will Reese

2,8512 gold badges17 silver badges28 bronze badges

answered Mar 16, 2015 at 5:43

gerald89

493 bronze badges

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

Comments

levi · Accepted Answer · 2015-03-16 05:42:44Z

2

You are assigning _fieldData.push(elm[widget.dataname]) to data and .push() method returns the new array length, so you need to push the element first and then assign the array itself.

_fieldData.push(elm[widget.dataname])
 return { name: elm[widget.seriesname], data:_fieldData};
      });

answered Mar 16, 2015 at 5:42

levi

22.8k7 gold badges79 silver badges76 bronze badges

Comments

Ramesh Rajendran · Accepted Answer · 2015-03-16 05:46:18Z

1

try this , You nee a loop for dataname

$rootScope.DashboardData.map(function(elm) 
{
   var _fieldData = [];
   for(var i=0;i<elm[widget.dataname.length;i++)
    {
      _fieldData.push(elm[widget.dataname][i]);
    }    
    return { name: elm[widget.seriesname], data: _fieldData
});

answered Mar 16, 2015 at 5:46

Ramesh Rajendran

38.9k49 gold badges159 silver badges241 bronze badges

Comments

Pankaj Parkar · Accepted Answer · 2015-03-16 05:50:45Z

0

Array .push() always returns an new length of an array

You should push that variable into _fieldData.push(elm[widget.dataname]); then assign that variable late in data.

$rootScope.DashboardData.map(function(elm) 
{
    var _fieldData = [].push(elm[widget.dataname]);
    return { name: elm[widget.seriesname], data: _fieldData }
});

answered Mar 16, 2015 at 5:50

Pankaj Parkar

136k23 gold badges241 silver badges307 bronze badges

Collectives™ on Stack Overflow

Create an array inside for loop javascript

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related