Google Script中使用变量行号设置单元格值失效求助

我帮你梳理下代码里的几个关键问题，这就是导致用变量i写入J列失效的核心原因：

1. 循环逻辑错位：核心操作在循环体外 #

你现在把邮件发送和单元格赋值的代码都写在了for循环外面，这会导致两个严重问题：

• 只有循环完全结束后才会执行一次sheet.getRange(i, 10).setValue("Yes")，此时i的值已经变成了rowCount（因为循环条件是i < rowCount，最后一次循环后i会递增到等于rowCount），大概率超出了你实际有数据的行范围，所以赋值没效果。
• 邮件也只会发送最后一次循环捕获到的那条数据，前面的所有请假请求都会被忽略。

2. 变量作用域问题 #

循环里定义的requesterName、approverEmail等变量，在循环体外访问时只会保留最后一次循环的值，这会导致邮件内容和预期完全不符。

修复后的完整代码 #

把邮件发送和单元格赋值的逻辑移到循环内部，同时调整循环条件（建议用i <= rowCount，避免漏掉最后一行数据），还优化了脚本性能：

function myFunction() { 
  var ss = SpreadsheetApp.getActiveSpreadsheet(); 
  var sheet = ss.getSheetByName("Form Responses 1"); 
  var rowCount = sheet.getRange('A:A').getLastRow(); 
  var PTOEmail = "lssnihit@gmail.com"; 

  // 调整循环条件为i <= rowCount，确保遍历所有有数据的行
  for(var i=2; i<=rowCount; i++) { 
    // 一次性获取当前行1-9列的值，减少getRange调用次数提升性能
    var rowValues = sheet.getRange(i, 1, 1, 9).getValues()[0];
    var colAValue = rowValues[0];
    var colJValue = sheet.getRange(i, 10).getValue();
    
    if(colAValue != "" && colJValue != "Yes") { 
      var requesterName = rowValues[1]; 
      var requesterEmail = rowValues[2]; 
      var startDate = rowValues[3]; 
      var endDate = rowValues[4]; 
      var sickHours = rowValues[5]; 
      var vacationHours = rowValues[6]; 
      var approverEmail = rowValues[7]; 
      var comStatus = rowValues[8]; 
      var mailSubject = "Leave Information for "+requesterName; 
      var mailBodyOK = "Hi, "+requesterName+" has requested for leave starting from "+startDate+" to "+endDate+". It contains "+sickHours+" sick hours and "+vacationHours+" vacation hours. \n"+requesterName+" has stated that communication with onsite FB manager is done and manager is fine with the leave."; 

      if ( comStatus == "Informed - OK with leave") { 
        MailApp.sendEmail( { to: approverEmail, cc: PTOEmail, subject: mailSubject, htmlBody: "<p> "+mailBodyOK+"</p>" });
      } 

      // 把赋值移到循环内部，确保每满足条件的行都能写入"Yes"
      sheet.getRange(i, 10).setValue("Yes"); 
    }
  }
}

额外优化说明 #

• 减少getRange调用：原来的代码每行多次调用getRange会拖慢脚本，改成一次性获取整行数据存到数组里再取用，能大幅提升运行效率。
• 循环条件修正：getLastRow()返回的是最后一个非空行的行号，用i <= rowCount才能确保遍历到最后一行有效数据。

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