Question

我目前正在参与一个涉及使用Geddy js框架的项目，这是我第一次使用它。我目前正在尝试在用户的模型中修复create方法。以下是代码：

this.create = function (req, resp, params) {
    var self = this
      , user = geddy.model.User.create(params);

    //need to ensure that the user agrees with the terms and conditions.

    // Non-blocking uniqueness checks are hard
    geddy.model.User.first({username: user.username}, function(err, data) {
      if (data) {
        params.errors = {
          username: 'This username is already in use.'
        };
        //self.transfer('add');
      }
      else {
        if (user.isValid()) {
          user.password = cryptPass(user.password);
          user.suburb = "";
          user.state = "";
          user.postcode = "";
        }
        user.save(function(err, data) {
          if (err) {
            params.errors = err;
            self.transfer('add');
          }
          else {
              // setup e-mail data with unicode symbols
              var mailOptions = {
                  from: "App ✔ <hello@app.com>", // sender address
                  to: user.email, // list of receivers
                  subject: user.username + " Thank you for Signing Up ✔", // Subject line
                  text: "Please log in and start shopping! ✔", // plaintext body
                  html: "<b>Please log in and start shopping!✔</b>" // html body
              }

              smtpTransport.sendMail(mailOptions, function(error, response){
                  if(error){
                      console.log(error);
                  }else{
                      console.log("Message sent: " + response.message);
                  }

                  // if you don't want to use this transport object anymore, uncomment following line
                  smtpTransport.close(); // shut down the connection pool, no more messages
              });
            self.redirect({controller: self.name});
          }
        });
      }
    });
};

如果查看代码，显然会检查所谓的用户是否有效：if (user.isValid()) { user.password = cryptPass(user.password); user.suburb = ""; user.state = ""; user.postcode = ""; }

无论用户是否有效，都会继续“保存”。我在想为什么这样的代码？这听起来毫无意义。我询问了关于它的项目的原始开发人员，他说模型显然是在创建项目时生成的。

所以有点困惑的状态，如果有人能告诉我为什么save方法首先在if语句之外？这是Geddy最初创作者的意图吗？或者真的是荒谬的，我应该改变它？

感谢。

Answer 1

如果数据无效，Geddy的save()调用将会出错（除非设置了强制标志，但不是这样）。它实际上使用相同的isValid()调用。所以，看起来你在这里只是有人为所有错误情况设置一个错误处理程序。

仅当数据看起来有效时才设置user.password加密数据，我猜这只是让'必须设置'类型的验证工作。有可能即使使用空密码，加密的字符串也将被计为设置。

Geddy收藏用户

1 个答案: