Node.js中If语句执行顺序异常问题求助

你遇到的核心问题是JavaScript异步回调的执行特性：getRoleName是一个依赖数据库查询的异步操作（用回调函数处理结果），当你调用它之后，JavaScript引擎不会等待回调执行完成，而是直接继续运行后面的同步代码——也就是判断changeUserToSellerFlag的逻辑，这时候flag还没被回调里的代码修改，所以自然进不了if(changeUserToSellerFlag == 1)分支。

修复方案：用Promise + Async/Await改造异步流程 #

要让代码按照你预期的顺序执行，最优方式是把回调风格的异步函数改成Promise风格，再用async/await控制执行顺序，让异步代码像同步代码一样按顺序运行。

步骤1：改造getRoleName为Promise风格（在user.service.js中） #

把原来的回调式函数改成返回Promise，这样我们就能用await等待它的结果：

// user.service.js
const getRoleName = (userId) => {
  return new Promise((resolve, reject) => {
    // 这里放你原来的数据库查询逻辑，示例如下：
    db.query('SELECT Name, RoleId FROM roles WHERE UserId = ?', [userId], (err, results) => {
      if (err) {
        reject(err); // 出错时传递错误信息
      } else {
        resolve(results[0]); // 成功时返回单行查询结果
      }
    });
  });
};

步骤2：改造controller中的sellerVerify函数 #

把函数标记为async，用await等待所有异步操作完成后再执行后续逻辑，同时用try/catch统一处理错误：

const { updateUserSellerStatus } = require("./seller.model.js");
const { genSaltSync, hashSync } = require("bcrypt");
const userid = require("../auth/token_validation.js");
const { updateUserRoleAfterSellerApproval, getRoleName } = require("../users/user.service.js");

module.exports = {
  sellerVerify: async (req, res) => {
    let sellerStatusName;
    let changeUserToSellerFlag = 0;
    const approvedByName = req.fullName;
    const approvedByEmail = req.email;

    console.log("Request body is below");
    console.log(req.body);

    try {
      // 封装updateUserSellerStatus为Promise并等待结果
      const updateResults = await new Promise((resolve, reject) => {
        updateUserSellerStatus(
          req.body,
          approvedByName,
          approvedByEmail,
          (err, results) => {
            if (err) reject(err);
            else resolve(results);
          }
        );
      });

      if (!updateResults) {
        return res.json({ success: 0, message: "No results from seller status update" });
      }

      // 处理卖家状态名称（用switch更简洁）
      switch (req.body.sellerStatus) {
        case 3:
          sellerStatusName = "Rejected";
          break;
        case 2:
          sellerStatusName = "Completed";
          break;
        case 1:
          sellerStatusName = "Pending";
          break;
        default:
          sellerStatusName = "Unknown";
      }

      // 等待角色查询结果，确保flag被正确赋值
      if (req.userId) {
        const roleResults = await getRoleName(req.userId);
        console.log("check rolename getting below");
        console.log(roleResults);
        
        if (roleResults.Name === "Customer") {
          console.log("User is Customer");
          changeUserToSellerFlag = 1;
          console.log("flag is :" + changeUserToSellerFlag);
        } else if (roleResults.Name === "Seller") {
          console.log("User is Seller");
          changeUserToSellerFlag = 0;
        }
      }

      // 现在flag已正确赋值，判断逻辑按预期执行
      console.log("see flag changed here ", changeUserToSellerFlag);
      if (changeUserToSellerFlag === 1) {
        console.log("i am here");
        // 等待角色更新操作完成
        const roleUpdateResults = await new Promise((resolve, reject) => {
          updateUserRoleAfterSellerApproval(
            req.userId,
            req.roleId,
            (err, results) => {
              if (err) reject(err);
              else resolve(results);
            }
          );
        });

        console.log("result details are below");
        console.log(roleUpdateResults);
        if (!roleUpdateResults) {
          return res.json({ success: 0, message: "No results from role update" });
        }
        console.log("Update into UserRole Completed successfully");
      } else {
        console.log("User is already a seller and dont need to update userrole table");
      }

      console.log(sellerStatusName);
      return res.status(200).json({ success: 1, status: sellerStatusName });

    } catch (err) {
      console.log(err);
      return res.status(500).json({ success: 0, message: err.message });
    }
  },
};

为什么这样能解决问题？ #

• async/await让异步代码的执行顺序变得直观：await getRoleName(req.userId)会暂停函数执行，直到数据库查询完成并返回结果，之后才会继续执行下面的flag判断逻辑。
• 用try/catch统一捕获异步操作中的错误，替代原来分散在各个回调里的错误处理，代码结构更清晰。
• 改造后，控制台的输出顺序会变成：先获取角色结果、修改flag，再判断flag并执行对应的分支，完全符合你的预期。

内容的提问来源于stack exchange，提问作者Sagufa Khanam