如何实现表格行数增加时触发sendSlackMessage并解决行数统计错误？

希望仅当表格行数增加时触发sendSlackMessage函数，当前使用的代码按每分钟间隔运行，但存储的行数属性数值存在错误（比如实际行数为34.0时，旧行数被存为37.0），请问问题出在哪？如何改进代码解决该问题？

现有代码：

function checkNewRows() {
  var sp = PropertiesService.getScriptProperties();
  var oldRows = sp.getProperty("rows") || 0; 
  var newRows = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("Main").getLastRow();
  if (newRows > oldRows) {
    sp.setProperty("rows", newRows);
    sendSlackMessage();
  } 
}

可能的问题原因 #

• 类型不匹配：getProperty返回的是字符串类型，而getLastRow返回数字类型。隐式类型转换时，若存储的字符串有异常（比如之前存过非数字值），就会导致比较逻辑出错，出现数值偏差。
• 并发执行冲突：如果脚本触发间隔过短，前一次执行还没完成就启动下一次，可能会出现旧行数还没更新就被读取的情况，导致存储值异常。
• 行计数逻辑偏差：getLastRow会把有格式（比如背景色）的空行也算入，若表格存在这类行，会导致返回的行数比实际有效行数多。

改进后的代码 #

function checkNewRows() {
  var sp = PropertiesService.getScriptProperties();
  // 明确将存储值转为数字，避免类型错误
  var oldRows = Number(sp.getProperty("rows")) || 0; 
  var sheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("Main");
  if (!sheet) return; // 防止找不到工作表报错
  
  // 可选：如果要统计真正有内容的行（排除空行/格式行），用这段替代getLastRow
  // var dataRange = sheet.getDataRange();
  // var values = dataRange.getValues();
  // var newRows = values.filter(row => row.some(cell => cell !== "")).length;
  
  var newRows = sheet.getLastRow();
  
  // 确保数值有效且确实新增了行，再执行后续操作
  if (typeof newRows === 'number' && newRows > oldRows) {
    // 加锁避免并发执行导致的存储值冲突
    var lock = LockService.getScriptLock();
    try {
      // 最多等待3秒获取锁，防止死锁
      if (lock.tryLock(3000)) {
        sp.setProperty("rows", String(newRows));
        sendSlackMessage();
      }
    } finally {
      if (lock.hasLock()) {
        lock.releaseLock();
      }
    }
  } 
}

额外建议 #

• 调整脚本触发间隔，确保大于单次脚本执行时间，避免并发冲突。
• 如果表格存在大量空行或格式行，优先用代码中注释的方式统计有效行数，替代getLastRow。
• 可以添加日志方便排查问题：

  Logger.log(`旧行数: ${oldRows}, 新行数: ${newRows}`);
  

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