Google Script中forEach与setValue无法正常工作的问题

你的代码核心问题是循环内的行索引变量i没有正确关联到当前遍历的行，导致每次满足条件时都会修改第1行第4列的单元格，而非当前行。另外直接在循环里逐个修改单元格效率较低，下面是具体修正方案：

核心问题拆解 #

• var i = 1放在forEach循环内部，每次迭代都会重置i为1，所以ws.getRange(i,4)永远指向第1行第4列，完全没用到当前遍历的行。
• forEach本身可以接收第二个参数（当前元素的索引），直接用这个索引就能定位到对应行，不需要手动维护i变量。
• 循环内频繁调用setValue会触发多次表格API请求，效率低下，建议先收集需要更新的行，最后批量写入。

修正后的代码 #

const ss = SpreadsheetApp.getActiveSpreadsheet();
const ws = ss.getSheetByName("Repairs");
const rows = ws.getDataRange().getValues();
// 收集需要更新的行号（表格行号从1开始，数组索引从0开始，需+1转换）
const rowsToUpdate = [];

rows.forEach(function(eachRow, index) {
  const rowNum = index + 1;
  // 空值保护：避免单元格为空时抛出错误
  if (eachRow[3]?.includes("Complete") && !eachRow[4]?.includes("Posted")) {
    const currentMessage = eachRow[0] + "\n" + eachRow[1] + "\n" + eachRow[2];
    RepairsCompleted(currentMessage);
    // 记录需要标记为"Posted"的行号
    rowsToUpdate.push(rowNum);
  }
});

// 批量更新，减少API调用次数提升效率
if (rowsToUpdate.length > 0) {
  const rangeList = rowsToUpdate.map(row => `D${row}`);
  ws.getRangeList(rangeList).setValue("Posted");
}

关键修改说明 #

1. 正确关联行号：利用forEach的第二个参数index，将数组索引转换为表格实际行号（index + 1）。
2. 空值容错：用?.链式调用避免单元格为空时，includes方法抛出报错。
3. 批量更新优化：先收集所有需要修改的行，通过getRangeList一次性完成写入，大幅减少API请求次数。
4. 简化冗余代码：移除了无意义的p变量，让代码更简洁。

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