干净地验证输入

Question

在谈论“干净的代码”时，我们通常会听到编写干净的函数只做一件事。

在处理用户输入时，它是否算作验证输入的一件事的一部分？例如，

/* Arg1 is passed directly from user input */
int foo(int arg1) {
    if (!arg1 || arg1 < 500) {
        return -1;
    }

    /* Do foo */
}

最好在函数内部编写这些验证测试，还是应该以某种方式将它们分开，如下所示：

if (argIsValid(arg1)) {
    foo(arg1);
}

无论如何，我想听听你的回答支持推理。谢谢！

Answer 1

最好在函数中编写验证逻辑，例如名称为validate_args()。

你看，当你想增强验证逻辑时，很明显你也要看看，即使你不是代码的作者，所以 +1 维护！

而且，它也让代码更优雅！