Google App Script性能优化与方法返回值问题求助

一、修复location方法返回undefined的问题 #

你的代码中location方法返回undefined的核心原因：调用lSearch(searchTerm)后未返回其执行结果，直接执行return会默认返回undefined；同时循环中创建函数存在闭包陷阱，后续调用location()时，searchTerm可能指向循环最后一次的值，而非当前item对应的搜索词。

修复后的代码示例 #

将原location函数修改为直接返回lSearch的执行结果，并通过立即执行函数绑定当前搜索词，避免闭包问题：

var item = {
  'entryMadeby': dataBase[i][1],
  'checkinDate': dataBase[i][2],
  'productCode': dataBase[i][3],
  'description': dataBase[i][4],
  'location': (function(currentTerm) {
    return function() {
      return lSearch(currentTerm);
    };
  })(searchTerm),
  'checkoutdate': dataBase[i][6]
};

修改后调用item.location()即可正确返回lSearch的执行结果。

二、优化多工作表数据读取的性能与可维护性 #

当前逐个声明工作表变量的方式代码冗余，新增工作表时需大量修改代码，且重复调用getDataRange()会增加Spreadsheet服务请求次数，影响性能。可通过循环批量处理工作表，提升代码简洁度与性能。

优化后的完整代码 #

function locationSearch(searchTerm, selectedLocation) {
  if (searchTerm == '') { return []; }

  const ss = SpreadsheetApp.getActive();
  const searchResults = [];
  const schoolLocationNames = [
    "ALTUS_ACADEMY", "ANNUNCIATA", "EPIPHANY", "GRACE", "HOLY_FAMILY",
    "INNOVATIVE_SCHOLARS", "ISLAMIC_CC_OF_IL", "JOSEPHINUM_HS",
    "MATERNITY_BVM", "POPE_FRANCIS", "PROVIDENCE", "PUI_TAK", "ST_AILBE",
    "ST_BORGIA", "ST_ELIZABETH", "ST_PAUL"
  ];

  // 批量读取所有工作表数据，关联工作表名称
  const schoolDataList = schoolLocationNames.map(name => {
    const sheet = ss.getSheetByName(name);
    if (!sheet) return null;
    const data = sheet.getDataRange().getValues().slice(1); // 跳过表头
    return { name, data };
  }).filter(item => item !== null); // 过滤无效工作表

  // 根据selectedLocation筛选目标数据
  const targetDataList = selectedLocation 
    ? schoolDataList.filter(item => item.name === selectedLocation)
    : schoolDataList;

  // 遍历搜索
  targetDataList.forEach(({ name, data }) => {
    data.forEach(row => {
      if (row.includes(searchTerm)) {
        const item = {
          entryMadeby: row[1],
          checkinDate: row[2],
          productCode: row[3],
          description: row[4],
          location: name, // 若无需调用lSearch，直接用工作表名称更高效
          checkoutdate: row[6]
        };
        searchResults.push(item);
      }
    });
  });

  return searchResults;
}

优化点说明 #

• 用map循环批量处理工作表，新增工作表只需在schoolLocationNames数组添加名称即可，无需修改其他代码
• 关联工作表名称与数据，若业务允许可直接用名称作为location值，避免额外调用lSearch，提升性能
• 减少重复的Spreadsheet服务调用，降低性能开销
• 添加无效工作表过滤逻辑，避免因工作表不存在导致报错

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