Node.js用户信息更新接口如何避免重复代码？

优化思路 #

核心是消除重复的数据库更新与响应逻辑，同时用async/await简化Promise嵌套，让代码结构更简洁、易维护。

优化后的代码 #

router.put("/editprofile", upload.single('image'), fileSizeLimitErrorHandler, verifyUser, async (req, res) => {
    try {
        let userInfo = req.body;
        const url = req.protocol + '://' + req.get('host');
        
        // 处理头像上传路径
        if (req.file) {
            userInfo.image = url + '/uploads/profile_images/' + req.file.filename;
        }

        // 处理密码哈希（仅当用户提交了新密码时）
        if (userInfo.password) {
            // bcrypt.hash可自动生成盐，无需单独调用genSalt
            userInfo.password = await bcrypt.hash(userInfo.password, 10);
        }

        // 统一执行用户更新与响应
        const updatedUser = await User.findByIdAndUpdate(id, userInfo, { new: true });
        res.status(200).json(updatedUser);
    } catch (err) {
        res.status(400).send("Something went wrong");
    }
})

关键优化点 #

• 消除重复逻辑：把原来两个分支里重复的findByIdAndUpdate和响应代码合并，通过async/await将流程线性化，统一执行。
• 简化密码哈希：bcrypt.hash支持直接传入盐轮数（如10），自动生成对应盐值，省去单独调用genSalt的步骤，减少代码嵌套。
• 集中错误处理：用try/catch捕获所有异步操作的异常，替代原来两个分支里重复的catch块，统一处理错误响应。
• 提升可读性：async/await替代嵌套Promise，避免"回调地狱"，代码结构更清晰，逻辑一目了然。

内容的提问来源于stack exchange，提问作者M.KH