Ошибка типа: невозможно прочитать длину свойства из нуля

Question

У нас был лист с данными о качестве рабочей формы. Это работает безупречно уже более года, с тех пор ничего не изменилось с листами, формой или сценарием.

Но по какой-то причине я теперь получаю сообщение об ошибке всякий раз, когда мы запускаем функцию SendEmail:

TypeError: Cannot read property "length" from null. (line 83, file "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+",david.mortlock@emialliance.com,peter.gresty@emialliance.com",bcc:"richard.priddes@emialliance.com"});
      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';
}```

Answer 1

Полученную ошибку можно воспроизвести, запустив fillInTemplateFromObject(emailTemplate, rowData) с любой строкой для emailTemplate, которая не содержит заполнитель в форме ${"foo"} (буквальные двойные кавычки необходимы для регулярного выражения /\$\{\"[^\"]+\"\}/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.")
}

Если вам по-прежнему нужна помощь в отладке ошибки, используйте отладчик сценариев приложений или используйте различные параметры ведения журнала, встроенные в сценарий приложений , для вывода состояния ваших данных во время выполнения.

Answer 2

Почему вы используете ++ i, а не i ++? Я предполагаю, что это проблема, если templateVars является массивом, вы попадаете в конец массива и пытаетесь запустить его один раз в несуществующей строке. Возможно, раньше это была пустая строка, но теперь последняя строка - это конец листа.

Не зная ничего о вашем коде, я бы изменил:

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

до

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