Как вызывать метод один раз а потом запоминать его значение?

Question

[Evgeniy]

Добрый день. В PHP записал класс

Class Neptun(){
  private function method1(){
    $users_count = get_users();
    return $users_count;
  }
  private function method2(){
    if($this->method1 > 10){
        return 'it more than 10';
     }
   }
private function method3(){
    if(in_array(10, $this->method1())){
         return 'there are user with ID 10';
     }

public function finish(){
   echo $this->method2() . ' ' . $this->method3();
}
}

В таком пример method 1 запускает функцию get_users() 2 раза. Как сделать чтобы функция get_users() сработала только 1 раз а во всех остальных случаях подставлялась статическая переменная? Надеюсь, объяснил все правильно)

Comments

[Evgeniy]

Есть вариант внутри метода создать свойства, и при каждом обращении проверять. Если свойства есть выдавать сразу свойство, а если свойства нет то запускать метод. Нормальный подход?

[Дмитрий]

Edgark, нормальный

[shurshur]

Можно погуглить по слову memoization идеи для решения такой задачи. Есть даже библиотеки.

[Evgeniy]

shurshur, спасибо за наводку, обязательно посмотрю)

[Виталий Артемьев]

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

Answer 1

[Михаил]

Статическая переменная в функции. И проверять - если не null - то возвращать её значение. Это один из самых простых вариантов кэширования. При наличии у функции аргументов - статический массив.

Проблемы возникают, как всегда с кэшированием, когда кэш устаревает. То есть, у вас добавится запись в БД (или откуда get_users берет пользователей) - а кэш останется старым. В случае PHP это частично снимается тем, что он перезапускает скрипт на каждый запрос, то есть такой кэш будет жить в пределах одного запуска скрипта. Но тем не менее, это может привести к глюкам.

Вообще, прежде чем добавлять такое кэширование нужно хорошо подумать:
а) а что я тут выиграю? Даже если это запрос в БД - у БД есть свой кэш, так что второй раз запрос отработает очень быстро. Критично ли что их будет 2?
б) можно ли переписать код, чтобы это работало не так?

По второму случаю у вас в коде нарушение принципа единственности ответственности: функции method2 и method3 жестко связаны (coupled) с функцией method1 (а функция finish - с ними всеми). Собственно это и порождает проблему. Код можно переписать, например так:

сlass Neptun(){
  private function method1(){
    $users_count = get_users();
    return $users_count;
  }

  private function method2($users){
    if($users > 10){
        return 'it more than 10';
     }
   }

  private function method3($users){
    if(in_array(10, $users)){
         return 'there are user with ID 10';
    }
  }

  public function finish(){
    $users = $this->method1();
     echo $this->method2($users) . ' ' . $this->method3($users);
  }
}

Все, функции method2 и method3 "расцеплены" (decoupled) с функцией method1. А "сборку" в общую конструкцию осуществляет публичный метод finish

Comments

[Evgeniy]

Спасибо Михаил, я как-то даже и не подумал об этом. В таком случае получается что method1 сработает только 1 раз и подставится к остальным методам. Очень интересное решение.

Answer 2

[Дмитрий]

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

Comments

[Evgeniy]

а в данном случае модификатор static может как-то помочь?
У меня же method1() не меняется, если проставлю static, это тоже вроде кэша будет работать?

[Дмитрий]

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

[Evgeniy]

Дмитрий, Хорошо, спасибо Дмитрий

[Evgeniy]

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

[Дмитрий]

trait GetCache
{
    public function __call(string $name, $arguments) {
        $action = substr($name, 0, 5);
        $property = strtolower(substr($name, 5));
        switch ($action) {
            case 'cache':
                if (!isset($this->$property)) {
                    $this->$property = $this->$property(...$arguments);
                }
                
                return $this->$property;
            break;
            default :
                return false; // Exception
            break;
    }
}

Как то так, с телефона писал, надо с компа проверить и поправить
Смысл такой, есть класс где методы для получения данных
Например news() authors()
Подключаем трейт к классу и вызываем $res = cacheNews();
Если нужны аргументы, их там тоже дописать можно
Так же наверное будет лучше использовать static вместо this
Дома с компа гляну

[Evgeniy]

Дмитрий, хорошая идея! Вместо того чтобы везде писать !isset($this->$property), можно использовать улучшенную версию с трейтом.