Question

我使用以下自定义模块作为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)

我做错了什么？

Answer 1

问题在于这一行：

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：）

Answer 2

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

如何在Node.JS中为另一个模块形成包装器？

2 个答案: