Express.js变量sumOfRevenue赋值后未更新，打印为空求助

核心原因：异步代码执行顺序错误 #

你遇到的问题是异步回调的执行时机导致的：

• 你在client.query前加了await，但如果你的数据库客户端的query方法是基于回调的（不返回Promise），await无法等待回调函数执行完成。
• 回调函数是异步触发的，会在当前事件循环结束后才执行，所以console.log会先于sumOfRevenue = 10运行，此时变量还是初始的空字符串。

解决方法 #

方法1：改用Promise风格的async/await（推荐）

如果你的数据库客户端支持Promise（比如大部分现代数据库库都支持），直接用await等待查询完成，不需要回调：

const express = require('express')
var router = express.Router()
const client = require('../models/db')

// 用let声明变量，避免隐式全局
let sumOfRevenue = ""  

router.get('/getSites', async(req, res) => {
  try {
    // await会等待查询完成，拿到结果后再执行后续代码
    const results = await client.query('select sum(revenue) from deals ');
    sumOfRevenue = 10; // 实际场景可以替换为results中的计算值
    console.log("sumOfRevenue is:" + sumOfRevenue);
    // 这里可以继续处理响应，比如res.send(...)
  } catch (error) {
    console.error('查询出错:', error);
    // 给客户端返回错误响应
    res.status(500).send('服务器内部错误');
  }
});

module.exports = router

方法2：保留回调，将打印逻辑移到回调内部

如果必须使用回调，把console.log放到回调函数里，确保赋值完成后再执行打印：

const express = require('express')
var router = express.Router()
const client = require('../models/db')

let sumOfRevenue = ""  

router.get('/getSites', (req, res) => {
  client.query('select sum(revenue) from deals ', (error, results) => {
    if (!error) {
      sumOfRevenue =10;
      // 赋值完成后再打印
      console.log("sumOfRevenue is:" + sumOfRevenue);
      // 处理响应
      res.send({ sum: sumOfRevenue });
    } else {
      console.error('查询出错:', error);
      res.status(500).send('服务器内部错误');
    }
  });
});

module.exports = router

额外注意事项 #

• 避免使用隐式全局变量（你之前的sumOfRevenue = ""没有声明，会变成全局变量），用let或const声明，防止多请求并发时变量被覆盖。
• 一定要处理数据库查询的错误，避免未捕获异常导致服务器崩溃。

内容的提问来源于stack exchange，提问作者Shital Sonawane