JavaScript单次使用变量：性能与可读性的权衡疑问

你提到的两种写法对比 #

写法1（单独声明仅使用一次的变量）

function doSomething ( somethingId = null ) {
    if ( !somethingId ) return false
    const sql = "SELECT * FROM something WHERE somethingId = :somethingId"
    const params = { somethingId }
    return db.query(sql, params)
}

写法2（直接内联传参）

function doSomething ( somethingId = null ) {
    return ( !somethingId )
        ? false
        : db.query("SELECT * FROM something WHERE somethingId = :somethingId", { somethingId })
}

性能结论 #

• 两种写法的可测量性能差异为0，你“内联写法更快”的直觉不符合现代JS引擎的实际表现。
• 现代JS引擎（V8、JavaScriptCore等）会对仅使用1次、没有被闭包捕获的临时变量做内联消除优化：
  • 单独声明的sql字符串常量，会被直接优化到db.query()的参数位置，不会在栈上产生额外的临时存储开销
  • 单独声明的params临时对象，会经过逃逸分析，要么直接内联到参数位置，要么被引擎复用内存空间，不会产生额外的GC压力
• 数据库查询的耗时99%以上来自网络IO、SQL解析、磁盘读取，JS层面的代码执行开销占比不到0.1%，完全没有必要为了这点微优化调整写法。
• 仅当代码属于执行频率极高的热路径（比如每秒执行上万次的纯计算逻辑、无IO的工具函数）时，才需要考虑这类写法的微性能差异，业务代码的数据库查询场景完全不适用。

可读性取舍标准 #

没有绝对的优劣，只看实际场景：

优先选单独声明变量的场景 #

• SQL语句超过1行、或者需要动态拼接过滤条件：单独声明能把查询逻辑和参数传递逻辑解耦，方便快速定位SQL内容、后续调整查询结构
• 参数对象后续大概率会扩展（比如加用户权限校验字段、多条件过滤参数）：单独声明后修改逻辑不需要改动函数调用的参数位置
• 团队代码规范明确要求抽离SQL、参数定义：统一风格比个人偏好重要

优先选内联传参的场景 #

• 逻辑非常简单（单个字段等值查询、SQL和参数都只有1行）：内联后代码行数更少，逻辑一目了然
• 工具类简单查询函数：减少不必要的变量声明冗余，避免代码嵌套层级过深

针对你给出的MySQL查询示例，两种写法都没有问题：如果后续这个查询可能会加更多过滤条件，建议保留单独声明的写法；如果就是固定的简单等值查询，内联写法完全没问题，不会有任何性能损失。

内容的提问来源于stack exchange，提问作者Simon Trichereau