脚本停止工作。我关闭了sumbissions并稍后重新启用。现在脚本又可以工作了，对谷歌来说是不是太密集了？

Question

我为谷歌创建了一个在提交表单上运行的脚本。 它需要一个在表单中输入的数字，然后在工作表中查找它。 如果找到匹配项，则会在提交表单中输入时间戳。

在这里

// this is the name of our script, myFunction can be anything.
// e is a special variable, its the form & answers that has 
// just been submitted by a user.

function myFunction(e) { 


    var sessionDate = e.values[0]; // This gets the timestamp of the submission
    var sessionID = e.values[1]; // This gets the session ID the user entered


    var ss = SpreadsheetApp.getActiveSpreadsheet(); // Gets the spreadsheet where the form is attached to and all sheets are stored
    var sheet = ss.getSheetByName("Data Dump"); // Specifies the data dump sheet, which has the IDs and the cell we want to add the timestamp to
    var range = sheet.getRange("B:B"); // Specifies the column that holds the session IDs, in the data dump sheet
    var values = range.getValues(); // Reads the data in the sessions ID column, that we specified earlier

    for(i=0;i<values.length;i++){ // This goes through the session ids one by one, and we can test each one


        if (values[i][0] == sessionID){ // This is the test. If the ID we are currently looking at matches the ID on the form response, then lets do something

            sheet.getRange(i+1, 7).setValue(sessionDate); // This is us doing that something. Using where we found the matching ID, lets write the timestamp in a cell.
            break; // This stops us going through anymore ID's as we have already found the one we are looking for

        }
    }

}

经过反复测试，效果很好。然后它被设置为上线。 与很多人共享表单。

有很多提交，然后脚本停止工作。即使在手动测试时，我也没有收到任何具体错误。只是电子表格服务错误。

所以我在一两天前就停止了对表单的回复。今天是我必须查看的第一个更改，并且一切正常。脚本在几秒钟内运行。

我唯一能想到的是脚本对于谷歌来说太密集了，需要多次提交，我特别想for 循环可能会给它带来问题。

有没有人有任何想法或任何我可以尝试的？

提前致谢

Answer 1

对于表单提交，您应该始终使用LockService。因为互联网可能连接不好，或者谷歌的服务器可能很慢等等。你应该在获取锁时尝试/捕获：

try {
  var lock = LockService.getScriptLock();
  var isLockt = lock.tryLock(10000);
catch (err) {
  Utilities.sleep(2000);//Wait 2 seconds
  lock = LockService.getScriptLock();//Try again
  isLockt = lock.tryLock(10000);
};

if (!isLockt) { // Not true
  //Email developer.  There was an error
  //Email user.  There was an error.
  //Keep running code?  Or quit?
  return;
};

您无需遍历您的价值观。有一个更快的方法。你只得到一列数据，所以你不需要二维数组。获取二维数组并将其展平为一维。

var values = range.getValues();
values = values.toString().split(",");//Convert 2D to 1D

var indexOfSessionID = values.indexOf(sessionID);//Gets index of first
     //occurrence of the sessionID

这为您提供了数据中第一次出现 sessionID 的索引。数据是零索引的，因此索引和行号不同步。索引零是数组中的第一个元素。如果范围从第一行开始，则要获得正确的行号，请将 1 添加到索引。如果范围从第二行开始，则添加 2。

如果有多个会话 ID，indexOf() 将只获得第一个。但是你的 for 循环在获取会话 ID 后会中断，所以我假设没关系。

另外，我会改变这一行：

var range = sheet.getRange("B:B");

到：

var range = sheet.getRange(1,2,sheet.getLastRow());

从第 1 行第 2 列开始，并获取到最后一行的行数。

Answer 2

在循环中输入range.offset(i, 7).setValue(sessionDate);，而不是sheet.getRange(i+1, 7).setValue(sessionDate);

请看这里：https://developers.google.com/apps-script/reference/spreadsheet/range#offsetrowoffset-columnoffset

但是，如果提交了多个表单，您可能会同时调用脚本，因此锁定脚本或节省时间或对表单进行排队可能会有所帮助。

使用锁（参见：https://developers.google.com/apps-script/reference/lock/lock#tryLock(Integer)）

节省时间/队列表单的更复杂的解决方案：
尝试完全不同的策略。
例如：不是在表单提交触发器时调用myFunction(e)，而是存储所有到达PropertiesService.getScriptProperties().setProperty的e.values，同时创建一个定时触发器以在一分钟内运行myFunction，从@987654332检索所有存储的e.values @ 然后将它们全部处理成一个块。同时，任何新的e.Values 都会存储在PropertiesService 中（参见https://developers.google.com/apps-script/reference/script/clock-trigger-builder#after(Integer)）

我意识到这些不是简单的解决方案。如果您对它们感兴趣，我可以尝试帮助编写脚本。我不是专家，但我会尝试使用这些选项。

Answer 3

所以经过大量思考和@David Tew 的一些改变游戏规则的建议后，我认为这就是解决方案。

 // this is the name of our script, myFunction can be anything. e is a special variable, its the form & answers that has just been submitted by a user.
 function myFunction(e) {


     var lock = LockService.getScriptLock();

     lock.waitLock(10000);




     var sessionDate = e.values[0]; // This gets the timestamp of the submission
     var sessionID = e.values[1]; // This gets the session ID the user entered


     var ss = SpreadsheetApp.getActiveSpreadsheet(); // Gets the spreadsheet where the form is attached to and all sheets are stored
     var sheet = ss.getSheetByName("Data Dump"); // Specifies the data dump sheet, which has the IDs and the cell we want to add the timestamp to
     var range = sheet.getRange(1, 2, sheet.getLastRow()); // Specifies the column that holds the session IDs, in the data dump sheet
     var values = range.getValues(); // Reads the data in the sessions ID column, that we specified earlier

     // Edits below
     values = values.toString().split(","); //Convert 2D to 1D

     var indexOfSessionID = values.indexOf(sessionID); //Gets index of first
     //occurrence of the sessionID

     if (indexOfSessionID > 0) {
         sheet.getRange(indexOfSessionID + 1, 7).setValue(sessionDate);
     }


     SpreadsheetApp.flush();
     lock.releaseLock();
 }

我现在正在与多个用户一起测试它。 桑迪 - 感谢平面数组的想法，但是当我对大卫发表评论时，它已经让我印象深刻了，但是复制和粘贴代码。嗯。谢谢。

虽然我已经逐行发布了我自己问题的答案（我认为）。 我认为 David 向我指出的信息必须是最佳答案，因为它迫使我重新审视我制作的旧脚本并改变我将来使用事件驱动触发器的方式。也可以表明您正在努力解决自己的问题并避免投反对票。

我恳请任何编写具有事件驱动触发器（例如 onEdit、onFormsubmit）的脚本的人，并且有多个用户触发该触发器，甚至阅读 David 共享的链接。

GAS Class Lock

在下面编辑

4 个不同的谷歌用户提交了 5 个回复，每个回复的时间为 2 分 10 秒，结果都奏效了。明天要做一个更大的测试。