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 Script会超时。此外，我还没有为所有员工提供电子表格。我只是使用虚拟数据进行测试。

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分钟的执行时间。

缓存您的服务电话，并执行批量操作。

Google脚本 - 超出了最长执行时间

1 个答案: