Google电子表格 - 如何避免发送电子邮件重复项?

时间:2014-08-12 02:06:23

标签: email google-apps-script duplicates google-sheets google-drive-api

我遇到了脚本问题。我使用Google Developers网站的以下脚本来编写简单的合并邮件。见https://developers.google.com/apps-script/articles/mail_merge

我修改了一下脚本,以防止电子邮件重复。但是,即使脚本似乎有效,因为它标记了EMAIL_SENT'每次发送电子邮件时在每一行中。如果邮件已经被标记并且仍然发送邮件,则不会注意。

我相信第16行和第34行有错误; var emailSent = rowData [6];"

如果有人能帮助我,我真的很感激。无论你提前感谢谁。

以下是修改过的脚本:

var EMAIL_SENT = "EMAIL_SENT";

function sendEmails() {
  var ss = SpreadsheetApp.getActiveSpreadsheet();
  var dataSheet = ss.getSheets()[0];
  var dataRange = dataSheet.getRange(2, 1, dataSheet.getMaxRows() - 1, 7);
  var templateSheet = ss.getSheets()[1];
  var emailTemplate = templateSheet.getRange("A2").getValue();
  var objects = getRowsData(dataSheet, dataRange);
  for (var i = 0; i < objects.length; ++i) {
    var Resume = DriveApp.getFilesByName('Resume.pdf') var Portfolio = DriveApp.getFilesByName('Portfolio.pdf') var rowData = objects[i];
    var emailText = fillInTemplateFromObject(emailTemplate, rowData);
    var emailSubject = "Architectural Internship";
    var emailSent = rowData[6];
    if (emailSent != EMAIL_SENT) {
      MailApp.sendEmail(rowData.emailAddress, emailSubject, emailText, {
        attachments: [Resume.next(), Portfolio.next()]
      });
      dataSheet.getRange(2 + i, 7).setValue(EMAIL_SENT);
      SpreadsheetApp.flush();
    }
  }
}

function fillInTemplateFromObject(template, data) {
  var email = template;
  var templateVars = template.match(/\${\"[^\"]+\"}/g);
  for (var i = 0; i < templateVars.length; ++i) {
    var variableData = data[normalizeHeader(templateVars[i])];
    email = email.replace(templateVars[i], variableData || "");
  }
  return email;
}

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));
}

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;
}

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;
}

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;
    }
    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'; }

3 个答案:

答案 0 :(得分:0)

我建议在执行前记录这两个值,以确保它们是相同的。我还猜测email_sent和EMAIL_SENT是不同的数据类型。也可以尝试将值强制为字符串以进行比较。

澄清:

logger.Log(emailSent);
logger.Log(EMAIL_SENT);
 if (emailSent.toString() != EMAIL_SENT.toString())
{...

答案 1 :(得分:0)

你的代码真的很难阅读,返回2个或更多对象的函数会让它变得更难......你使用的变量名也有点令人困惑....但这可能是个人的pov: - )

无论如何,我想我已经找到了问题:当你写var rowData = objects[i];

这个“对象”实际上是getRowData函数的结果,但是如果你看看这个函数,你会看到它返回2个对象,第一个对象本身就是另一个函数的结果(getObjects)... < / p>

您正在检查值是数组的第6个元素,它实际上是一个对象并将其与字符串进行比较。平等永远不会成真。

我没有进一步分析,因为我发现它真的很混乱(正如我已经说过的),但至少你有第一个要检查的元素。

我建议您以更简单的方式重写此代码,并使用更合适的变量名来帮助您进行调试。

答案 2 :(得分:0)

错误出现在这行代码中 -

var dataRange = sheet.getRange(startRow,1,numRows,2)

它只考虑范围内的2列。改变2到3,它工作正常。