スプレッドシートでgetValuesで取得した二次元配列の処理について

Question

スプレッドシートでgetValuesで取得した二次元配列の処理について教えてください。
なお、エラーが出る等の質問ではなく、より良い（正しい？）書き方の指南をいただきたいというお願いです。

以下のコードは 日にち毎のシートに分かれているデータを集計シートに統合する処理で、日付が古いものがシート番号の後ろのほうに来ている為、古い方から for文で データを取得しています。

集計シートの A列には 通し番号（3行目にタイトル行があり、データは4行目から）
集計シートの B列に取得したデータが入っていたシート名を 入れるようにしています。

元々はシートのデータ取得、貼付け、シート名取得、貼付け を forで回してたのですが、処理速度向上と関数での配列処理の勉強の為、 mapとunshiftを使って 2次元配列の各要素の最初に シート名を追加と、そのデータを concat で結合することで、貼付け処理を最後の1回にしています。

質問としては、

1. Data.map(function(arr){ の処理の箇所ですが、本当はアロー関数で書きたいのですが、unshiftが返り値がないため、書き方がわからず funciton を使って記述しています。こういった処理をアロー関数でより短く書くことは可能でしょうか？

2. for文の最後の処理 var Array = Array.concat(Data2); ですが、こちらは逆に pushでの追加と違って返り値が欲しい為、このような書き方となっています。理解不足ですが、const ではうまくいかない為です。ただ、ネット上では varを使うケースは殆どないという記事を見たことがあり、書き方として正しくないのでは？と気になっています。

このような for文で 配列を結合していく場合の書き方として、より良いものがあれば教えてください。

3.　最後の.setFormula("=SEQUENCE(COUNTA(B4:B))")ですが、単に集計シートのA列に連番をふりたいというだけなのですが、上記の二次元配列の処理に組み込むよい方法があれば知りたいです。

以上、よろしくお願いいたします。

function Sheetsummary() {


  const SS = SpreadsheetApp.getActiveSpreadsheet();
  const sheets = SS.getSheets();
  const sheetcount = sheets.length;
  const SumSheet = SS.getSheetByName("集計");

  //最初に集計シートの該当範囲をクリア
  SumSheet.getRange(4,1,SumSheet.getLastRow(),20).clearContent();

  var Array =[];

  for(i=sheetcount-1 ; i > 0 ; i=i-1){
    const sheetName =  sheets[i].getSheetName();
    const DataHigh = sheets[i].getLastRow()-3;
    const Data = sheets[i].getRange(4,2,DataHigh,8).getValues();
     
    const Data2 = Data.map(function(arr){
      arr.unshift(sheetName);
      return arr;
      });
    
    var Array = Array.concat(Data2);
    
    }
    
    const TRow = SumSheet.getLastRow()+1;
    SumSheet.getRange(TRow,2,Array.length,9).setValues(Array);
    
    SumSheet.getRange(4,1).setFormula("=SEQUENCE(COUNTA(B4:B))");
  
}

Answer 1

こんな感じでしょうか。

function q273985() {
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheets = ss.getSheets();
  const sumSheetName = '集計';
  const sumSheet = ss.getSheetByName(sumSheetName);

  //最初に集計シートの該当範囲をクリア
  const lastRow = sumSheet.getLastRow();
  if (lastRow > 4) {
    sumSheet.getRange(4, 2, lastRow - 3, 19).clearContent();
  }

  const data = [];

  sheets.reverse().forEach(sheet => {
    const sheetName = sheet.getSheetName();
    if(sheetName !== sumSheetName) {
      const dataRow = sheet.getLastRow();
      data.push(...(sheet.getRange(4, 2, dataRow - 3 , 8).getValues().map(arr => [sheetName, ...arr])));
    }
  });
  Logger.log(data);
  sumSheet.getRange(4,2, data.length, 9).setValues(data);
}

Comments

[sawa]

全体手直しありがとうございます。

なるほど。reverse()を使うと sheetsの配列の並びが逆転するので、forEach で後ろのシートから順に処理が出来るのですね。そして1文で私のやりたかった配列処理を一気にやって pushしていくと。ご提示いただければ読解はなんとか出来るようになってきたのですが、まだまだ自分でこう書けるまでは遠そうです。勉強になりました。

Answer 2

一個一個解決しちゃうと、ツギハギになってしまうような気もしますが、一応一つずつ回答しておきます。

1. unshiftを使わずに記述します。

const Data2 = Data.map(arr => [sheetName, ...arr]);

2.pushしましょう。（なんで返り値がほしいのかがわからないです。）

const values = [];

for(i=sheetcount-1 ; i > 0 ; i=i-1){
  /* 省略 */
  values.push(...Data2);
}

3.これでいいなら初期化（クリア）しなきゃいいんじゃないですか？

Comments

[sawa]

macaron_XXX様
ありがとうございます。3つとも回答いただき助かります。3は確かにおっしゃる通りですね。
最初のクリアする範囲を変更することで対応しました。

1、2に関しては、私が ... の使い方を理解してない故に 質問のような記述になった次第です。
ちなみに この... は、使ってるケースを見たことはあるのですが、検索でもうまく引っかからず、読み方もよくわからない為、自分の中で消化できていません。解説されているサイト等あれば 教えていただければ幸いです。

[macaron_xxx]

スプレッド構文といいます。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Operators/Spread_syntax

[sawa]

早速ありがとうございます。活用できるケースが多々ありそうなので、読み込んで色々試したいと思います。