0

我有以下自定义模块用作 NodeMailer 的包装器

var nodemailer = require("nodemailer");

function Emailer(to, subject, message) {
  this.to = to;
  this.subject = subject;
  this.message = message;
  this.smtpTransport = nodemailer.createTransport("SMTP",{
      service: "Gmail",  // sets automatically host, port and connection security settings
      auth: {
        user: "*******",
        pass: "*******"
      }
    });
  this.send = send;
  function send() {
    this.smtpTransport.sendMail({  //email options
      from: "******* <*****>", // sender address.  Must be the same as authenticated user if using Gmail.
      to: this.to, // receiver
      subject: this.subject, // subject
      text: this.message // body
    }, function(error, response){  //callback
      if(error){
        //console.log(error);
      }else{
        //console.log("Message sent: " + response.message);
      }

      smtpTransport.close(); // shut down the connection pool, no more messages.  Comment this line out to continue sending emails.
    });
  };
}

module.exports = Emailer;

我正在这样实现它：

var emailer = require('./models/emailer.js');
var myEmailer = new emailer('--------', 'my subject', 'my message');
myEmailer.send();

它有效，但我仍然收到此错误：

ReferenceError: smtpTransport is not defined
    at MailComposer.returnCallback (/Users/drewwyatt/Sites/JS/Node/Tutorials/email/models/emailer.js:28:7)

我究竟做错了什么？

4

2 回答 2

2

问题是这一行：

smtpTransport.close();

您可以ReferenceError通过将回调绑定到邮件程序的上下文来避免这种情况。

this.smtpTransport.sendMail({ /* options */ }, function (err, response) {
  // do stuff
  this.smtpTransport.close();
}.bind(this));

send将作为方法放在原型上也更有意义，因为它使用实例的状态。

Emailer.prototype.send = function () {
  // this.smtpTransport ...
};

最后，我会命名它Email而不是Emailer:)

于 2014-03-10T04:06:53.000 回答
1 
...
var self = this;    
function send() {
    this.smtpTransport.sendMail({  //email options
      from: "******* <*****>", // sender address.  Must be the same as authenticated user if using Gmail.
      to: this.to, // receiver
      subject: this.subject, // subject
      text: this.message // body
    }, function(error, response){  //callback
      if(error){
        //console.log(error);
      }else{
        //console.log("Message sent: " + response.message);
      }

      self.smtpTransport.close(); // shut down the connection pool, no more messages.  Comment this line out to continue sending emails.
    });
于 2014-03-10T03:46:12.577 回答