“无法读取未定义的属性‘indexOf’”

如何解决“无法读取未定义的属性‘indexOf’”

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

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 的日志（位于“indexOf”行之后），我不明白它如何在第 18 行找到错误。

解决方法

问题：

问题 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]。这将在第二次迭代后返回 undefined，当 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 和 flat() 直接处理每个网址：

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

  var urlRange = sheet.getRange(2,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。

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