javascript - Google 脚本 - 超出最大执行时间

Question

我构建了一个脚本，可以从多个电子表格中提取数据并计算总值。基本上发生的情况是多名员工输入为特定客户执行特定任务的时间。然后，我想在主电子表格上计算特定客户完成了多少工作，并进行一些分析。

因此，在主电子表格上，每个客户都有一个选项卡。此函数从主电子表格中获取工作表名称，并创建所有客户名称的数组。我这样做的原因是，如果创建新选项卡，该客户名称将自动包含在客户数组中。

function getCurrentCustomers() {
  var sheets = servicesSpreadsheet.getSheets();
  for (var i=0 ; i < sheets.length ; i++) {
    if (sheets[i].getName() != "Services" && sheets[i].getName() != "Employee Files") {
      currentCustomers.push(sheets[i].getName());
    };
  };
};

下一个函数查看特定 Google 云端硬盘文件夹中的所有文件，并以数组形式返回 ID。这允许我创建存储在该特定文件夹中的员工电子表格的副本，并且该电子表格的值将在更改时自动计算。

function listFilesInFolder() {
  var folder = DriveApp.getFolderById("0B7zrWIHovJrKVXlsaGx0d2NFT2c");
  var contents = folder.getFiles();

  var cnt = 0;
  var file;

  while (contents.hasNext()) {
    //Finds the file in the specified folder
    var file = contents.next();
    //Increases the count
    cnt++;
    //Gets the Id of the file
    data = [
      file.getName(),
      file.getId()
      ];
    //Appends it to the employeeId list
    employeeIds.push(data);
  };
  return employeeIds;
};

最后一个函数会大大减慢速度。

首先，我为所有可能的服务创建一个数组。不幸的是，有137个单独的服务。

然后我循环遍历所有客户。对于每个客户，我都会循环查看每项服务，看看它是否出现在任何员工电子表格上。我认为有一种更有效的方法可以做到这一点。在我联系一位完整的客户之前，Google 脚本就超时了。另外，我什至还没有包含所有员工的电子表格。我只是使用虚拟数据进行测试。

function calculateNumbers(){
  var allServices = servicesSpreadsheet.getSheetByName("Services").getRange("Z2:Z137").getValues();
  Logger.log(allServices);
  Logger.log(allServices[0][0]);
  employeeList = listFilesInFolder();

  //Gets services spreadsheet range


  /*Loops through all of the current customers (currentCustomers comes from function getCurrentCustomers)*/
  for (var c = 0; c < currentCustomers.length; c++) {

    var currentCustomer = currentCustomers[c];
    var lastColumn = servicesSpreadsheet.getSheetByName(currentCustomer).getLastColumn();
    var servicesRange = SpreadsheetApp.openById("1X3RRR3UVeot-DYCyXOsfVo0DoKjHezltwBPwUm8ZYig").getSheetByName(currentCustomer).getRange("A4:BC227").getValues();  

    //Loops through all of the services
    var serviceTotal = 0;    
    for (var service = 0; service < allServices.length; service++){

      //Loops through employee spreadsheet Ids
      for (var i = 0; i < employeeList.length; i++) {
        //Get employee spreadsheet ID
        var spreadsheetId = employeeList[i][1];

        //Open the employee spreadsheet by ID
        var employeeSpreadsheet = SpreadsheetApp.openById(spreadsheetId);

        //Get the sheets from the particular employee spreadsheet
        var sheets = employeeSpreadsheet.getSheets();

        //Gets the name of each sheet in the employee spreadsheet
        var sheetsName = [];
        for (var j = 0; j < sheets.length; j++) {
          sheetsName.push(sheets[j].getName());
        };

        //Loops through all of the sheets in an employee spreadsheet ignoring the Services spreadsheet
        for (var q = 0; q < sheetsName.length; q++) {
          if (sheetsName[q] != "Services") {

            //Gets the range of the spreadsheet
            var range = employeeSpreadsheet.getSheetByName(sheetsName[q]).getRange("A5:E1000").getValues();

            //Loops through the range to see if range matches service and customer
            for (var r = 0; r < range.length; r++) {
              if (range[r][3] == allServices[service][0] && range[r][1] == currentCustomer) {
                serviceTotal += range[r][4];
              }; 
            };
          };
        };         
      };

      //Adds the service total to the correct customer's spreadsheet
      for (var serviceServices = 4; serviceServices <= servicesRange.length; serviceServices++){
        var end = 0;
        if (end > 0) {break}
        else if (allServices[service][0] == servicesSpreadsheet.getSheetByName(currentCustomer).getRange(serviceServices,1).getValues()) {          
          servicesSpreadsheet.getSheetByName(currentCustomer).getRange(serviceServices,6).setValue(serviceTotal);
          end += 1;
        };
      };
    };

  }; 
};

第一张图片显示了员工电子表格的样子。第二个显示了单个客户表的样子。主电子表格上有许多客户表。

您会注意到的一件事是，每个服务都有一个类别。我想也许检查一下类别，然后检查特定的服务。有23个类别。

我还希望有一种方法可以只查看已实际完成工作的服务。如果没有人做过营销事件设置，也许可以忽略它。

任何帮助将不胜感激。我为冗长的帖子表示歉意!

Answer 1

您在循环内多次调用服务。这些过程每次可能需要几秒钟，并且通常被认为非常慢。这是反对Apps Script best practice因为它大大增加了您的执行时间。

TL;DR:您可能正在调用 Apps 脚本服务数千次。每个调用最多可能需要几秒钟的时间来执行。您需要在循环之外缓存您的服务调用，否则您的性能将受到严重影响。

示例:

1:

    if (sheets[i].getName() != "Services" && sheets[i].getName() != "Employee Files")

使用工作表名称创建并设置一个变量一次，然后检查该变量，而不是调用 getName() 方法两次。这不是什么大问题，但会增加执行时间。

2:

这是一个大问题，因为它在 calculateNumbers 的循环中深一层

var lastColumn = servicesSpreadsheet.getSheetByName(currentCustomer).getLastColumn();
var servicesRange = SpreadsheetApp.openById("1X3RRR3UVeot-DYCyXOsfVo0DoKjHezltwBPwUm8ZYig").getSheetByName(currentCustomer).getRange("A4:BC227").getValues();

2a: 您将打开一个新电子表格、一个新工作表，然后获取同一张工作表的范围，并为 servicesRange 每次循环获取一次该范围的值。这些服务调用会快速堆积起来，并增加您的执行时间。

2b: 我看到您正在获取 lastColumn，但我没有看到它在任何地方使用？也许我错过了一些东西，但是在为每个循环进行服务调用时未使用它会增加更多的执行时间。

3:

这是巨大的，您可能会在这里调用 Apps 脚本服务数千或数万次。该代码片段已经有两个循环层深。

//Loops through all of the sheets in an employee spreadsheet ignoring the Services spreadsheet
        for (var q = 0; q < sheetsName.length; q++) {
          if (sheetsName[q] != "Services") {

            //Gets the range of the spreadsheet
            var range = employeeSpreadsheet.getSheetByName(sheetsName[q]).getRange("A5:E1000").getValues();

            //Loops through the range to see if range matches service and customer
            for (var r = 0; r < range.length; r++) {
              if (range[r][3] == allServices[service][0] && range[r][1] == currentCustomer) {
                serviceTotal += range[r][4];
              }; 
            };
          };
        };         
      };

      //Adds the service total to the correct customer's spreadsheet
      for (var serviceServices = 4; serviceServices <= servicesRange.length; serviceServices++){
        var end = 0;
        if (end > 0) {break}
        else if (allServices[service][0] == servicesSpreadsheet.getSheetByName(currentCustomer).getRange(serviceServices,1).getValues()) {          
          servicesSpreadsheet.getSheetByName(currentCustomer).getRange(serviceServices,6).setValue(serviceTotal);
          end += 1;
        };
      };
    };

您在调用相同服务的嵌套循环中有多层嵌套循环。如果您有一个三次嵌套循环，每个级别迭代 10 次，并且底层每个循环调用一次服务。您将调用 Apps 脚本服务 1,000 次。保守地说，假设每次服务调用 0.5 秒，您已经有 8.3 分钟的执行时间。

缓存您的服务调用并执行批量操作。