Как оптимизировать класс PHP?

Question

[Максим Иванов]

В классе много функций, и в каждой функции есть одна и таже строчка кода
$nameF = __FUNCTION__; $nameF = str_replace("_", ".", $nameF);

Как сделать так, чтобы это было автоматически в каждой функции класса, вдруг есть всякие спец. функции по типу __constract()?

Answer 1

[Павел Шорохов]

Добавьте кеширование. В определенных случаях это сильно соптимизирует дело.

protected $_aliasesCache = [];

public function getAlias($method)
{
	if (! array_key_exists($method, $this->_aliasesCache)) {
		$this->_aliasesCache[$method] = str_replace('_', '.', $method);
	}

	return $this->_aliasesCache[$method];
}

// Usage

public function some_method()
{
	$this->getAlias(__FUNCTION__);
}

Еще можно поиграться со стеком вызовов, чтобы не передавать аргумент, но я бы лучше это дело оставил так. При определении имя метода через стек, не забудьте изменить модификатор на protected/private.

Comments

[Сергей Протько]

имхо, излишнее кешировать подобное. и метод getAlias стоит сделать приватным.

[Павел Шорохов]

@Fesor все зависит от контекста. Я думаю быстрее будет миллион раз обратиться к кэшу, чем миллион раз вызывать str_replace(). Все решает контекст.

Answer 2

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

class Util
{
  public static function ConvertFunctionName ($function_name)
  {
    return str_replace("_", ".", $function_name);
  }
}

class Class
{
  public function foo ()
  {
    $function_name = Util::ConvertFunctionName(__FUNCTION__);
  }
}

Других способов, насколько я знаю, не существует.

Comments

[Павел Шорохов]

Существенно ваше предложение ничем не отличается от оригинального.

[vdem]

@comm1x Отличается, тем что функциональность вынесена в класс-хелпер и теперь нет ее дублирования.

[Павел Шорохов]

@vdem боюсь, что процессору вы это не объясните

[Максим Иванов]

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

[Сергей Протько]

@vdem а зачем вам класс хэлпер? Что за мода вообще пошла на каждую функцию вспомогательную класс хэлпер заводить? Этот метод нужен только в одном классе. так что этому методу самое место сисдеть в лике приватных методов и помогать редьюсить дублирование кода.

[vdem]

@Fesor В одном классе - да, приватный метод лучше конечно.

[Сергей Протько]

@vdem в нескольких классах - лучше инджектить в конструктор инатанс хэлпера или сделать трейт. Вообще проще было бы функции использовать, если бы в PHP было хоть какое-то подобие модулей и поддерживалась автозагрузка оных. Пока это единственное оправдание использования статических классов.

Answer 3

[Сергей Протько]

мне кажется стоит разобраться зачем вообще эта строчка там нужна (реально не понимаю, это же метод класса, зачем там брать имя себя же? Оно же вам уже известно...

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

От дублирования кода нормально избавляться вынося онвый в приватные методы.

p.s. пожалейте людей, код класса ни за что не показывайте.

Comments

[Максим Иванов]

троль

[Сергей Протько]

@omaxphp и чем я троль, позвольте спросить? Ну да, мне кажутся весьма странными привязка логики работы класса и его методов к названию самих методов.

решение я вам предложил даже с магией что бы вообще отчистить код от этой строчки, дальше ваше дело.

[Максим Иванов]

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

[Максим Иванов]

выше comm1x описал как сделать, собственно вы тоже самое через приватные методы предложили

[Сергей Протько]

@omaxphp это по сути рефлексия кода. До появления late static binding без константы __CLASS__ вообще тяжко обходиться было к примеру. Ну и удобно в вашем случае. С учетом того что появилось это дело в версии 4,3 PHP сейчас это не особо актуально.

С другой стороны я бы пользовался константами класса и хэшмэпой константа => алиас так как считаю такой подход более лаконичным. Хотя все от задачи зависит пожалуй.