我注意到有一些函数，例如 is_int() 或 isset() 或 file_exists() 或 functions_exists( ) 这在某种程度上非常有用。当我编写一些代码时，我总是会考虑我的网站可能发生的每件坏事，但有时我会遇到以下问题:

Wait, this variable is set inside the PHP file; this means that no one could ever edit it, right? And if this "user" could edit it I would have a lot of more troubles because it would be able to manage the PHP file.

或

Does this really worth something to keep checking for file that should exists always?

让我们考虑以下示例，它本身没有任何意义，但可以帮助我让您理解我在说什么。 PS:我故意夸大了代码。

config.php

$doActions = true;

函数.php

function getID() {
    return $_COOKIE['userid'];
}

class eye {
    public static function see() {
        // gain the object the user is looking at
        return $object;
    }
}

index.php

class viewer {

    private $poniesSeen = 0;

    public function __construct() {
        /* Magic ponies are created here */
    }

    public function sawAPony($id) {
        if (file_exists('config.php')) {
            if (isset($doActions)) {
                if (is_bool($doActions)) {
                    if ($doActions) {
                        if (file_exists('functions.php')) {
                            if (function_exists('getID')) {
                                $id = getID();
                                if (!empty($id)) {
                                    if (!is_int($id)) {
                                        settype($id, 'int');
                                    }

                                    if (class_exists('eye')) {
                                        if (method_exists('eye', 'see')) {
                                            $o = eye::see();
                                            if (is_string($o)) {
                                                if ($o = 'pony') {
                                                    if (isset($this->poniesSeen) and is_int($this->poniesSeen)) {
                                                        ++$this->poniesSeen;
                                                        return true;
                                                    } else {
                                                        return false;
                                                    }
                                                } else {
                                                    return false;
                                                }
                                            } else {
                                                return false;
                                            }
                                        } else {
                                            return false;
                                        }
                                    } else {
                                        return false;
                                    }
                                } else {
                                    return false;
                                }
                            } else {
                                return false;
                            }
                        } else {
                            return false;
                        }
                    } else {
                        return false;
                    }
                } else {
                    return false;
                }
            } else {
                return false;
            }
        } else {
            return false;
        }
    }
}

现在，我想，这些条件中哪些应该保留，哪些因毫无意义而被丢弃？为什么我不应该检查它们，为什么我应该检查它们？是否存在关于这种痴迷的黄金法则？

最佳答案

这就是语言(通常)带有错误消息的原因。如果出现问题，它不会静静地崩溃，它会告诉你。因此，您可以对您认为应该始终正确的内容做出合理的假设，如果由于某种原因它不正确，您会收到通知。

以下检查没有用，IMO:

1. file_exists - 如果文件名已知并且您将其放在那里。如果参数是一个变量，那么就可以了。
2. is_bool, is_int等，如果坚持类型正确，可以使用===进行比较。我用过的唯一一个是 is_array。
3. function_exists 等 - 如果您知道文件在那里，并且将函数放入文件中，则可以安全地假设该函数存在。同样，在某些特殊情况下此功能很有用 - 您的情况不是其中之一。

isset 在某些情况下很有用，例如检查数组是否包含给定键的非空值，或者是否在 $_REQUEST 中传递了参数。还有其他不涉及 isset 的检查方法，但这不是我的观点。

如果您可以合理地期望您的断言始终为真，那么您的数据损坏或巨大错误就会使其为假，请不要费心检查。如果一百万次尝试中确实出现一次错误，那么您可以捕获错误并在下次修复它。否则，代码可读性的代价太大了。

关于php - 开发强大的应用程序，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/5342378/