【GAS】getRange.setValues() エラーの解消方法

Question

前提

(10/27　20:14追記)
投稿しましたが、エラーは自己解決できました。
下記<2><3>部分のみアドバイスをいただけます
でしょうか?

次の2つのスプレッドシートファイルがあります。
①個票
②全体票

<①個票のイメージ>

<②全体票>のイメージ

実現したいこと

GASで①個票のB4:E9の内容を全体票に転記させたいです。
個票のデータはgetRange().getValues()で一度にまとめて
二次元配列として取得。

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

ログで見ると二次元配列でデータは取得できていそうです。
が、下のエラーがたってしまいました。
何か基本的なことを見落としている気がするのですが、
コードのどこがマズイのかが分かりません。

<1>解消するためのアドバイスをいただけないでしょうか?

<2>次のどちらがおすすめでしょうか?
setValuesで配列内の列単位で転記していくコードで
記載しましたが、報告1、(入力日、入力者、推移)　報告2、、
と行単位で転記していく方法も可能かと思います。
どちらのアプローチがよいのでしょうか?
その理由も教えて欲しいです。

<3>上記<2>で行単位で転記の場合のコード記載
let rep1　= fromRange[r];　、、、等
行インデックスのみ指定する方法に変更する形でしょうか?
この場合、転記先ファイルへのデータ転記部分のコードのgetRangeの
(行,列,行分,列分)の4つ目の引数? は3に変更でしょうか?

Exception: The parameters ((class)) don't match the method signature for SpreadsheetApp.Range.setValues.
myFunction	@ 個票⇒全体票.gs:49

該当のソースコード

以下、コード全文記載します。

(自己解決コード) ------------------------------------------------------------
function myFunction(indvToall) {

  //ALLファイルURL
  // https://docs.google.com/spreadsheets/d/1NbSg5b2aW3U1-RaOPHsh6ZUTFp5WAxeKQ1WpL3oF5sQ/edit#gid=0

  //ALLファイルID
  //1NbSg5b2aW3U1-RaOPHsh6ZUTFp5WAxeKQ1WpL3oF5sQ

  const fromss = SpreadsheetApp.getActiveSpreadsheet();
  const toss = SpreadsheetApp.openById("1NbSg5b2aW3U1-RaOPHsh6ZUTFp5WAxeKQ1WpL3oF5sQ");
  const fromSheet = fromss.getSheetByName('入力シート');
  const toSheet = toss.getSheetByName('全体進捗');

  //▼個票ファイルのファイルID(スプレッドシートID)の取得
  fromId = fromss.getId();
  fromURL = fromss.getUrl();
  console.log('fromId ' + fromId);
  console.log('fromURL ' + fromURL);

  //▼転記先ファイル(ALL)の転記行(targetRow)の取得
  //個票のセルF10から取得
  const targetRow = fromSheet.getRange(10, 6, 1, 1).getValue(); //　6が正解
  console.log('targetRow ' + targetRow);

  //▼転記先ファイル(ALL) B列に転記元ファイル(個票)URLを転記
  fromUrl = toSheet.getRange(targetRow, 2, 1, 1).setValue(fromURL);
  console.log('fromUrl ' + fromUrl);

  //▼転記先ファイル(ALL) U列に転記元ファイル(個票)IDを転記
  fromUrl = toSheet.getRange(targetRow, 21, 1, 1).setValue(fromId);
  console.log('fromId ' + fromId);

  fromRange = fromSheet.getRange(4, 2, 6, 3).getValues();
  console.log('fromRange ' + fromRange);

  //▼個票ファィルのデータ取得
  //B4～D9までを二次元配列(fromRange)で取得
  //二次元配列(fromRange)
  for (let r = 0; r <= 5; r++) {
    let date = fromRange[r][0];　　  //入力日
    let name = fromRange[r][1];　　  //入力者
    let cond = fromRange[r][2];　　  //概要  
    console.log('date ' + date);
    console.log('name ' + name);
    console.log('cond ' + cond);

    //▼転記先ファイル(ALL)へのデータ転記
    //状況報告1欄への転記
    //getRange(行,列,行分,列分)
    const setdate = toSheet.getRange(targetRow, 3 + r *3, 1, 1).setValue(date);
    const setname = toSheet.getRange(targetRow, 4 + r *3, 1, 1).setValue(name);
    const setcond = toSheet.getRange(targetRow, 5 + r *3, 1, 1).setValue(cond);

    //▼個票ファィルの「自動処理日時」欄(E列)にタイムスタンプを追記  
    const timeStamp = new Date();
    console.log(timeStamp);
    fromSheet.getRange(4 + r, 5, 1, 1).setValue(timeStamp);

  }
}

(誤コード) --------------------------------------------------------------------
function myFunction(indvToall) {

  //ALLファイルURL
  // https://docs.google.com/spreadsheets/d/1NbSg5b2aW3U1-RaOPHsh6ZUTFp5WAxeKQ1WpL3oF5sQ/edit#gid=0

  //ALLファイルID
  //1NbSg5b2aW3U1-RaOPHsh6ZUTFp5WAxeKQ1WpL3oF5sQ

  const fromss = SpreadsheetApp.getActiveSpreadsheet();
  const toss = SpreadsheetApp.openById("1NbSg5b2aW3U1-RaOPHsh6ZUTFp5WAxeKQ1WpL3oF5sQ");
  const fromSheet = fromss.getSheetByName('入力シート');
  const toSheet = toss.getSheetByName('全体進捗');

  //▼個票ファイルのファイルID(スプレッドシートID)の取得
  fromId = fromss.getId();
  fromURL = fromss.getUrl();
  console.log('fromId ' + fromId);
  console.log('fromURL ' + fromURL);

  //▼転記先ファイル(ALL)の転記行(targetRow)の取得
  //個票のセルF10から取得
  const targetRow = fromSheet.getRange(10, 6, 1, 1).getValue(); //　6が正解
  console.log('targetRow ' + targetRow);

  //▼転記先ファイル(ALL) B列に転記元ファイル(個票)URLを転記
  fromUrl = toSheet.getRange(targetRow, 2, 1, 1).setValue(fromURL);
  console.log('fromUrl ' + fromUrl);

  //▼転記先ファイル(ALL) U列に転記元ファイル(個票)IDを転記
  fromUrl = toSheet.getRange(targetRow, 21, 1, 1).setValue(fromId);
  console.log('fromId ' + fromId);

  //▼個票ファィルのデータ取得
  //B4～D9までを二次元配列(fromRange)で取得
  //二次元配列(fromRange)
  for (let r = 0; r <= 6; r++) {
    fromRange = fromSheet.getRange(4, 2, 6, 3).getValues();
    console.log('fromRange ' + fromRange);
    let date = fromRange[r][0];　　  //入力日
    let name = fromRange[r][1];　　  //入力者
    let cond = fromRange[r][2];　　  //概要  
    console.log('date ' + date);
    console.log('name ' + name);
    console.log('cond ' + cond);

    //▼転記先ファイル(ALL)へのデータ転記
    //状況報告1欄への転記
    //getRange(行,列,行分,列分)
    const setdate = toSheet.getRange(targetRow, 3 + r *3, 1, 1).setValues(date);
    const setname = toSheet.getRange(targetRow, 4 + r *3, 1, 1).setValues(name);
    const setcond = toSheet.getRange(targetRow, 5 + r *3, 1, 1).setValues(cond);

    //▼個票ファィルの「自動処理日時」欄(E列)にタイムスタンプを追記  
    const timeStamp = new Date();
    console.log(timeStamp);
    fromSheet.getRange(4 + r, 5, 1, 1).setValue(timeStamp);

  }
}

試したこと

(参考) ログイメージ

補足情報（FW/ツールのバージョンなど）

お忙しいところ申し訳ありませんが、
非エンジニアビギナーでも理解できるレベルのアドバイス、解説を
いただけると助かります。
よろしくお願いいたします。

Answer 1

<2>

「setValuesで配列内の列単位で転記していくコードで記載しましたが、報告1、(入力日、入力者、推移)　報告2、、と行単位で転記していく方法も可能かと思います。」
どちらのアプローチがよいのでしょうか?
その理由も教えて欲しいです。

　
仮に、後半の「報告1、(入力日、入力者、推移)　報告2、、と行単位で転記していく方法」というのが
「(入力日、入力者、推移)の３つをまとめて setValues を使って転記する方法」ということであるならば、そちらを推奨します。

• 理由

現状は、入力日、入力者、推移 それぞれを転記する際に１つ１つ setValue() を呼び出しているため、「個票」シートの6行のデータを書き込むのに setValue を18回呼び出しています。
仮に setValues で入力日、入力者、推移 をまとめて書き込めば、「個票」シートの6行のデータを書き込むのに setValues を 6 回呼び出すだけで済みます。
結果、処理の高速化が期待できます。（現状は個票１ページの転記だけなのでほとんど差は感じられないと思いますが）

<3>上記<2>で行単位で転記の場合のコード
下記のように、 setValues で date、name、cond を配列にしてまとめて書き込みます。
具体的には、コード終盤の下の3行を

    const setdate = toSheet.getRange(targetRow, 3 + r *3, 1, 1).setValues(date);
    const setname = toSheet.getRange(targetRow, 4 + r *3, 1, 1).setValues(name);
    const setcond = toSheet.getRange(targetRow, 5 + r *3, 1, 1).setValues(cond);

↓
以下の1行に書き換えます。

    const setdate = toSheet.getRange(targetRow, 3 + r *3, 1, 3).setValues([[date, name, cond]]);
    // const setname = toSheet.getRange(targetRow, 4 + r *3, 1, 1).setValues(name);  // この行は削除
    // const setcond = toSheet.getRange(targetRow, 5 + r *3, 1, 1).setValues(cond);  // この行も削除

Comments

[donguriko]

qnoirさま
お忙しい中、レクチャーありがとうございます。

どう記載すれば(入力日、入力者、推移)の3つを1setで配列から呼び足せるかが
分からなかったのですが、setValues([[date, name, cond]])と記載すれば
良かったのですね。しっかり覚えて自分のモノにしたいと思います。

配列は、
getValues()でデータを取得する際は、できるだけ大きな範囲で値を取得し、
setValues()取得したデータを呼び出す時には、できるだけ呼び出し回数が
少なくなるように呼び出すべし、ですね。
勉強になりました。ありがとうございました。(感謝)

以前よりは配列について、少し分かるようになってきたものの
まだまだですね。。。
引き続き勉強していきたいと思います。
お忙しい中、いつも丁寧な解説　本当にありがとうございます!! (感謝)