Лучшая практика: PHP Magic Methods set и get [дубликат]

вам нужно использовать магию, только если объект действительно "волшебный". Если у вас есть классический объект с фиксированными свойствами, используйте сеттеры и геттеры, они работают нормально.

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

я использую __get (и public properties) как можно больше, потому что они делают код гораздо более читаемым. Сравнить:

этот код недвусмысленно говорит о том, что я делаю:

echo $user->name;

этот код заставляет меня чувствовать себя глупо, что мне не нравится:

function getName() { return $this->_name; }
....

echo $user->getName();

разница между ними особенно очевидна при одновременном доступе к нескольким свойствам.

echo "
    Dear $user->firstName $user->lastName!
    Your purchase:
        $product->name  $product->count x $product->price
"

и

echo "
    Dear " . $user->getFirstName() . " " . $user->getLastName() . "
    Your purchase: 
        " . $product->getName() . " " . $product->getCount() . "  x " . $product->getPrice() . " ";

ли $a->b действительно надо do что-то или просто возвращать значение является обязанностью абонента. Для звонящего, $user->name и $user->accountBalance должно выглядеть одинаково, хотя последнее может включать сложные вычисления. В моих классах данных я использую следующий небольшой метод:

 function __get($p) { 
      $m = "get_$p";
      if(method_exists($this, $m)) return $this->$m();
      user_error("undefined property $p");
 }

когда кто-то звонит $obj->xxx и класс get_xxx определено, этот метод вызывается неявно. Таким образом, вы можете определить геттер, если вам это нужно, сохраняя при этом свой интерфейс однородным и прозрачным. Как дополнительный бонус это обеспечивает элегантный способ запоминания расчетов:

  function get_accountBalance() {
      $result = <...complex stuff...>
      // since we cache the result in a public property, the getter will be called only once
      $this->accountBalance = $result;
  }

  ....


   echo $user->accountBalance; // calculate the value
   ....
   echo $user->accountBalance; // use the cached value

итог: php-это динамический язык сценариев, используйте его таким образом, не притворяйтесь, что вы делаете Java или C#.

Я делаю смесь ответа Эдема и вашего второго кода. Таким образом, у меня есть преимущества общих геттеров / сеттеров (завершение кода в вашей IDE), простота кодирования, если я хочу, исключения из-за несуществующих свойств (отлично подходит для обнаружения опечаток: $foo->naem вместо $foo->name), читать только свойства и свойства соединения.

class Foo
{
    private $_bar;
    private $_baz;

    public function getBar()
    {
        return $this->_bar;
    }

    public function setBar($value)
    {
        $this->_bar = $value;
    }

    public function getBaz()
    {
        return $this->_baz;
    }

    public function getBarBaz()
    {
        return $this->_bar . ' ' . $this->_baz;
    }

    public function __get($var)
    {
        $func = 'get'.$var;
        if (method_exists($this, $func))
        {
            return $this->$func();
        } else {
            throw new InexistentPropertyException("Inexistent property: $var");
        }
    }

    public function __set($var, $value)
    {
        $func = 'set'.$var;
        if (method_exists($this, $func))
        {
            $this->$func($value);
        } else {
            if (method_exists($this, 'get'.$var))
            {
                throw new ReadOnlyException("property $var is read-only");
            } else {
                throw new InexistentPropertyException("Inexistent property: $var");
            }
        }
    }
}

Я голосую за третий вариант. Я использую это в своих проектах, и Symfony использует что-то вроде этого:

public function __call($val, $x) {
    if(substr($val, 0, 3) == 'get') {
        $varname = strtolower(substr($val, 3));
    }
    else {
        throw new Exception('Bad method.', 500);
    }
    if(property_exists('Yourclass', $varname)) {
        return $this->$varname;
    } else {
        throw new Exception('Property does not exist: '.$varname, 500);
    }
}

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

вы должны использовать stdClass Если вам нужны магические члены, если вы пишете класс-определите, что он содержит.

лучшей практикой было бы использование традиционных геттеров и сеттеров из-за самоанализа или размышления. В PHP (точно так же, как в Java) есть способ получить имя метода или всех методов. Такая вещь вернет " _ _ get "в первом случае и" getFirstField"," getSecondField " во втором (плюс сеттеры).

подробнее об этом: http://php.net/manual/en/book.reflection.php

теперь я возвращаюсь к сеттерам и геттерам, но я также помещаю геттеров и сеттеров в magic methos __get и __set. Таким образом, у меня есть поведение по умолчанию, когда я это делаю

$class - >var;

Это просто вызовет геттер, который я установил в _ _ get. Обычно я просто использую геттер напрямую, но есть еще несколько случаев, когда это просто проще.

второй пример кода гораздо более правильный способ сделать это, потому что вы берете полный контроль над данными, которые даны class. Есть случаи, когда __set и __get полезны, но не в этом случае.