GASでSpreadsheetに書き込みをしたいが、エラーが出る

Question

前提・実現したいこと

google Spreadsheetにデータ（1行4列）がシート1に書き込まれたら、書き込まれたデータとシート2にすでにあるデータを比較して、シート2に上書きor最終行に挿入したいが、エラーが出てうまく動かない。
参考にしているページ：
https://qiita.com/rf_p/items/f423ed2b23b789b2ebe9

発生している問題・エラーメッセージ

Exception: パラメータ（number[]）が SpreadsheetApp.Range.setValues のメソッドのシグネチャと一致しません。（行 24、ファイル「コード」）

該当のソースコード

ほとんど上記参考ページのコードのコピペです、、、

function createList(){
  var sheetRegist = SpreadsheetApp.getActiveSpreadsheet().getSheetByName('sheet1');
  var lastRowRegist = sheetRegist.getLastRow();
  var newData = sheetRegist.getRange(lastRowRegist, 1, 1, 4).getValues();
 
  var sheetTarget =SpreadsheetApp.getActiveSpreadsheet().getSheetByName('sheet2');

  insertOrUpdate(sheetTarget, newData);
}

function insertOrUpdate(sheetTarget, newData) {
  var sheetRegist = SpreadsheetApp.getActiveSpreadsheet().getSheetByName('sheet1');
  var lastRowRegist = sheetRegist.getLastRow();
  var newData = sheetRegist.getRange(lastRowRegist, 1, 1, 4).getValues();
  var A = newData[0][0];
  var B = newData[0][1];
  var C = newData[0][2];
  var D = newData[0][3];

  Logger.log(newID);
  
  var row = findRow(sheetTarget, newData[0]);
  if (row) { 
    sheetTarget.getRange(row, 1, 1, 4).setValues([A, B, C, D]);
  } else { 
    sheetTarget.appendRow([newDate, newID, newType, newTime]);
  }
}

function findRow(sheetTarget, date) {
  var searchDate = Utilities.formatDate(new Date(date), 'Asia/Tokyo','yyyy/MM/dd'); 
  var values = sheetTarget.getDataRange().getValues();

  for (var i = values.length - 1; i > 0; i--) {
    var dataDate = Utilities.formatDate(new Date(values[i][0]), 'Asia/Tokyo','yyyy/MM/dd');
    if (dataDate == searchDate) {
      return i + 1; 
    }
  }
  return false;
}

試したこと

setValuesの中の引数をnewDataにしていてもエラーが出たので、
https://qiita.com/takuya_yamamot/items/54757d4511723a00ec6d
を参考にデータを個別に分けたけれど、エラーが解消されません。

https://teratail.com/questions/245539
上記で「V8 Runtimeのバグ」とありますが、どこをどう修正すると動くようになるでしょうか。。。

Answer 1

//    sheetTarget.getRange(row, 1, 1, 4).setValues([A, B, C, D]);
    sheetTarget.getRange(row, 1, 1, 4).setValues([[A, B, C, D]]);

V8 は関係ないですし、その質問は関係ないです。
(そもそもその質問でもバグはなかった派です)

Comments

[4242]

いつもありがとうございます。ご指摘のところ修正して動きました。

追加で申し訳ありませんが、同じところを
setValues([newData])
にすると
[Ljava.lang.Object;@3219cd26
が入力されるのは何故でしょうか。。。

[papinianus]

[A, B, C, D] → 配列
[[A, B, C, D]] → 配列の配列
- setvalues は配列の配列を設定する必要があります。

newData はそれ自身配列の配列です。なので [newData] は "配列の配列の配列" になります。
setValues は配列の配列の中身をセルに入れますので、getRange(row, 1, 1, 1).setValues([newData]) は特定の 1 つセルに(配列の配列の配列から、配列の配列の皮を剥いて)配列を入れようとします。なのでそういう変なものが入るんじゃないかと想像します。
(あくまで想像ですが、getRange(row, 1, 1, 4).setValues([newData]) ではエラーになるように思いました)
いずれにしても配列の次元があっていないと望むものは設定できません。

[4242]

申し訳ありません。
setValues([[A, B, C, D]])のご指摘で二次元配列かーと気付いたので、
じゃあ
setValues(newData)
でも入るかな？と思ったのですが、なんか変なのが入ってしまうのです。
しかも同じコードを2回動かすときちんと日付が入力されるという謎仕様。。。
とりあえず最初のコードでやりたいことはできるので、素人は深く考えないことにします。
ありがとうございました。

[papinianus]

> setValues([newData]) → ひとつ前のコメント
> setValues(newData) →今回のコメント
は違います。今回のコメントのものなら動くので、きちんと〜、は謎でないです。