如何将输入变量验证代码移至独立函数以简化doSomething函数？

原问题代码 #

function doSomething (inputVar, callback) {
    // 输入变量验证
    if (inputVar === 'malformed input') {
        callback(new Error('malformed input'))
        return // 阻止函数剩余代码执行
    }
    console.log("You shouldn't see this message when something goes wrong with the validation.")
    // 执行其他操作
    callback(null)
}

提问：将输入变量验证代码移至独立函数，使doSomething()函数结构更简洁的最佳实现方式是什么？

这是个很棒的重构方向！把输入验证逻辑抽离成独立函数，既能遵循单一职责原则，又能让主函数的核心逻辑更清晰。下面是我推荐的最佳实现：

重构后的代码 #

// 独立的输入验证函数，专注于输入合法性检查
function validateInputVar(inputVar) {
    if (inputVar === 'malformed input') {
        return new Error('malformed input');
    }
    // 后续可以轻松扩展更多验证规则，比如类型校验、长度限制等
    return null; // 验证通过时返回null
}

function doSomething(inputVar, callback) {
    // 调用验证函数，快速判断输入是否合法
    const validationError = validateInputVar(inputVar);
    if (validationError) {
        callback(validationError);
        return;
    }

    console.log("You shouldn't see this message when something goes wrong with the validation.");
    // 执行核心业务操作
    callback(null);
}

方案优势 #

• 职责清晰：validateInputVar只处理输入验证，doSomething专注于核心业务逻辑，代码分工明确，后期维护时更容易定位问题。
• 复用性强：如果其他函数需要相同的输入校验逻辑，直接调用这个验证函数即可，避免重复造轮子。
• 可读性高：主函数的流程一目了然——"先验证，再执行核心逻辑"，减少了嵌套层级，阅读和理解成本更低。

如果你的项目中有大量类似的验证需求，还可以把这类验证函数统一放到一个工具模块（比如inputValidators.js）里，让整个项目的验证逻辑更易管理。

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