GAS GoogleSpreadsheet 複数のセルを指定したシートの一番下へコピーさせたい

Question

前提・実現したいこと

お世話になっております。
以前に引き続きGAS GoogleSpreadSheetを使って簡易ＤＢを作っています。
現在、前日のログをを集計用のブックの一番下の行へ保存する機能を実装中に以下のエラーメッセージが発生しました。

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

データの行数が範囲の行数と一致しません。データは 899 行ですが、範囲は 1.行です。（行 200

function mySaveDaylog()
{
//前日のログを呼び出す。
var setdate = Utilities.formatDate(new Date(),"JST","yyyy年M月d日");
var Cellday = SpreadsheetApp.getActiveSpreadsheet();
var Cellday1 = Cellday.getSheetByName('関数用');
var Cellday2 = Cellday1.getRange('A1').setValue(setdate);

//月毎のログへコピーさせる。

var copyrange = SpreadsheetApp.getActiveSpreadsheet()
var copyrange1 = copyrange.getSheetByName('結果用');

var copy_to = SpreadsheetApp.openById('コピー先のスプレッドシート')
var copy_to2 = copy_to.getSheetByName('ログ')

var copy_value = copyrange1.getRange('A2:FX900').getValues();

var targetRow = copy_to2.getLastRow()+1;
copy_to2.getRange('A'+targetRow+':FX'+targetRow).setValues(copy_value);

}

GAS

試したこと

コピー元が１行のみの場合はコピーすることができました。
もしかして複数の行をコピーする場合は別の関数を使ったほうがよいのでしょうか。

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

Google Chrome 最新版

参考にした質問/回答：https://teratail.com/questions/112154

Answer 1

setValuesするデータは複数行に対して、getRangeしているのは1行だけなので、エラーが出ています。
copy_to2.getRange('A'+targetRow+':FX'+(targetRow+copy_value.length-1)).setValues(copy_value);
こうすればうまくいくんじゃないかなぁ。

Comments

[int_hori]

さっそく試してみましたら解決しました
ありがとうございます！お世話になりました

Answer 2

copy_to2.getRange('A'+targetRow+':FX'+ (targetRow + copy_value.length - 1)).setValues(copy_value);

Comments

[papinianus]

ひさびさにだだかぶったので、追記。
要件次第だけど、なんか他にやりようがある気がする。

[int_hori]

回答いただきありがとうございます。
今回は回答の早かったmacaron_xxxさんをＢＡにさせていただきました。

＞要件次第だけど、なんか他にやりようがある気がする。
　やはりそうですか。
コードも無駄が多いなと感じてはいたので今度ももっと勉強していきたいです。
　またなにかありましたらよろしくお願いします。

[papinianus]

どういうログわからないのですがあらかじめ月毎シートに記録できないのかなーと

[int_hori]

直に月毎シートへ記入させる方法も考えたのですが（そちらのほうが簡単ですので）
入力フォームが複数あり、それら一つひとつに関数を作っているので
月毎の忙しいときに更新するのは面倒だなぁと思い
「データを一度集約させる」→「月毎シートへまとめてコピー」という形をとりました
また前日のログを呼び出す部分をＧＡＳではなく
スプレッドシートで関数を組んでおこなっているためというのもあります。
（これはＧＡＳ上でうまく組めなかった完全な私の力不足ですね）