2

我编写了一个脚本，它从电子表格中获取行数据并循环遍历它们，如果行的数据满足某些条件（例如，有电话号码并且尚未发送），则调用一个函数来发送 SMS。

然而，在添加大约 600 行之后，脚本执行时间超过了它的限制，根据我的研究，这似乎是 5 分钟。我正在使用 JavaScript 对象来读取数据，并使用 for 循环来遍历行。

如果有可能让它更快，谁能给我打电话？我对编程很陌生，但是对于所有这些计算能力来说，这似乎是一项轻而易举的任务，我不明白为什么需要这么长时间

提前致谢！

这是我正在使用的函数的代码：

// Will send SMS on the currently active sheet
function sendSms() {

  // Use the send sms menu to trigger reconcile
  var user = ScriptProperties.getProperty(PROPERTY_USER_RECONCILE);
  if (user == null)
    reconcileUser();

  // The sheets
  var sheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("Registo");
  var settingsSheet = SpreadsheetApp.getActiveSpreadsheet().getSheetByName("Settings");

  // Fetch values for each row in the Range.
  var startRow = 2;
  var apiKey = settingsSheet.getRange("B2").getValue();
  var apiSecret = settingsSheet.getRange("B3").getValue();
  var prefix = settingsSheet.getRange("B4").getValue();
  var numRows = sheet.getMaxRows() - 1;
  var numCols = 16;
  var statusColNum = 15;  // IMPT: To keep track status in col 15
  var dataRange = sheet.getRange(startRow, 1, numRows, numCols);

  // Make sure there is API key and secret
  if (apiKey == "" || apiSecret == "") {
    Browser.msgBox("You MUST fill in your API key and secret in Settings sheet first!");
    return;
  }

  // Create one JavaScript object per row of data.
  var objects = getRowsData(sheet, dataRange);

  var totalSent = 0;

  for (var i = 0; i < objects.length; ++i) {
    // Get a row object
    var rowData = objects[i];
    var ss = SpreadsheetApp.getActiveSpreadsheet();
    var templateSheet = ss.getSheetByName("SMS Modelo");
    var template = templateSheet.getRange("A1").getValue();

    // jump loop iteration if conditions not satisied
    if (rowData.resolv == "x" || rowData.contactoUtente == null || rowData.contactoUtente == "" || rowData.reserv == null || rowData.reserv == "" || rowData.cont == "x" || rowData.sms !== null) continue;
      var message = fillInTemplateFromObject(template, rowData);
      var senderName = "Farm Cunha"
      var mobile = rowData.contactoUtente;
      // Send via Nexmo API
      var response = nexmoSendSms(apiKey, apiSecret,"+351" + mobile, message, senderName);
      if (response.getResponseCode() == 200) {
        var object = JSON.parse(response.getContentText());
        if (object.messages[0]['status'] == "0") {
          // Set to QUEUE status - We assumed SENT, as we don't handle delivery status.
          //sheet.getRange(startRow + i, statusColNum).setValue(STATUS_QUEUE);
          sheet.getRange(startRow + i, statusColNum).setValue(STATUS_SENT);
          // Set the reference id
          sheet.getRange(startRow + i, 19).setValue(object.messages[0]['message-id']);
          // sheet.getRange(startRow + i, statusColNum+3).setValue(new Date()); linha pode ser activada para fazer timestamp do envio
          totalSent++;
          }
        else {
          // If status is not 0, then it is an error.
          // Set status to the error text
          sheet.getRange(startRow + i, statusColNum).setValue(object.messages[0]['error-text']);
        }
      }
      else {
        // Non 200 OK response
        sheet.getRange(startRow + i, statusColNum).setValue("Error Response Code: " + response.getResponseCode);
      }
      SpreadsheetApp.flush();
      // Need a wait. Need to throttle else will have "Route Busy" error.
      Utilities.sleep(2000);
  }

  // Update total sent
  var lastTotalSent = parseInt(ScriptProperties.getProperty(PROPERTY_SMS_SENT_FOR_RECONCILE));
  if (isNaN(lastTotalSent)) lastTotalSent = 0;
  ScriptProperties.setProperty(PROPERTY_SMS_SENT_FOR_RECONCILE, (lastTotalSent + totalSent).toString());
  Logger.log("Last sent: " + lastTotalSent + "  now sent: " + totalSent);
  reconcileApp();
}
4

2 回答 2

5

您的循环中有一些太耗时的事情：电子表格读数和 API 调用 + 2 秒睡眠！

我显然会建议你把这些从循环中拿出来（特别是模板表阅读总是一样的！）。一种可能的解决方案是检查行对象的条件并将有效条目保存在数组中......然后在此数组中迭代以调用 API。

如果这仍然太长，则分小批进行，将部分迭代的结束位置保存在 scriptproperties 中，并使用计时器触发器，每 5 分钟继续该过程直到完成（并在最后终止触发器）。

这个论坛上有一些这种“机制”的例子，我最近提出的一个例子是here（它更像是一个草稿，但想法就在那里）

于 2013-03-08T22:10:35.570 回答
4

好的，正如 Serge（谢谢）告诉我的那样，我已经通过将这 3 行从循环中取出来解决了这个问题：

 var ss = SpreadsheetApp.getActiveSpreadsheet();
    var templateSheet = ss.getSheetByName("SMS Modelo");
    var template = templateSheet.getRange("A1").getValue();

这太简单了，我不知道我怎么没看到。

这个简单的更改使脚本更快。例如，遍历 600 行需要 5 分钟以上。现在，超过 5000 行只需要几秒钟。

于 2013-03-12T12:47:54.000 回答