Магические методы PHP __get и __set

Question

Если я не полностью ошибаюсь, методы __get и __set должны разрешать перегрузку → get и set.

Например, следующие операторы должны вызывать__get method:

echo $foo->bar;
$var = $foo->bar;

И следующее должно использовать метод __set:

$foo->bar = 'test';

Это не работает в моем коде и воспроизводимо с помощью этого простого примера:

class foo {

    public $bar;
    public function __get($name) {

        echo "Get:$name";
        return $this->$name;
    }

    public function __set($name, $value) {

        echo "Set:$name to $value";
        $this->$name = $value;
    }
}


$foo = new foo();

echo $foo->bar;
$foo->bar = 'test';

echo "[$foo->bar]";

Это приводит только к следующему:

[test]

Добавление туда некоторых вызовов die() показывает, что оно вообще не срабатывает.

ПокаЯ только что сказал, привинтите его, и я вручную использую __get там, где это необходимо на данный момент, но это не очень динамично и требует знания, что «перегруженный» код на самом деле не вызывается, если не вызывается специально.Я хотел бы знать, не должен ли он функционировать так, как я понял, или он почему-то не работает.

Это работает на php 5.3.3.

Answer 1

__get, __set, __call и __callStatic вызываются, когда метод или свойство недоступны. Ваш $bar общедоступен и поэтому не доступен.

См. Раздел о перегрузке свойств в руководстве:

• __set() запускается при записи данных в недоступные свойства.
• __get() используется для чтения данных из недоступных свойств.

Магические методы не заменяют геттеры и сеттеры. Они просто позволяют обрабатывать вызовы методов или доступ к свойствам, которые в противном случае привели бы к ошибке. Таким образом, есть гораздо больше, связанных с обработкой ошибок. Также обратите внимание, что они значительно медленнее, чем использование правильных методов получения и установки или прямых вызовов методов.

Answer 2

Я бы порекомендовал использовать массив для хранения всех значений через __set().

class foo {

    protected $values = array();

    public function __get( $key )
    {
        return $this->values[ $key ];
    }

    public function __set( $key, $value )
    {
        $this->values[ $key ] = $value;
    }

}

Таким образом, вы убедитесь, что не можете получить доступ к переменным другим способом (обратите внимание, что $values защищен), чтобы избежать столкновений.

Answer 3

Из руководства PHP :

• __ set () запускается при записи данных в недоступные свойства.
• __ get () используется для чтения данных из недоступных свойств.

Это вызывается только для чтения / записи недоступных свойств.Ваша собственность, однако, является публичной, что означает, что она доступна.Изменение модификатора доступа на защищенный решает проблему.

Answer 4

Чтобы расширить ответ Берри, установка уровня доступа на защищенный позволяет использовать __get и __set с явно объявленными свойствами (по крайней мере, при доступе вне класса), а скорость значительно ниже, я процитирую комментарий из другого вопроса на эту тему и аргументируйте это тем, что используете:

Я согласен, что __get более медленен для пользовательской функции get (делает то же самое), это 0,0124455 времени для __get (), и этот 0,0024445 для пользовательского get () после 10000 циклов. - Melsi 23 ноября '12 в 22:32 Лучшая практика: магические методы PHP __set и __get

Согласно тестам Мелси, значительно медленнее, примерно в 5 раз медленнее. Это определенно значительно медленнее, но также обратите внимание, что тесты показывают, что вы все еще можете обращаться к свойству с помощью этого метода 10000 раз, считая время для итерации цикла примерно за 1/100 секунды. Он значительно медленнее по сравнению с фактическими определенными методами get и set, и это преуменьшение, но в общей схеме, даже в 5 раз медленнее никогда не бывает на самом деле медленно.

Время вычисления операции все еще незначительно и не стоит учитывать в 99% реальных приложений. Единственный раз, когда этого действительно следует избегать, это когда вы на самом деле собираетесь обращаться к свойствам более 10 000 раз за один запрос. Сайты с высоким трафиком делают что-то действительно неправильное, если они не могут позволить себе задействовать еще несколько серверов для поддержки своих приложений. Однострочная текстовая реклама в нижнем колонтитуле сайта с высоким трафиком, где скорость доступа становится проблемой, вероятно, может заплатить за ферму из 1000 серверов с этой строкой текста. Конечный пользователь никогда не станет стучать пальцами, задаваясь вопросом, что так долго загружает страницу, потому что доступ к свойству приложения занимает миллионную долю секунды.

Я говорю это как разработчик, пришедший из среды .NET, но невидимые методы get и set для потребителя не являются изобретением .NET. Они просто не являются свойствами без них, и эти магические методы являются спасительной милостью разработчика PHP для того, чтобы даже назвать свою версию свойств «свойствами» вообще. Кроме того, расширение Visual Studio для PHP поддерживает intellisense с защищенными свойствами, с учетом этого трюка, я думаю. Я думаю, что при достаточном количестве разработчиков, использующих магические методы __get и __set, разработчики PHP настроят время выполнения для удовлетворения потребностей сообщества разработчиков.

Редактировать: Теоретически защищенные свойства выглядели так, как будто они работают в большинстве ситуаций. На практике оказывается, что вы часто захотите использовать свои методы получения и установки при доступе к свойствам в пределах определения класса и расширенных классов. Лучшим решением является базовый класс и интерфейс для расширения других классов, так что вы можете просто скопировать несколько строк кода из базового класса в реализующий класс. Я делаю немного больше с базовым классом моего проекта, поэтому у меня нет интерфейса для предоставления прямо сейчас, но вот непроверенное урезанное определение класса с получением и установкой магического свойства с использованием отражения для удаления и перемещения свойств в защищенный массив:

/** Base class with magic property __get() and __set() support for defined properties. */
class Component {
    /** Gets the properties of the class stored after removing the original
     * definitions to trigger magic __get() and __set() methods when accessed. */
    protected $properties = array();

    /** Provides property get support. Add a case for the property name to
     * expand (no break;) or replace (break;) the default get method. When
     * overriding, call parent::__get($name) first and return if not null,
     * then be sure to check that the property is in the overriding class
     * before doing anything, and to implement the default get routine. */
    public function __get($name) {
        $caller = array_shift(debug_backtrace());
        $max_access = ReflectionProperty::IS_PUBLIC;
        if (is_subclass_of($caller['class'], get_class($this)))
            $max_access = ReflectionProperty::IS_PROTECTED;
        if ($caller['class'] == get_class($this))
            $max_access = ReflectionProperty::IS_PRIVATE;
        if (!empty($this->properties[$name])
            && $this->properties[$name]->class == get_class()
            && $this->properties[$name]->access <= $max_access)
            switch ($name) {
                default:
                    return $this->properties[$name]->value;
            }
    }

    /** Provides property set support. Add a case for the property name to
     * expand (no break;) or replace (break;) the default set method. When
     * overriding, call parent::__set($name, $value) first, then be sure to
     * check that the property is in the overriding class before doing anything,
     * and to implement the default set routine. */
    public function __set($name, $value) {
        $caller = array_shift(debug_backtrace());
        $max_access = ReflectionProperty::IS_PUBLIC;
        if (is_subclass_of($caller['class'], get_class($this)))
            $max_access = ReflectionProperty::IS_PROTECTED;
        if ($caller['class'] == get_class($this))
            $max_access = ReflectionProperty::IS_PRIVATE;
        if (!empty($this->properties[$name])
            && $this->properties[$name]->class == get_class()
            && $this->properties[$name]->access <= $max_access)
            switch ($name) {
                default:
                    $this->properties[$name]->value = $value;
            }
    }

    /** Constructor for the Component. Call first when overriding. */
    function __construct() {
        // Removing and moving properties to $properties property for magic
        // __get() and __set() support.
        $reflected_class = new ReflectionClass($this);
        $properties = array();
        foreach ($reflected_class->getProperties() as $property) {
            if ($property->isStatic()) { continue; }
            $properties[$property->name] = (object)array(
                'name' => $property->name, 'value' => $property->value
                , 'access' => $property->getModifier(), 'class' => get_class($this));
            unset($this->{$property->name}); }
        $this->properties = $properties;
    }
}

Приношу свои извинения, если в коде есть ошибки.

Answer 5

Это потому, что $ bar является публичной собственностью.

$foo->bar = 'test';

Нет необходимости вызывать магический метод при запуске вышеупомянутого.

Удаление public $bar; из вашего класса должно исправить это.

Answer 6

Лучшее использование магических методов set / get с предопределенными пользовательскими методами set / get, как в примере ниже.Таким образом, вы можете объединить лучшее из двух миров.С точки зрения скорости, я согласен, что они немного медленнее, но вы можете даже почувствовать разницу.Пример ниже также проверяет массив данных по заранее определенным установщикам.

"Магические методы не заменяют геттеры и сеттеры. Они просто позволяют обрабатывать вызовы методов или доступ к свойствам, которые в противном случае привели бы к ошибке."

Вот почему мы должны использовать оба.

ПРИМЕР КЛАССА

    /*
    * Item class
    */
class Item{
    private $data = array();

    function __construct($options=""){ //set default to none
        $this->setNewDataClass($options); //calling function
    }

    private function setNewDataClass($options){
        foreach ($options as $key => $value) {
            $method = 'set'.ucfirst($key); //capitalize first letter of the key to preserve camel case convention naming
            if(is_callable(array($this, $method))){  //use seters setMethod() to set value for this data[key];      
                $this->$method($value); //execute the setters function
            }else{
                $this->data[$key] = $value; //create new set data[key] = value without seeters;
            }   
        }
    }

    private function setNameOfTheItem($value){ // no filter
        $this->data['name'] = strtoupper($value); //assign the value
        return $this->data['name']; // return the value - optional
    }

    private function setWeight($value){ //use some kind of filter
        if($value >= "100"){ 
            $value = "this item is too heavy - sorry - exceeded weight of maximum 99 kg [setters filter]";
        }
        $this->data['weight'] = strtoupper($value); //asign the value
        return $this->data['weight']; // return the value - optional
    }

    function __set($key, $value){
        $method = 'set'.ucfirst($key); //capitalize first letter of the key to preserv camell case convention naming
        if(is_callable(array($this, $method))){  //use seters setMethod() to set value for this data[key];      
            $this->$method($value); //execute the seeter function
        }else{
            $this->data[$key] = $value; //create new set data[key] = value without seeters;
        }
    }

    function __get($key){
        return $this->data[$key];
    }

    function dump(){
        var_dump($this);
    }
}

INDEX.PHP

$data = array(
    'nameOfTheItem' => 'tv',
    'weight' => '1000',
    'size' => '10x20x30'
);

$item = new Item($data);
$item->dump();

$item->somethingThatDoNotExists = 0; // this key (key, value) will trigger magic function __set() without any control or check of the input,
$item->weight = 99; // this key will trigger predefined setter function of a class - setWeight($value) - value is valid,
$item->dump();

$item->weight = 111; // this key will trigger predefined setter function of a class - setWeight($value) - value invalid - will generate warning.
$item->dump(); // display object info

ВЫХОД

object(Item)[1]
  private 'data' => 
    array (size=3)
      'name' => string 'TV' (length=2)
      'weight' => string 'THIS ITEM IS TOO HEAVY - SORRY - EXIDED WEIGHT OF MAXIMUM 99 KG [SETTERS FILTER]' (length=80)
      'size' => string '10x20x30' (length=8)
object(Item)[1]
  private 'data' => 
    array (size=4)
      'name' => string 'TV' (length=2)
      'weight' => string '99' (length=2)
      'size' => string '10x20x30' (length=8)
      'somethingThatDoNotExists' => int 0
object(Item)[1]
  private 'data' => 
    array (size=4)
      'name' => string 'TV' (length=2)
      'weight' => string 'THIS ITEM IS TOO HEAVY - SORRY - EXIDED WEIGHT OF MAXIMUM 99 KG [SETTERS FILTER]' (length=80)
      'size' => string '10x20x30' (length=8)
      'somethingThatDoNotExists' => int 0

Answer 7

Intenta con:

__GET($k){
 return $this->$k;
}

_SET($k,$v){
 return $this->$k = $v;
}

Answer 8

Удалите объявление public $bar;, и оно должно работать как положено.