Лучшая практика: Волшебные методы PHP устанавливают и получают [дубликат]

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

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

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

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

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 действительно что-то делать или просто возвращать значение, это ответственность вызываемого абонента. Для вызывающего абонента $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);
    }
}

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

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

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

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

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

$класс->var;

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

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