Как кешировать переменные на уровне php?

Question

[vitaly_74]

имеется такой код:

static $graph = false;
        if($graph !== false){
            return $graph;
        }
        $graph =  DB::getRecords();
        return $graph;

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

Comments

[FanatPHP]

static $graph;
return isset($graph) ? $graph : $graph = getRecords();

и не забивать себе голову ерундой
три строчки - это не тот случай, когда надо бояться "повторяющейся конструкции"

[vitaly_74]

FanatPHP, у меня условный graph меняется от класса к классу, так не пойдет. т.е. его значение не равно какой то определенной функции, эти функции всегда разные, и значения разные в зависимости от класса.
по сути в другом классе может быть так:
static $client;
return isset($client) ? $client : $client = DB::find()->where(['id'=>123])->one();
и каждый раз такую конструкцию не хотелось бы городить

[FanatPHP]

не о том думаете.
это не тот случай, который надо оптимизировать.

Answer 1

[rPman]

Что именно убрать? слишком большой код?, заверни в функцию с передачей имени переменной в аргументах:

function cache($name,$callback)
{
	static $cache = [];

	if(isset($cache[$name])) return $cache[$name];
	return $cache[$name] = $callback();
}

var_dump(cache('graph',fn()=>DB::getRecords()));

и ошибки получения данных разрули, и инвалидацию (тогда к cache нужен доступ из разных методов, а значит лучше оформить в виде класса, или если процедурно, добавить управляющий параметр в функцию)

Comments

[vitaly_74]

у меня условный graph меняется от класса к классу, так не пойдет. т.е. его значение не равно какой то определенной функции, эти функции всегда разные, и значения разные в зависимости от класса.
по сути в другом классе может быть так:
static $client;
return isset($client) ? $client : $client = DB::find()->where(['id'=>123])->one();
и каждый раз такую конструкцию не хотелось бы городить

[FanatPHP]

что ж вы все такие непонятливые-то
не "слишком большой код", а "повторяющиеся конструкции"
чтобы эта функция поддерживала shraf, в неё надо будет добавить ещё две строчки. вот их-то автор и боится.

vitaly_74, ради бога, добавьте в вопрос второй пример кода, а то люди не понимают, что такое "повторяющийся"

[rPman]

тогда решение очевидно, нужно научить класс ORM DB кешировать данные (многие умеют но синтаксически приятным это не назовешь)

upd. обновил пример в ответе, юзай лямбды

[vitaly_74]

rPman, да действительно умеет кешировать, но настраивать кеш для каждого запроса тоже геморно, либо он будет кушировать всё

[FanatPHP]

Это всё шило на мыло.
Бывает такой оптимизационный зуд, когда хочется что-то улучшить, а в итоге получается только хуже.
Вот здесь именно такой случай.

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