"cannot read property 'indexOf' of undefined"

Question

我正在写一个简单的代码来遍历范围内的所有链接，从这些电子表格（链接）中收集信息并粘贴到循环的当前行。

function UpdateProjects()
{

//----------EDIT THIS FOR A DIFFERENT CLIENT-------------
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName("Projects");
  var etilastrow = sheet.getLastRow();

  var urlRange = sheet.getRange(2, 1, etilastrow);
  var urlsvalues = urlRange.getValues();
  var urls = urlRange.getFormulas();
  Logger.log(urls)

  //looping through all of the rows
  for (var i = 0; i < etilastrow; ++i) {
   currentRow = 2 + i;
  var dirturl = urls[i];
  var remove_before = dirturl[i].indexOf('",');
    var url = dirturl[i].substring(12, remove_before);

Logger.log(currentRow);
    Logger.log(url);

//-----------------Loop code below until last project-------
  var projectPCBdesignsheet = SpreadsheetApp.openByUrl(url).getSheetByName("Main");
    var lstrow = projectPCBdesignsheet.getLastRow(); 

  const vA=projectPCBdesignsheet.getRange(7,2,lstrow).getValues();//get data from project's PCB Design sheet
   
//add last edit values  
  sheet.getRange(currentRow, 4).setValue(vA[10]);
  sheet.getRange(currentRow, 5).setValue(vA[11]);    

//additional data from PCB design sheet
  sheet.getRange(currentRow, 6).setValue(vA[0]);
  sheet.getRange(currentRow, 7).setValue(vA[1]);
  sheet.getRange(currentRow, 8).setValue(vA[2]);
  sheet.getRange(currentRow, 9).setValue(vA[3]);  
  sheet.getRange(currentRow, 11).setValue(vA[5]); 
}
}

如果我将 url 硬编码到变量中，脚本就会工作。现在，从日志中，我可以看到行号和干净的 url.

问题是脚本因以下错误而停止： “类型错误：无法读取未定义的属性 'indexOf' ETI更新项目 @ UpdateProjects.gs:18"

我不明白如果它向我显示 currentRow 和 url 的日志，它如何在第 18 行找到错误，它位于“indexOf”行之后。

Answer 1

问题：

问题 1：

从第二行开始，数据中多了一个空单元格：

var urlRange = sheet.getRange(2, 1, etilastrow);

这会给你这样的东西：[[=link],['']]

这可以通过将 etilastrow 更改为 etilastrow-1 来解决。

在for循环中也是如此。

问题 2：

即使您解决了问题 1，当前代码中也会产生您遇到的错误的主要问题是在这些行中：

var dirturl = urls[i];
var remove_before = dirturl[i].indexOf('",');

请记住，urls 是一个包含 etilastrow 行但只有一个列的数组。如果你解释你的代码，你所说的本质上是 urls[i][i]。这样会在第二次迭代后returnundefined，当i变成1，因为数据只有一列

您的代码之所以有效，是因为您在测试它时只填充了一个单元格，因此 urls[0][0] 可以正常工作，但是如果您添加更多 URL，第二个索引将增加到超过 0 而这例如 urls[0][1] 将丢失并出错，因为 urls 只有一列。

要解决此问题，您可以更改：

var dirturl = urls[i];
var remove_before = dirturl[i].indexOf('",');
var url = dirturl[i].substring(12, remove_before);

至：

var dirturl = urls[i];
var remove_before = dirturl[0].indexOf('",');
var url = dirturl[0].substring(12, remove_before);

改进：

改进 1：

我建议您阅读 for 循环，因为它们在处理索引和数组时可能会造成混淆，您需要足够自在地处理它们。使用 forEach and flat() 直接与每个 url 一起工作：

  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var sheet = ss.getSheetByName("Projects");
  var etilastrow = sheet.getLastRow();

  var urlRange = sheet.getRange(2, 1, etilastrow-1);
  var urlsvalues = urlRange.getValues();
  var urls = urlRange.getFormulas();


  urls.flat().forEach((r,i)=>{
    let currentRow = i + 2;
    let remove_before = r.indexOf('",');
    let url = r.substring(12, remove_before);
    console.log(url)
    console.log(currentRow)
    // put here the rest of your code
  })

改进2：

从 Google Apps 脚本的角度来看，多次调用 setValue 但也在 for 循环中调用会消耗大量性能。根据 Best Practices ，正确的方法是将值存储在数组中，然后在 for 循环之外使用 setValues 而不是。这样，你只用了一次setValues。

我可以尝试优化你的那部分代码，但答案不会有足够的帮助，因为已经有很多关于此优化的帖子，而且你的代码应该与我指出的更正一起工作。

"cannot read property 'indexOf' of undefined"

"cannot read property 'indexOf' of undefined"

javascript

google-apps-script

google-sheets

indexof

问题：

改进：