二次元配列同士の引き算

Question

スプレッドシートに集計したデータが蓄積されています。

（例↓）
|(A2セル)|項目1|項目2|項目3|項目4|
|:--|:--:|--:|
|1/1|100|150|200|160|
|1/2|105|165|206|180|
|1/3...|107|180|206|215|

A列に日付、2行目に項目名が並び、日が進むごとにデータが追加されていきます。
ここに集計されるデータは累計データのため、日毎のデータをこの表から取得し、別シートに転記したいと思っています。

(転記先シート)
|(別シート)|項目1|項目2|項目3|項目4|
|:--|:--:|--:|
|1/1|-|-|-|-|
|1/2|5|15|6|20|
|1/3...|2|15|0|35|

○やりたかったこと
1/1と1/2のデータを比較(引き算)し、残った数値を別の二次元配列に格納する(全日付分繰り返し)

function myFunction(){
  var difference = [];
  var data = pivotSheet.getRange(3,2,pivotSheet.getLastRow(),pivotSheet.getLastColumn()).getValues();
  var nextData = pivotSheet.getRange(4,2,pivotSheet.getLastRow(),pivotSheet.getLastColumn()).getValues();
　for(var j = 0; j < pivotSheet.getLastRow; j++){
    for(var k = 0; k < data[0].length-1; k++){
      difference.push(nextData[j][k] - data[j][k])
      Logger.log(difference)
    }
  }
}

これで差分データをログで見られると思ったのですが、ログが表示されないという現象が発生しています。

ちなみに、

function myFunction(){
  var difference = [];
  var data = pivotSheet.getRange(3,2,pivotSheet.getLastRow(),pivotSheet.getLastColumn()).getValues();
  var nextData = pivotSheet.getRange(4,2,pivotSheet.getLastRow(),pivotSheet.getLastColumn()).getValues();
　//for(var j = 0; j < pivotSheet.getLastRow; j++){
    for(var k = 0; k < data[0].length-1; k++){
      difference.push(nextData[0][k] - data[0][k])
      Logger.log(difference)
    }
  //}
}

最初のfor文を外して1行分だけ取得すると、ちゃんとデータが取れます。
すべての行のデータを取得するにはどうしたら良いでしょうか？

Answer 1

const q316203 = () => {
    const sourceSheetName = "シート1";
    const targetSheetName = "シート2";
    const pad = "-";
    const book = SpreadsheetApp.getActive();
    const source = book.getSheetByName(sourceSheetName).getDataRange().getValues().slice(2);
    const diff = source.slice(1).map((r,i)=>r.map((c,j)=>j===0?c:c-source[i][j]));
    const first = [source[0][0],...Array.from({length:source[0].length - 1},_=>pad)]
    const data = [first,...diff];
    book.getSheetByName(targetSheetName).getRange(1,1,data.length,data[0].length).setValues(data);
}

縦方向でみたとき数値が入るはずの列に - を入れるのはあまりよくない発想なので 0 か、せめて空白にすることをおすすめします。

Answer 2

ループ内の処理も意図とは違う気がしますが、とりあえずおかしいところだけ指摘しておきます。

- for(var j = 0; j < pivotSheet.getLastRow; j++){
+ for(var j = 0; j < pivotSheet.getLastRow(); j++){

Comments

[gatten1217]

おかしいところのご指摘ありがとうございます。
「ループ内の処理も意図と違う」こともおかげさまでわかりましたｗ
すべての値が普通の配列に入ってしまっているのですが、二次元配列として納めるにはどのようなコードが正しいのでしょうか。もう少し教えていただけると幸いです

Answer 3

まず、一点先に述べておきたいと思います。

・表題の質問が二次元配列同士の引き算
・途中のやりたかった事の記述内容
・コードでやろうとしている事、ログの表示
の3つに於いて、私の視点だと、
それぞれ別の事について話をしているように見えたので、
何に対して回答しようかと混乱してしまいました。

質問の意図が理解できていない可能性もございますが、
その際はご指摘していただければ幸いだと、存じます。

今回は、コードを主体として、
ログの表示についてまとめてみました。

僭越ではございますが、
これが問題なのではなかろうかという指摘をさせていただきます。

forのループを2つ使用しておりますが、
内側のループはループの判定基準は扱うデータが対象になっていますが、
外側のループの判定基準は、シートそのものになっております。

私からの提案としては、
ループの判定基準を揃えたほうが良いと思います。

具体的な方法は色々あると思いますが、
外側のループの判定基準を、
内側と同じように、データを対象とする
その方が良いと思いました。

pivotSheet.getLastRow　　　　　　//こちら正確には、関数の実行なのでgetLastRow()になります。
ではなく、
data.length 　　　　　　　　　　　//こちらはプロパティを参照するので、()無しです。
に変更する感じです。

これでループは回るはずですが、
・differenceにpushし続ける事、
・多分やりたい事は、外側でのログ表示だと思っている
という事が気になっていたので、
外側のループの何処かで、

differenceのログを出して、
differenceをクリアしてたほうが良いと思います。

今回コードに対する返答のみとなりました。
質問そのものに対する私なりの返答も用意できたのですが、
gatten1217様の環境などが分からない事が理由で、
今回は見送っております。

私からは以上になります。