Using a loop to improve jquery code

Question

How can I using a loop to improve this jquery code?

$.getJSON("@Url.Action("GetPainel", "Home")", {}, function(data) {
  var json = data;
  var PieData = [{
    value: data[0].value,
    color: data[0].color,
    highlight: data[0].highlight,
    label: data[0].label
  },{
    value: data[1].value,
    color: data[1].color,
    highlight: data[1].highlight,
    label: data[1].label
  }];

Rory McCrossan · Accepted Answer · 2017-08-17 12:28:01Z

3

You don't event need a loop**, you can just use map() to build a new array of objects from the one retrieved from your AJAX request:

$.getJSON("@Url.Action("GetPainel", "Home")", function(data) {
  var pieData = data.map(function(o) {
    return {
      value: o.value,
      color: o.color,
      highlight: o.highlight,
      label: o.label
    }
  }

  // work with pieData here...
});

** I mean explicit loop. I realise that map() loops internally.

answered Aug 17, 2017 at 12:28

Rory McCrossan

338k41 gold badges322 silver badges353 bronze badges

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

4 Comments

ArcX Over a year ago

elegant and efficient solution. Kudos.

KarelG Over a year ago

It would be kinda silly to map over data if each element has just 4 keys and he's trying to copy it over...

Rory McCrossan Over a year ago

@KarelG I agree, although I'm giving the OP the benefit of the doubt that they wouldn't be asking that question if it were possible to transpose an existing array

Gustavo Borges Over a year ago

Very simple. Thx.

Anas AL-zghoul · Accepted Answer · 2017-08-17 12:29:33Z

0

You can make it like this

 $.getJSON("@Url.Action("GetPainel", "Home")", {}, function (data) {
    var PieData=[];
    data.forEach(function(entry){
        PieData.push({
          value:entry.value,
          color:entry.color,
          highlight:entry.highlight,
          label:entry.label
       });
    });

answered Aug 17, 2017 at 12:29

Anas AL-zghoul

4676 silver badges15 bronze badges

Collectives™ on Stack Overflow

Using a loop to improve jquery code

2 Answers 2

4 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

4 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related