使用Apps Script结合For循环与If条件查找列值并跨工作表拷贝同行数据

核心问题答复：不能将Device_attachment的判断条件直接放到代码最后一行。 拷贝配方的操作必须建立在附件匹配成功的前提上，若将判断后置，会先把所有行的配方全部拷贝到目标表后再做匹配校验，完全不符合业务需求。

现有代码问题排查 #

你遇到的匹配到Device_attachment2/Device_attachment3等非目标附件的问题，核心是现有代码存在以下缺陷：

• 语法错误：最外层条件if(category == "Device"{缺少右括号，会导致整个逻辑块的判断完全失效，是出现非预期操作的核心诱因
• 匹配逻辑不严谨：没有处理单元格前后空格、换行符等不可见字符，可能出现字符串比对异常
• 冗余操作过多：循环内重复调用getSheetByName、getRange等接口，数据量较大时执行效率极低
• 硬编码风险：循环内的匹配值写死为固定字符串，没有复用外层的attachment变量，后续修改匹配规则时容易出现逻辑不一致

优化后可直接使用的代码 #

if(category == "Device"){
  if(attachment == "Device_attachment"){
    const sheet_RC = ss.getSheetByName("RecipeCollection");
    const sheet_TP = ss.getSheetByName("TestPlan");
    // 批量读取第二列（附件）、第三列（配方名）的所有有效数据，减少接口调用次数
    const rcAllData = sheet_RC.getRange(2, 2, sheet_RC.getLastRow() - 1, 2).getValues();
    const matchedRecipes = [];
    // 遍历匹配目标附件
    for(let i = 0; i < rcAllData.length; i++){
      // trim()清除前后不可见空格，避免空格导致的匹配错误
      if(rcAllData[i][0].trim() === attachment){
        matchedRecipes.push([rcAllData[i][1]]);
      }
    }
    // 批量写入所有匹配到的配方到目标表
    if(matchedRecipes.length > 0){
      sheet_TP.getRange(sheet_TP.getLastRow() + 1, 2, matchedRecipes.length, 1).setValues(matchedRecipes);
    }
  }
}

额外优化建议 #

如果调整后仍然存在异常匹配问题，可在循环内增加日志打印确认实际读取到的单元格内容：console.log("第"+(i+2)+"行附件值："+rcAllData[i][0])，运行后在 Apps Script 执行日志里查看对应单元格的实际值即可快速定位问题。

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