Javascript loop through an array of objects and return the first value

Question

I have created a function that is supposed to loop through an array of objects and return the first value of each object.

function getSheetSectionData(name){
  var sheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(name);
  var sheetData = sheet.getDataRange().getValues();
  
  var data = [];
  
  for (var i = 0; i < sheetData.length; i++){
      var obj = {};
      obj = sheetData[i][0];
      return sheetData[i][0];
  }
      data.push(obj);
}

It's only returning the first item in the first row/column. Any clues on what I'm missing?

You only execute the loop once and the you break from it by returning the value. — Eduard Malakhov
– Eduard Malakhov, Commented Apr 7, 2017 at 0:00
if its to be used as a custom function, none of the answers will work. — Zig Mandel
– Zig Mandel, Commented Apr 7, 2017 at 0:21
first confirm you plan to use as a custom function (called from a cell formula) — Zig Mandel
– Zig Mandel, Commented Apr 7, 2017 at 11:16

kind user · Accepted Answer · 2017-04-07 00:02:58Z

2

You could use Object.keys together with Array#map to get just the first key value from each object.

data = sheetData.map(v => v[Object.keys(v)[0]]);

Working example:

var arr = [{foo: 'bar', bar: 'foo'},{foo: 'war', bar: 'foo'},{foo: 'mar', bar: 'foo'}],
    res = arr.map(v => v[Object.keys(v)[0]]);
    
    console.log(res);

answered Apr 7, 2017 at 0:02

kind user

42k8 gold badges69 silver badges78 bronze badges

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

Comments

HenryDev · Accepted Answer · 2017-04-07 00:20:36Z

0

How about this solution. Hope it helps!

var sheetData = [{name : "Mike", id: 10},{name : "Laura", id: 23},{name : "carl", id: 25},{name : "Lori", id: 23}];
    var arr = []
    for(var i in sheetData){
        var someObject = sheetData[i];
        arr.push(someObject[Object.keys(someObject)[0]]);
    }
    console.log(arr);

edited Apr 7, 2017 at 0:20

answered Apr 7, 2017 at 0:07

HenryDev

5,0136 gold badges31 silver badges70 bronze badges

Comments

kind user · Accepted Answer · 2017-04-07 14:06:09Z

0

You have to move the return statement outside the loop.

function getSheetSectionData(name){
  var sheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(name),
  sheetData = sheet.getDataRange().getValues(),
  data = [];
  
  for (var i = 0; i < sheetData.length; i++){
      var obj = {};
      obj = sheetData[i][0];
      data.push(obj);
  }
   return data; 
}

edited Apr 7, 2017 at 14:06

kind user

42k8 gold badges69 silver badges78 bronze badges

answered Apr 7, 2017 at 0:08

Affes Med Fedi

572 bronze badges

Comments

Eduard Malakhov · Accepted Answer · 2017-04-07 07:03:54Z

-1

I'm not sure what your intent is, but probably it should be something like this?

function getSheetSectionData(name){
  var sheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(name);
  var sheetData = sheet.getDataRange().getValues();

  var data = [];

  for (var i = 0; i < sheetData.length; i++){
      var obj = {};
      obj = sheetData[i][0];
      data.push(obj);
  }

  return data;
}

UPDATE

As @grogx noted below, creation of a temporary object appears unnecessary in this context and the sample above could be optimized to

function getSheetSectionData(name){
  var sheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName(name);
  var sheetData = sheet.getDataRange().getValues();

  var data = [];

  for (var i = 0; i < sheetData.length; i++){
      data.push(sheetData[i][0]);
  }

  return data;
}

Which can further be shortened to

function getSheetSectionData(name){
    return SpreadsheetApp.getActiveSpreadsheet()
        .getSheetByName(name)
        .getDataRange()
        .getValues()
        .map((e) => e[0]);
}

However, we do not really know, what the original intent of the OP was. It may be the case, that that temporary object was indeed required for some sort of intermediate transformation, which was striped out from the MCVE.

edited Apr 7, 2017 at 7:03

answered Apr 7, 2017 at 0:01

Eduard Malakhov

1,1224 gold badges13 silver badges25 bronze badges

1 Comment

Devmix Over a year ago

Why create a new object "obj"?? there's no need of doing that. You already have objects coming back!!

Collectives™ on Stack Overflow

Javascript loop through an array of objects and return the first value

4 Answers 4

Comments

Comments

Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related