Promise 和 Callback，哪个更适合与 NodeJS 一起使用？-解网

问：

我发现有 2 种不同的方法可以使用 or 编写节点函数，第一种方法就像下面定义函数一样：promisecallbackfindByEmail

class Users{
  static async findByEmail(email: any ) : Promise<Users | undefined>{
    const user: any = await Pools.execute(
      "SELECT * FROM users WHERE email = ?",
      [email])
      .then(rows => { 
        return rows[0];
       })
      .catch(err => console.log(err) );
      return user;
  };
}

router.post(
  "/api/users/signin",
  async (req: Request, res: Response , next: NextFunction) => {
     const { email, password } = req.body;
     const existingUser = await Users.findByEmail(email);
});

第二种方式是这样的：

declare global {
  namespace Express {
    interface Response {
      user?: Users;
    }
  }
}

  static async findByEmail(req: Request, res: Response) {
    const user = await Pools.execute(
      "SELECT * FROM users WHERE email = ?",
      [req.body.email])
      .then(rows => { 
         res.user = rows[0];
       })
      .catch(err => console.log(err) );
  };




router.post(
  "/api/users/signin",
  async (req: Request, res: Response , next: NextFunction) => {
    await Users.findByEmail(req, res);
    const existingUser = res.user;
});

我不确定这是否是一个“基于意见”的问题？然而，我问这个问题的目的是想知道哪种方式是更好的做法，为什么？根据性能和其他可能的问题？

特别是我想知道最好使用返回值编写函数或使用响应对象将返回值添加到 then（）函数中的函数中，例如 .then（res.user = user）而不是 const user = await pool.execute（SELECT ...） ？

MySQL 节点.js TypeScript 承诺回调

请注意，你的函数是一个带有 promise 的奇怪实现，因为如果你得到一个数据库错误，返回的 promise 仍然会被解析，而不是被拒绝。而且，调用方不进行错误检查。我从不建议以这种方式编写代码。如果存在数据库错误，则该错误或其他错误应作为拒绝传播回调用方。findByEmail()

1赞 jfriend00 1/11/2023

不，它也有问题。您没有将所有错误传播回调用方，没有验证来自请求的传入参数，没有不一致地记录错误，没有在出现错误时返回错误状态，等等......

1赞 jfriend00 1/11/2023

请记住，这吃掉了错误。它将 promise 转换为具有已解决值的已解决 promise。调用方永远不会看到错误。如果要记录和传播错误，则必须在记录后保持承诺被拒绝。在我建议的解决方案中，我不会在较低级别记录错误，我只是让它传播回去，因此不必面对这个问题。.catch(err => console.log(err))undefinedthrow err

答：

1赞 jfriend00 1/11/2023 #1

这是一种刺穿的方法，可以进行以下改进：

变成一个独立于和对象的实用函数，因此可以普遍使用。findByEmail()reqres
正确地将所有错误从调用方传播回调用方。findByEmail()
对传入的电子邮件字段实施一些验证检查，并为此创建单独的错误路径。
在服务器上记录所有错误
检查数据库请求中的所有错误条件
不混合和 ..then()await

代码如下：

// resolves to null if email not found
// rejects if there's a database error
static async findByEmail(email) {
    const rows = await Pools.execute("SELECT * FROM users WHERE email = ?", [email]);
    if (!rows || !rows.length || !rows[0]) {
        return null;
    }
    return rows[0];
};

router.post("/api/users/signin", async (req: Request, res: Response, next: NextFunction) => {
    try {
        // validate incoming parameters
        if (!req.body.email) {
            let errMsg = "No email value present in incoming signin request";
            console.log(errMsg);
            res.status(400).send(errMsg);
            return;
        }
        let user = await Users.findByEmail(req.body.email);
        if (!user) {
            // do whatever you would do if user tries to signin with non-existent email
            // presumably return something like a 404 status
        } else {
            // do whatever you wanted to do here with the user object after login
        }
    } catch(e) {
        // some sort of server error here, probably a database error, not the client's fault
        console.log(e);
        res.sendStatus(500);
    }
});

Promise 和 Callback，哪个更适合与 NodeJS 一起使用？

Promise or Callback, which one is better to use with NodeJS?

评论

评论