合并行移动脚本时Switch函数失效问题排查求助

存在的错误点 #

• 变量引用错误：
  • 已定义r = e.range，但后续错误使用range.sheet，应改为r.sheet
  • 多处使用未声明的变量s，实际应为之前定义的sheet变量
• 逻辑匹配错误：
  • 第9列（Col9）的目标工作表误写为"Bokad"，根据需求应改为"Nej"
  • Switch的case 13不符合需求，需求中Col10对应"Avtal"工作表，应改为case 10
  • 部分注释内容与代码逻辑不符（如case9中注释仍指向column8）
• 语法错误：
  • moveTo(target).range.clearContent()写法错误，moveTo返回目标范围，无需额外.range，且原行内容清除需单独操作原行范围
• 结构错误：
  • 代码末尾多了一个冗余的闭合括号}

修正后的完整代码 #

function allInOne(e) {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var r = e.range;
  var sheet = r.sheet; // 修正变量引用
  var sheetName = sheet.getName();
  
  if (e.value != "") {
    if (sheetName == "Pipe") {
      switch (r.getColumn()) {
        case 7:
          var row = r.getRow();
          var numColumns = sheet.getLastColumn(); // 替换未定义的s为sheet
          var targetSheet = ss.getSheetByName("Återkomster");
          var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
          // 先获取原行范围，移动后清除原内容
          var sourceRange = sheet.getRange(row, 2, 1, numColumns);
          sourceRange.moveTo(target);
          sourceRange.clearContent();
          break;

        case 8:
          var row = r.getRow();
          var numColumns = sheet.getLastColumn();
          var targetSheet = ss.getSheetByName("Bokad");
          var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
          var sourceRange = sheet.getRange(row, 2, 1, numColumns);
          sourceRange.moveTo(target);
          sourceRange.clearContent();
          break;

        case 9:
          var row = r.getRow();
          var numColumns = sheet.getLastColumn();
          var targetSheet = ss.getSheetByName("Nej"); // 修正目标工作表
          var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
          var sourceRange = sheet.getRange(row, 2, 1, numColumns);
          sourceRange.moveTo(target);
          sourceRange.clearContent();
          break;

        case 10: // 修正case编号为10，对应Col10
          var row = r.getRow();
          var numColumns = sheet.getLastColumn();
          var targetSheet = ss.getSheetByName("Avtal");
          var target = targetSheet.getRange(targetSheet.getLastRow() + 1, 1);
          var sourceRange = sheet.getRange(row, 2, 1, numColumns);
          sourceRange.moveTo(target);
          sourceRange.clearContent();
          break;

        default:
          return;
      }
    }
  }
}

额外优化建议 #

• 可以提取重复逻辑为单独函数，减少代码冗余，比如将移动行的操作封装成moveRowToSheet函数
• 增加容错处理，比如判断目标工作表是否存在，避免因工作表名称错误导致脚本崩溃

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