开发者问题收集

类型错误:无法从 null 读取属性长度

2019-04-23
747

我们有一张工作表,用于获取工作质量表单的数据。一年多来,它一直运行顺利,工作表、表单或脚本从未发生过任何变化。

但出于某种原因,现在每当我们运行 SendEmail 函数时,我都会收到错误:

TypeError:无法从 null 读取属性“length”。(第 83 行,文件“SendEmail”)

这就是现在发生错误的地方:

for (var i = 0; i < templateVars.length; ++i) {

每当运行脚本时,第一列(项目)应该自动输入连续的数字。但现在这一切都保持空白,电子邮件模板永远不会被通过电子邮件发送。

function SendEmail() {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var dataSheet = ss.getSheets()[0];
  var dataRange = dataSheet.getRange(2, 1, dataSheet.getMaxRows() - 1, 19);
  var lastRow = dataSheet.getMaxRows();
  for (var x = 0; x < dataSheet.getMaxRows() - 2; ++x) {
      if(dataSheet.getRange(x+2, 3).getValue()!=""){
        if(dataSheet.getRange(x+2, 15).getValue()==""){
             var EMInum = dataSheet.getRange((x + 1), 1).getValue() + 1;
             dataSheet.getRange(x+2, 1).setValue(EMInum);
             dataSheet.getRange(x+2, 2).setFormula("=\"QoW-\"&text(A" + (x+2) + ",\"000\")");
             dataSheet.getRange(x+2, 15).setFormula("=vlookup(I" + (x+2) + ",ContactDetails,2,0)");
             dataSheet.getRange(x+2, 16).setFormula("=vlookup(I" + (x+2) + ",ContactDetails,3,0)");
             dataSheet.getRange(x+2, 17).setFormula("=match(J" + (x+2) + ",CompanyName,1)");
             dataSheet.getRange(x+2, 18).setFormula("=vlookup(I" + (x+2) + ",ContactDetails,Q" + (x+2) + ",0)");
             dataSheet.getRange(x+2, 19).setFormula("=vlookup(I" + (x+2) + ",ContactDetails,Q" + (x+2) + "+1,0)");
             dataSheet.getRange(x+2, 27).setFormula("=I" + (x+2)+ "&if(Z" + (x+2)+ "=\"\" ,\" OPEN\",\" CLOSED\")");
             dataSheet.getRange(x+2, 28).setFormula("=if(right(AA" + (x+2) + ",4)=\"OPEN\",now()-datevalue(I" + (x+2)+ "),0)");
             dataSheet.getRange(x+2, 29).setFormula("=if(AB" + (x+2) + "=0,\"e) Closed\",if(AB" + (x+2) + "<90,\"a) 0 - 90 days\",if(AB" + (x+2) + "<180,\"b) 90 - 180 days\",if(AB" + (x+2) + "<360,\"c) 180 - 360 days\",\"d) Over 360 days\"))))");
             //Browser.msgBox("");
          }
        }
      }  

  //


  //


  var templateSheet = ss.getSheets()[1];
  var emailTemplate = templateSheet.getRange("A1").getValue();
  var defaultCCAddress = templateSheet.getRange("B17").getValue(); // Added by zumzum as per projectid:a0CD000000xFxWQ

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

  // For every row object, create a personalized email from a template and send
  // it to the appropriate person.
  for (var i = 0; i < objects.length; ++i) {
    // Get a row object
    var rowData = objects[i];

    // Generate a personalized email.
    // Given a template string, replace markers (for instance ${"First Name"}) with
    // the corresponding value in a row object (for instance rowData.firstName).
    var emailText = fillInTemplateFromObject(emailTemplate, rowData);
    var emailSubject = "NO REPLY: EM&I Quality of Work Issue Submission " + dataSheet.getRange(i + 2, 2).getValue();
    var emailCheck = dataSheet.getRange(i + 2, 13).getValue();
    //Browser.msgBox(emailCheck);
    var emailAddress = dataSheet.getRange(i + 2, 18).getValue();
    var ccemailAddress = dataSheet.getRange(i + 2, 15).getValue();
    if(emailCheck=="")
    {
      /* original code commented by zumzum as per projectid:a0CD000000xFxWQ
      MailApp.sendEmail(emailAddress, emailSubject, emailText,{cc:ccemailAddress+",[email protected],[email protected]",bcc:"[email protected]"});
      dataSheet.getRange(i + 2, 13).setValue("sent - " + Date());
      */
      // Start od new code added by zumzum as per projectid:a0CD000000xFxWQ
      MailApp.sendEmail(emailAddress, emailSubject, emailText,{cc:ccemailAddress+","+defaultCCAddress}); 
      dataSheet.getRange(i + 2, 13).setValue("sent - " + Date()); 
      // End of new code added by zumzum as per projectid:a0CD000000xFxWQ
    }   
  }
}


// Replaces markers in a template string with values define in a JavaScript data object.
// Arguments:
//   - template: string containing markers, for instance ${"Column name"}
//   - data: JavaScript object with values to that will replace markers. For instance
//           data.columnName will replace marker ${"Column name"}
// Returns a string without markers. If no data is found to replace a marker, it is
// simply removed.
function fillInTemplateFromObject(template, data) {
  var email = template;
  // Search for all the variables to be replaced, for instance ${"Column name"}
  var templateVars = template.match(/\$\{\"[^\"]+\"\}/g);

  // Replace variables from the template with the actual values from the data object.
  // If no value is available, replace with the empty string.
  for (var i = 1; i < templateVars.length; i++) {
    // normalizeHeader ignores ${"} so we can call it directly here.
    var variableData = data[normalizeHeader(templateVars[i])];
    email = email.replace(templateVars[i], variableData || "");
  }

  return email;
}





//////////////////////////////////////////////////////////////////////////////////////////
//
// The code below is reused from the 'Reading Spreadsheet data using JavaScript Objects'
// tutorial.
//
//////////////////////////////////////////////////////////////////////////////////////////

// getRowsData iterates row by row in the input range and returns an array of objects.
// Each object contains all the data for a given row, indexed by its normalized column name.
// Arguments:
//   - sheet: the sheet object that contains the data to be processed
//   - range: the exact range of cells where the data is stored
//   - columnHeadersRowIndex: specifies the row number where the column names are stored.
//       This argument is optional and it defaults to the row immediately above range;
// Returns an Array of objects.
function getRowsData(sheet, range, columnHeadersRowIndex) {
  columnHeadersRowIndex = columnHeadersRowIndex || range.getRowIndex() - 1;
  var numColumns = range.getEndColumn() - range.getColumn() + 1;
  var headersRange = sheet.getRange(columnHeadersRowIndex, range.getColumn(), 1, numColumns);
  var headers = headersRange.getValues()[0];
  return getObjects(range.getValues(), normalizeHeaders(headers));
}

// For every row of data in data, generates an object that contains the data. Names of
// object fields are defined in keys.
// Arguments:
//   - data: JavaScript 2d array
//   - keys: Array of Strings that define the property names for the objects to create
function getObjects(data, keys) {
  var objects = [];
  for (var i = 0; i < data.length; ++i) {
    var object = {};
    var hasData = false;
    for (var j = 0; j < data[i].length; ++j) {
      var cellData = data[i][j];
      if (isCellEmpty(cellData)) {
        continue;
      }
      object[keys[j]] = cellData;
      hasData = true;
    }
    if (hasData) {
      objects.push(object);
    }
  }
  return objects;
}

// Returns an Array of normalized Strings.
// Arguments:
//   - headers: Array of Strings to normalize
function normalizeHeaders(headers) {
  var keys = [];
  for (var i = 0; i < headers.length; ++i) {
    var key = normalizeHeader(headers[i]);
    if (key.length > 0) {
      keys.push(key);
    }
  }
  return keys;
}

// Normalizes a string, by removing all alphanumeric characters and using mixed case
// to separate words. The output will always start with a lower case letter.
// This function is designed to produce JavaScript object property names.
// Arguments:
//   - header: string to normalize
// Examples:
//   "First Name" -> "firstName"
//   "Market Cap (millions) -> "marketCapMillions
//   "1 number at the beginning is ignored" -> "numberAtTheBeginningIsIgnored"
function normalizeHeader(header) {
  var key = "";
  var upperCase = false;
  for (var i = 0; i < header.length; ++i) {
    var letter = header[i];
    if (letter == " " && key.length > 0) {
      upperCase = true;
      continue;
    }
    if (!isAlnum(letter)) {
      continue;
    }
    if (key.length == 0 && isDigit(letter)) {
      continue; // first character must be a letter
    }
    if (upperCase) {
      upperCase = false;
      key += letter.toUpperCase();
    } else {
      key += letter.toLowerCase();
    }
  }
  return key;
}

// Returns true if the cell where cellData was read from is empty.
// Arguments:
//   - cellData: string
function isCellEmpty(cellData) {
  return typeof(cellData) == "string" && cellData == "";
}

// Returns true if the character char is alphabetical, false otherwise.
function isAlnum(char) {
  return char >= 'A' && char <= 'Z' ||
    char >= 'a' && char <= 'z' ||
    isDigit(char);
}

// Returns true if the character char is a digit, false otherwise.
function isDigit(char) {
  return char >= '0' && char <= '9';
}```
2个回答

您收到的错误可以通过使用任何不包含 ${"foo"> 格式的占位符的 emailTemplate 字符串运行 fillInTemplateFromObject(emailTemplate, rowData) 来重现(正则表达式 /\$\{\"[^\"]+\"\}/g 要求使用文字双引号)。

SendEmail 中, emailTemplate 的值来自第一张工作表单元格 A1。检查该单元格中的值。它是否包含正确的 ${"foo"> 格式的模板?

由于此错误可能是由用户操作电子表格引起的,请考虑改进 fillInTemplateFromObject ,方法是在尝试使用预期之前检查是否成功匹配 match 数组。

function fillInTemplateFromObject(template, data) {
  var email = template;
  // Search for all the variables to be replaced, for instance ${"Column name"}
  var templateVars = template.match(/\$\{\"[^\"]+\"\}/g);
  if (! templateVars) {
    throw new Error("No templates found in your template sheet.  Aborting.")
  }
  // Replace variables from the template with the actual values from the data object.
  // If no value is available, replace with the empty string.
  for (var i = 0; i < templateVars.length; i++) {
    // normalizeHeader ignores ${"} so we can call it directly here.
    var variableData = data[normalizeHeader(templateVars[i])];
    email = email.replace(templateVars[i], variableData || "");
  }

  return email;
}

您可以在 SendEmail 中捕获该错误并提供更多有用信息:

// inside SendEmail
try {
  var emailText = fillInTemplateFromObject(emailTemplate, rowData);
} catch (error) {
  throw new Error("In spreadsheet " + ss.getUrl() + " sheet " + templateSheet.getSheetName() + " row " + (i+1) + " there were no templates found.")
}

如果您仍然需要帮助调试错误,请 使用应用脚本调试器 或使用 应用脚本内置的各种日志记录选项 在运行时输出数据状态。

dwmorrin
2019-04-26

为什么您使用 ++i 而不是 i++?我猜这是问题所在,如果 templateVars 是一个数组,您将到达数组的末尾并尝试在不存在的行上运行一次。也许之前,这是一个空行,但现在最后一行是工作表的末尾。

由于对您的代码一无所知,我会将其更改为:

for (var i = 0; i < templateVars.length; ++i) {

for (var i = 1; i < templateVars.length; i++) {
J. G.
2019-04-23