您是否认为在 PHP 中访问类方法中的超级全局变量是一种不好的形式？

Question

以 Account 类 中的 login() 函数为例。

class Account {
 /* Class variables */

    public function login() {
        if(isset($_POST['username']) && isset($_POST['password']))
            return $this->_formLogin();
        else if(isset($_SESSION['accountId']))
            return $this->_sessionLogin();
        else if(isset($_COOKIE['username']) && isset($_COOKIE['password']))
            return $this->_cookieLogin();
        else return false;
    }

    private function _formLogin() {
        //perform login actions using $_POST data
    }
    /* All that other stuff */
}

现在尝试忽略对数据清理、密码加盐等看不见的方法的任何担忧。 严格关注login()，这个全局访问是坏juju吗？ 我通常避免在类中使用 PHP 超级全局变量，但我想不出在这种情况下不这样做的好理由。

我可以理解为什么您不希望在跨类交互的全局变量中发生后台魔法，但这些全局变量内置于 PHP 中，不会被类修改，并且仅由此类使用。

这会导致在页面开头出现这样的情况，您需要用户登录：

$user = new Account($whatever, $objects, $we, $depend, $on);
if($user->login()) {
    //Do this stuff when logged in
}

而不是在每个页面上都这样，稍后可能需要更改其逻辑：

$user = new Account($whatever, $objects, $we, $depend, $on);
if(isset($_POST['username']) && isset($_POST['password']))
    $user->formLogin($_POST['username'], $_POST['password']);
else if(isset($_SESSION['accountId']))
    $user->sessionLogin($_SESSION['accountId']);
else if(isset($_COOKIE['username']) && isset($_COOKIE['password']))
    $user->cookieLogin($_COOKIE['username'], $_COOKIE['password']);
if($user->isLoggedIn() {
    //Do this stuff when logged in
}

虽然我知道在类之外创建一个函数处理这个问题是一种选择，那不是和在类中混淆全局变量一样糟糕吗？

Answer 1

我不会说这个问题有一个直接的是或否的答案。 （所有超全局变量 $_GET $_POST $_SESSION）的想法是，您正在请求位于整个应用程序中的数据，并且不在您要求的范围内。

这些超全局变量可能发生的情况是，如果它们在函数执行之前或（上帝禁止）期间由于某种原因在某个地方发生了变化，该怎么办？ 这可能会成为一个非常烦人的错误。

所以我想说这是不好的形式。

Answer 2

为了回答我自己的问题，我会说是的，访问超级全局变量很好，只要您不修改它们以跨多个类访问即可。 后台并没有发生什么魔法——您只是读取状态，而超级全局变量就是 PHP 为您提供的方式。

但是，您永远不应该修改类中的全局变量并在其他地方访问它。 那就是你使单元测试变得不可能的时候。

Answer 3

一种方法是将所有超全局变量包装到它自己的类中。 我非常确定 Zend Framework 有一个自己的类，例如用于操作 Cookie。

Answer 4

我想说这取决于您的应用程序的设计。 如果该类是一些通用的、松散耦合的类或模块，并且突然使用此类全局变量，那么在我的书中，这将是不好的做法。 但是，如果该类显然是针对需要这些特定全局变量的特定任务（例如您的登录示例），我没有看到任何明确的反对意见。