GASを用いた自動転記をしたい(onEdit(e)を使用)

Question

前提・実現したいこと

現在使用しているスプレッドシートにて
1)B列に"済"との変更があった場合、2)該当行のD列からF列に入力されている情報を取得して、3)隣のシートの最終行のD列からF列に自動転記
を行いたく思います。

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

function onEdit(e){　を使用しているため、ログでの確認等が困難で、具体的なミスが解明できません。

該当のソースコード

function onEdit(e) {
 var bk = SpreadsheetApp.getActiveSpreadsheet();
 var sht2 = bk.getSheetByName('sheet2');
 var sht1 = bk.getActiveSheet(); 
  if(e.range.getColumn() === 2) {
   if(e.range.getValue() === "済") {
  //ここまで動作確認済
    var i = e.range.getRow();
    var j = sht1.getRange(i,4,1,3).getvalues();  
      var x = sht2.getRange('D:D');
      var k = x.getLastRow;
    sht2.getRange(k+1,4,1,3).setvalues(j);
    sht2.getRange(k+1,2).setValue("未対応");
  }
 }
}

試したこと

動作確認済まではBrowser.msgboxで確認できましたが、以降はメッセージボックスを記述しても出てこず困っています。

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

以前同様の処理をVBAで記述しており、そちらでの動作確認はできています。
GASに明るくないため、うまく動かせていないのが現状です。

Answer 1

const onEdit = (e) => {
  const targetSheetName = "";
  const sourceSheetName = "";
  if (e.value !== "済") return;
  const curRangw = e.range;
  if (curRange.getSheet().getSheetName() !== sourceSheetName) return;
  if (curRange.getColumn() !== 2) return;
  const dat = ["","未対応",""].concat(curRange.offset(0,2,1,3).getValues()[0]);
  e.source.getSheetByName(targetSheetName).appendRow(dat);
}

onEdit でもログは出せるはずです。出せないとしても、適当なセルに書き込みすればどうなっているかは確認可能だと思います。

Comments

[qck]

丁寧にご回答頂きありがとうございます。
.getLastRow()でシートの最終行(1000行目)を取得していたのが、大きな問題点でした。
JavaScriptも含めてもう少し勉強していければと思います。
ありがとうございました。

Answer 2

目視でチェックしただけですが、getValuesがgetvalues(Vが小文字)になっているのとgetLastRowの呼び出しに()がありません。

 function onEdit(e) {
   var bk = SpreadsheetApp.getActiveSpreadsheet();
   var sht2 = bk.getSheetByName("sheet2");
   var sht1 = bk.getActiveSheet();
   if (e.range.getColumn() === 2) {
     if (e.range.getValue() === "済") {
       //ここまで動作確認済
       var i = e.range.getRow();
-      var j = sht1.getRange(i, 4, 1, 3).getvalues();
+      var j = sht1.getRange(i, 4, 1, 3).getValues();
       var x = sht2.getRange("D:D");
-      var k = x.getLastRow;
+      var k = x.getLastRow();
-      sht2.getRange(k + 1, 4, 1, 3).setvalues(j);
+      sht2.getRange(k + 1, 4, 1, 3).setValues(j);
       sht2.getRange(k + 1, 2).setValue("未対応");
     }
   }
 }

---

function onEdit(e){　を使用しているため、ログでの確認等が困難で、具体的なミスが解明できません。

try ～ catch使ってエラーが出てないか確認するとか色々方法あります。

Comments

[qck]

丁寧にご回答頂きありがとうございます。
.getLastRow()でシートの最終行(1000行目)を取得していたのが、大きな問題点でした。
調べながら改良して無事作動させることができました。
ありがとうございました。