Правильно ли использовать __call так?

Question

[WQP]

Здравствуйте. На сколько правильно использовать __call в таком случае (класс ниже)? Будет несколько методов set и не хотелось бы каждый раз $this возвращать, когда это можно сделать всего 1 раз

Class A

class A
{
    protected $data;

    public function __call($name, $arguments)
    {
        if ( method_exists($this, $name) ) {
            call_user_func_array([$this, $name], $arguments);
        }

        return $this;
    }
    
    public function get()
    {
        return $this->data;
    }

    protected function setPreset()
    {
        // тут код манипуляции с $data, без return
    }
}

$A = new A();

$data = $A->setPreset()
          ->get();

Answer 1

[Dmitry MiksIr]

__сall вызывается только если метода в классе нет, по-этому в вашем случае будете разное поведение при обращении к объекту извне и при обращении внутри класса, что не есть хорошо.
Ради экономии одной строчки с return такое не стоит делать.
Если уж делать, то полностью магию убрав setPreset вообще, или используйте IDE, который умеет генерить fluent setter

Comments

[WQP]

__сall вызывается на protected и private. Я и так ide использую.

[Dmitry MiksIr]

__call вызывается не "на protected", а на недоступности метода в конкретной области видимости. По-этому, на $A->setPreset() то он вызовется, а вот на $this->setPreset(), когда такой понадобится, не вызовется. И будете через пол года голову ломать, что не так, почему не работает.

[Алексей Уколов]

WQP: нет, не вызывается - ideone.com/cd3850

Answer 2

[Алексей Уколов]

На мой взгляд, соотношение принципа наименьшего удивления к полезности у такого подхода недопустимое и я бы так писать не стал.
Код становится менее понятным, нужно для каждого set-метода писать doc-блок с /** @return self */ (чтобы IDE не сходила с ума), что и вовсе убивает весь профит.
Ну и он накладывает определённый оверхед по производительности.

Comments

[WQP]

В любом случае приходится писать @retrun $this

[Алексей Уколов]

В случае явного return внутри функции 1) doc-блок можно не писать, 2) а если и писать, то часть с @return $this добавится автоматически.

Я сейчас поигрался с вашим классом и так и не смог научить IDE понимать все эти заковыки.

Answer 3

[nepster-web]

Старайтесь вообще не использовать магию и наследование, а так-же сеттеры.

1 - ide умеет само все генерировать. Если не умеет, смените Ваш текстовый редактор на ide.
2 - магия приводит к не очевидным вещам и сложности тестирования
3 - аккуратнее с сеттерами, используйте их реже, а все помещайте в конструктор.