如何改进 Google Apps 脚本中用于拖动多个单元格的脚本？

Question

我这里有一个 google 脚本，它将行从一个 sheet 发送到另一个，然后根据单元格值删除源行。目的是将旧任务发送到单独的 spreadsheet（即从 'task list' 选项卡到 'archived task list' 选项卡）以停止在一个 sheet.[=13= 中过度拥挤]

当我将状态（第 5 列）更改为“存档”时，我的代码成功运行，但是当我尝试一次更改多行（即向下拖动或复制）时，脚本仅适用于一行.我想我必须创建某种循环来阻止这个问题，但我是脚本的新手，所以任何帮助都会很棒！

这是我目前的代码：

function onEdit(event) {
  // assumes source data in sheet named Task List
  // target sheet of move to named Archive Task List
  // test column with "Archive" is col5
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var s = event.source.getActiveSheet();
  var r = event.source.getActiveRange();

  if(s.getName() == "Task List" && r.getColumn() == 5 && r.getValue() == "Archive") {
    var row = r.getRow();
    var numColumns = s.getLastColumn();
    var targetSheet = ss.getSheetByName("Archive Task List");
    var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
    s.getRange(row, 1, 1, numColumns).moveTo(target);
    s.deleteRow(row);
  }
}

更新：我已经实施了建议的代码更改，并且脚本对于单行编辑仍然可以正常工作，但是如果我一次进行 2 次或更多次编辑，循环将归档选定范围之外的一些行。一个例子是，如果我复制单元格 F16:F19 全部带有条目“存档”并粘贴到 col5。我更新的代码是：

function onEdit(event) {
  // assumes source data in sheet named Task List
  // target sheet of move to named Archive Task List
  // test column with "Archive" is col5 
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var s = event.source.getActiveSheet();
  var r = event.source.getActiveRange();

  if(s.getName() == "Task List" && r.getColumn() == 5 && r.getValue() == "Archive") {
    var row = r.getRow();
    var lastRow = r.getLastRow(); //Save the last edited row index
    var numColumns = s.getLastColumn();
    var numRows = lastRow - row == 0? 1: lastRow - row; // numRows has to be at least 1 =, this is in case of a single-line row.
    var targetSheet = ss.getSheetByName("Archive Task List");
    var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
    //s.getRange(row, 1, 1, numColumns).moveTo(target);
    s.getRange(row, 1, numRows, numColumns).moveTo(target); // Take in account the lastRow when archiving the edited range.     
    //s.deleteRow(row);
    for (let i = row; i<=lastRow; i++) {
      s.deleteRow(i);
      console.log(`Deleted row ${i}`);
    }
  }
}

Answer 1

考虑

在您当前的脚本中，您仅使用编辑范围的第一行： var row = r.getRow();。这就是为什么它只需要 multi-lines 编辑范围的第一行。要解决这个问题，您应该考虑编辑范围的最后一行，您可以使用 var lastRow = r.getLastRow();.

轻松获得它

牢记这一点，我建议对您的脚本进行以下修改：

解决方案

第一名：

   var row = r.getRow();
   var lastRow = r.getLastRow(); //Save the last edited row index

第二（已编辑）：

获取整个编辑行的正确语法：getRange(row, column, numRows, numColumns)

    var numRows = lastRow-row + 1; // Corrected this calculation
    s.getRange(row, 1, numRows, numColumns).moveTo(target); // Take in account the lastRow when archiving the edited range.

第 3（已编辑）：

正在删除所有存档行：

for (let i = row; i<=lastRow; i++) {
      s.deleteRow(row); // The row deletion will slide up all the other rows so the next row to delete will have always the same index
      console.log(`Deleted row ${row} at pass number ${i}`);
    }

这是一个循环操作，循环编辑范围内的行。

参考资料

getRange(row,col,numRow,numCol)

for loop

Event Objects

如何改进 Google Apps 脚本中用于拖动多个单元格的脚本？

How do I improve my script for dragging multiple cells in Google Apps Scripts?

javascript

scripting

google-sheets

google-apps-script

考虑

解决方案

参考资料