Google Apps Script循环中If语句仅匹配首个项问题求助

先直接拆解几个核心问题，这些应该就是导致你遇到匹配异常的根源：

1. 学生邮箱数组被反复覆盖，丢失前序列数据 #

你的第一个循环里，每次遍历P1到P6列时，只要该列有值就会重新赋值sendToStudents变量，而不是把新学生添加到数组里。比如遍历到P2列时，P1列的学生数据就被覆盖了，最后sendToStudents里只会保留P6列的学生——这直接导致前面列的学生根本没进入后续匹配循环。

修复方式：把sendToStudents初始化为空数组，每次遍历到有学生的列时，用concat合并拆分后的数组：

// 初始化空数组，避免反复覆盖
var sendToStudents = []; 
for (let c = 4; c <= 9; c++) { 
  var students = sheet1.getRange(lastRow, c).getValue(); 
  if (students) { 
    // 将当前列的学生合并到总数组
    sendToStudents = sendToStudents.concat(students.split(',')); 
  } 
}

2. 循环变量拼写错误 #

第二个循环的条件里，你写的是sendToStudent.length（少了末尾的s），这会导致引用未定义变量。虽然你提到首个学生能匹配，大概率是笔误，但这个必须修正：

// 修正变量名：sendToStudents 而非 sendToStudent
for (let s = 0; s < sendToStudents.length; s++) { 

3. 匹配逻辑未使用处理过空格的字符串 #

你已经用replace(/\s+/g, '')处理了当前学生的邮箱，但匹配判断时却用的是原始的sendToStudents[s]，而非处理后的student变量。哪怕日志里确认字符串一致，也可能因为判断逻辑没用到处理后的值导致匹配失败：

var student = sendToStudents[s].replace(/\s+/g, ''); 
for (let r = 1; r < allRosterValues.length; r++) { 
  var studentInRoster = allRosterValues[r][2].replace(/\s+/g, ''); 
  // 使用处理后的纯字符串做匹配
  if (student === studentInRoster) { 
    Logger.log("SEND EMAIL HERE"); 
    break; 
  }
}

4. 未声明循环变量，存在全局污染风险 #

你的c、s、r都没有用let/const/var声明，会变成全局变量，可能在循环过程中出现意外的变量值覆盖，建议都加上声明。

5. 循环条件优化（非致命，但更高效） #

用allRosterValues.length代替sheet2.getLastRow()作为循环上限——你已经获取了allRosterValues二维数组，直接用它的长度比再次调用getLastRow()更高效，也能避免数据不一致的可能。

修正后的完整代码 #

function homeworkEmail() { 
  var lastRow = sheet1.getLastRow(); 
  // 初始化空数组存储所有学生邮箱
  var sendToStudents = []; 

  // 遍历P1到P6列，收集所有需要发送提醒的学生
  for (let c = 4; c <= 9; c++) { 
    var students = sheet1.getRange(lastRow, c).getValue(); 
    if (students) { 
      // 拆分当前列的邮箱并合并到总数组
      sendToStudents = sendToStudents.concat(students.split(',')); 
    } 
  } 

  var period = sheet1.getRange(lastRow, 3).getValue(); 
  var assignment = sheet1.getRange(lastRow, 10).getValue(); 
  var due = sheet1.getRange(lastRow, 11).getValue(); 

  var allRosterValues = sheet2.getDataRange().getValues(); 

  // 遍历所有待发送学生，匹配花名册
  for (let s = 0; s < sendToStudents.length; s++) { 
    // 清理当前学生邮箱的空格
    var student = sendToStudents[s].replace(/\s+/g, ''); 
    // 遍历花名册查找匹配项
    for (let r = 1; r < allRosterValues.length; r++) { 
      var studentInRoster = allRosterValues[r][2].replace(/\s+/g, ''); 
      // 用处理后的纯字符串做匹配判断
      if (student === studentInRoster) { 
        Logger.log("SEND EMAIL HERE"); 
        // 找到匹配后跳出内层循环，避免无效遍历
        break; 
      } 
    } 
  } 
}

你可以先测试这个修正后的代码，核心解决了学生数组被覆盖的问题，同时修正了匹配逻辑和变量拼写错误。如果还有问题，可以打印sendToStudents的完整数组，确认所有需要的学生都在里面，再检查匹配时的日志输出。

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