使用 __get、__set 和 __unset 时类中出现意外的数组内容

Question

我已经为我遇到的问题创建了一个简短的演示。这并不完全是我的实现方式，但似乎会产生相同的结果。

<?php

class mainclass {

    var $vardata = array();

    function &__get ($var) {
        if ($this->vardata[$var]) return $this->vardata[$var];
        if ($var == 'foo') return $this->_loadFoo();
        return NULL;
    }

    function __set ($var, $val) {
        $this->vardata[$var] = $val;
    }

    function __unset($var) {
        unset($this->vardata[$var]);
    }

}

class extender extends mainclass {

    function __construct() {

        var_dump($this->foo);
        $this->_loadFoo();
        echo '<br>';
        var_dump($this->foo);

    }

    function _loadFoo () {

        unset($this->foo);
        $this->foo = array();

        $this->foo[] = 'apples';
        $this->foo[] = 'oranges';
        $this->foo[] = 'pears';

        return $this->foo;

    }

}


$test = new extender;

?>

上面代码的输出是：

array(3) { [0]=>  string(6) "apples" [1]=>  string(7) "oranges" [2]=>  string(5) "pears" }
array(5) { [0]=> string(6) "apples" [1]=> string(7) "oranges" [2]=> string(5) "pears" [3]=> string(7) "oranges" [4]=> string(5) "pears" } 

正如我所期望的：

array(3) { [0]=>  string(6) "apples" [1]=>  string(7) "oranges" [2]=>  string(5) "pears" }
array(3) { [0]=>  string(6) "apples" [1]=>  string(7) "oranges" [2]=>  string(5) "pears" }

__get、__set 和 __unset 函数都在正确的位置被调用，所以我期望第二次直接调用该函数来简单地取消设置 $this- >foo 并用相同的数据再次填充它。这意味着 var_dumping 会给出相同的输出。相反，它最终会执行上述操作...用直接设置的三个字符串中的两个填充它，并保留前三个最初设置的字符串。

也许只是一个愚蠢的错误或对重载函数的误解 - 不管怎样，我已经被困在这里太久了，所以非常感谢任何帮助！

Answer 1

您的代码有很多问题：

• if ($this->vardata[$var]) 不是检查变量是否存在的方法； isset 或（如果允许 null，array_key_exists）是。
• mainclass 调用了它没有的函数。您应该将其声明为抽象的（否则它仍然可以工作，但是如果您实例化主类，您将使自己面临错误）。
• 您没有决定是否希望 _loadFoo 设置缺失的数组元素（与附带效果一起使用）或返回 foo 属性的默认值。由于您在 __get 中通过引用返回，我想您想要一个实际的变量而不仅仅是一个返回值，所以我选择了第一条路线。您让 _loadFoo 完成这两件事 – 设置缺失的数组索引并返回值；让事情变得复杂的是，您没有直接设置数组索引，而是依赖于重载。

更正版本：

<?php

abstract class mainclass {

    var $vardata = array();

    function &__get ($var) {
        if (isset($this->vardata[$var])) return $this->vardata[$var];
        if ($var == 'foo') {
            $this->_loadFoo(); /* called for collaterals */
            return $this->foo;
        }
        return NULL;
    }

    abstract function _loadFoo();

    function __set ($var, $val) {
        $this->vardata[$var] = $val;
    }

    function __unset($var) {
        unset($this->vardata[$var]);
    }

}

class extender extends mainclass {

    function __construct() {

        var_dump($this->foo);
        $this->_loadFoo();
        echo '<br>';
        var_dump($this->foo);

    }

    function _loadFoo () {

        unset($this->foo);
        $this->foo = array();

        $this->foo[] = 'apples';
        $this->foo[] = 'oranges';
        $this->foo[] = 'pears';

    }

}

$test = new extender;