Есть ли возможность оптимизировать вызов функции по имени?

Question

[rapidum_alder]

У меня есть кусок из класса.
Хочу что бы при разном значении $type у меня вызывалась соответствующая функция.
Изначальный CASE я заменил на такую конструкцию, но теперь вопрос а можно сделать вызов ещё проще? (В одну строчку)

public function getData($id, $type){
        $functionName = 'get' . $type . 'Data';
        return $this->$functionName($id);
    }

Comments

[Adamos]

Можно ее еще успешнее сократить. Выкинуть на хрен и вызывать специализированные функции по имени, раз оно все равно передается как аргумент.

[rapidum_alder]

Adamos, Вот как раз хочется так сделать (напрашивается) Но я не знаю как сделать вызов функции вида return $this->'get' . $type . 'Data'($id);
Так точно ошибочно.

[Adamos]

rapidum_alder, тогда зачем вы делаете разные функции, а не switch в одной?

[rapidum_alder]

Adamos,
У меня был Switch / Case, Но со временем элементов стола больше 5-ти и возможно будет увеличиваться.
А так как есть ещё функция типа $this->'get' . $type . 'HTML'($id) и для некоторых $this->'out' . $type . 'DataBase'($id).
Рациональней сделать вызов в 1-ну или 2 строки.

[Adamos]

rapidum_alder, нет. Если у вас есть подобные сущности, которые нужно обрабатывать однообразно, но по-разному - имеет смысл просто превратить их в набор классов с общим интерфейсом. А не маяться склеиванием строк, которое однажды неизбежно соберет несуществующее имя функции.

[FanatPHP]

Adamos, я бы оформил это ответом. Даже если этот непризнанный гений все равно сделает по-своему, то хотя бы другие будут знать что проблему можно и нужно решать по-человечески.

[Adamos]

FanatPHP, автор так сформулировал вопрос, что вряд ли кто-либо когда-либо придет сюда по подобной проблеме ;)

Answer 1

[Александр]

public function getData($id, $type){
        return $this->{'get' . $type . 'Data'}($id);
    }

// или, понятно дело

// ...
$this->{'get' . $type . 'Data'}($id)
// ...

правда не понятно зачем...

Answer 2

[Илья]

То что вы сделаете вызов в 1 строку - не значит что он станет проще. Сейчас все читаемо и понятно. А вообще для таких целей использовать бы фабричный метод, но мы же говорим про динамический язык, поэтому в топку)

Comments

[rapidum_alder]

А можно уточнить каким методом вызывается в одну строчку. Вдруг потом понадобится.

[nokimaro]

rapidum_alder,

Речь про то что изначальный вариант более наглядный и читабельный в плане кода

$functionName = 'get' . $type . 'Data';
return $this->$functionName($id);

а такой хуже
return $this->{'get' . $type . 'Data'}($id);

при том что разницы в плане производительности нет

[Александр]

nokimaro, смотря в каком контексте хуже. Возможно, у него там таких переменных сотни, а некоторые из них глобальные, загадит память тока так.

[Илья]

Александр, что простите? Это 1 метод, глобальный контекст не используется без вызова global

[Александр]

Илья, правда чтоль? вот блин, а я то говорил

Возможно

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