勾选/取消勾选时工作表间行移动脚本的问题排查

问题说明 #

• 我是Apps Script及编程新手，使用网上找到的脚本实现：勾选第一列复选框时，将行从"To Do"工作表移到"Completed"；取消勾选时移回原表。仅修改了工作表名称和目标列，但出现两个问题：
  1. 取消勾选移回"To Do"时，行有时会消失，或移回后误删其他行
  2. 函数名从onEdit改为moveChecked后，第3、4行的mainSheet和sheetToMoveTheRow变量提示"未使用声明"（之前用onEdit无报错，已删除原触发器）

原代码如下：

function moveChecked() {

  var mainSheet = "To Do";
  var sheetToMoveTheRow = "Completed";
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = SpreadsheetApp.getActiveSheet();
  var range = SpreadsheetApp.getActiveRange();

  if(sheet.getName() == "To Do" && range.getColumn() == 1 && range.getValue() == true) {
    var row = range.getRow();
    var numColumns = sheet.getLastColumn();
    var targetSheet = ss.getSheetByName("Completed");
    var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
    sheet.getRange(row, 1, 1, numColumns).moveTo(target);
    sheet.deleteRow(row);}
  
  else if(sheet.getName() == "Completed" && range.getColumn() == 1 && range.getValue() == false) {
    var row = range.getRow();
    var numColumns = sheet.getLastColumn();
    var targetSheet = ss.getSheetByName("To Do");
    const aVals = targetSheet.getRange(1,1,targetSheet.getLastRow(),1).getValues();
    const targetRow = aVals.filter(String).length + 1;
    var target = targetSheet.getRange(targetRow , 1);
    sheet.getRange(row, 1, 1, numColumns).moveTo(target); 
    sheet.deleteRow(row);}
    
}

修复后的代码 #

function moveChecked() {
  const ss = SpreadsheetApp.getActiveSpreadsheet();
  const sheet = SpreadsheetApp.getActiveSheet();
  const range = SpreadsheetApp.getActiveRange();
  const row = range.getRow();
  
  // 跳过表头行（假设第一行是表头）
  if (row <= 1) return;

  // 从To Do移到Completed（勾选复选框）
  if (sheet.getName() === "To Do" && range.getColumn() === 1 && range.getValue() === true) {
    const numColumns = sheet.getLastColumn();
    const targetSheet = ss.getSheetByName("Completed");
    const target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
    sheet.getRange(row, 1, 1, numColumns).moveTo(target);
    sheet.deleteRow(row);
  } 
  // 从Completed移回To Do（取消勾选复选框）
  else if (sheet.getName() === "Completed" && range.getColumn() === 1 && range.getValue() === false) {
    const numColumns = sheet.getLastColumn();
    const targetSheet = ss.getSheetByName("To Do");
    // 统一用getLastRow()+1获取目标行，避免空值导致的计算错误
    const target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
    sheet.getRange(row, 1, 1, numColumns).moveTo(target);
    sheet.deleteRow(row);
  }
}

问题修复说明 #

1. 未使用变量提示问题：直接删除定义了但未使用的mainSheet和sheetToMoveTheRow变量即可；也可以用这两个变量替换代码中的硬编码工作表名，让代码更易维护。
2. 行消失/误删问题：
   • 原代码移回To Do时，用aVals.filter(String).length计算目标行，若To Do表第一列存在空值，会导致目标行计算错误，可能覆盖已有行或插入到错误位置，后续删除操作就会误删其他行。
   • 修复后统一使用targetSheet.getLastRow() + 1获取目标行，确保行总是追加到目标表的最后，不会出现错位或覆盖。
   • 添加了表头判断if (row <= 1) return，避免误操作表头行。

内容的提问来源于stack exchange，提问作者GrisCasey