Хак: Оптимизация загрузки классов кэша в InstantCMS 2.0.1 2 194 просмотра

В InstantCMS 2 в версиях до 2.0.1 включительно загрузка классов кэширования производится независимо от того, разрешено ли кэширование в настройках сайта или нет. Мотивацию разработчиков для этого я точно не знаю. Скорее всего это желание следовать принципам ООП, по которым проверка работы с кэшем должна осуществляться предпочтительно в классе кэширования.

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

Изменения делаются в файлах:
\index.php
\system\core\core.php
\system\core\eventsmanager.php
\system\core\model.php

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

Код тут

\index.php

cmsCache::getInstance()->start();

меняем на

if (cmsConfig::get('cache_enabled')) { cmsCache::getInstance()->start(); }

и строку

cmsCache::getInstance()->stop();

меняем на

if (cmsConfig::get('cache_enabled')) { cmsCache::getInstance()->stop(); }

\system\core\core.php

В функции runWidget()

        $cache_key = "widgets.{$widget['id']}";
        $cache = cmsCache::getInstance();
 
        if (!$widget_object->isCacheable() || false === ($result = $cache->get($cache_key))){
            $result = call_user_func_array(array($widget_object, 'run'), array());
            if ($result){
                // Отдельно кешируем имя шаблона виджета, поскольку оно могло быть
                // изменено внутри виджета, а в кеш у нас попадает только тот массив
                // который возвращается кодом виджета (без самих свойств $widget_object)
                $result['_wd_template'] = $widget_object->getTemplate();
            }
            $cache->set($cache_key, $result);
        }

меняем на

        $result = false;
        if ($widget_object->isCacheable() && cmsConfig::get('cache_enabled')){
            $cache_key = "widgets.{$widget['id']}";
            $cache = cmsCache::getInstance();
            $result = $cache->get($cache_key);
        }
 
        if ($result === false){
            $result = call_user_func_array(array($widget_object, 'run'), array());
            if ($result){
                // Отдельно кешируем имя шаблона виджета, поскольку оно могло быть
                // изменено внутри виджета, а в кеш у нас попадает только тот массив
                // который возвращается кодом виджета (без самих свойств $widget_object)
                $result['_wd_template'] = $widget_object->getTemplate();
            }
            if (isset($cache)) { $cache->set($cache_key, $result); }
        }

\system\core\eventsmanager.php

В функции getAllListeners начало и окончание изменить на:

    public static function getAllListeners(){
 
        $config = cmsConfig::getInstance();
        if ($config->cache_enabled) {
          $cache = cmsCache::getInstance();
          $cache_key = 'events';
 
          if (false !== ($structure = $cache->get($cache_key))){
              return $structure;
          }
        }
 
	...
 
        if ($config->cache_enabled) {$cache->set($cache_key, $structure, 86400); }
 
        return $structure;

\system\core\model.php

В функции getItem() в четырёх местах (строки 935, 959, 1047 и 1089)

if ($this->cache_key){

меняем на

if ($this->cache_key && cmsConfig::get('cache_enabled')){

Класс расширенной отладки для InstantCMS 2.0.1 (v.6) - UPD 2014.07.05 18:15

Хак: Оптимизация инклудов в InstantCMS 2.0.1 (v.1)

instantcms2 оптимизация хак

5 комментариев

Реклама #

lokanaft 10 лет назад #

Вся фишка в том, что её просто нельзя без кэша юзать

WebMan 10 лет назад #

Почему? Поясните Вашу мысль, пожалуйста.

lokanaft 10 лет назад #

Например вот этот код:

Код PHP:

    /**
     * Обновляет кеш списка привязки слушателей к событиям
     * @return boolean
     */
    public static function getAllListeners(){
 
        $cache = cmsCache::getInstance();
        $cache_key = 'events';
 
        if (false !== ($structure = $cache->get($cache_key))){
            return $structure;
        }

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

WebMan 10 лет назад #

Если в настройках кэш отключён, то никакие операции с кэшем не производятся. Это видно в \system\core\cache.php:

Код PHP:

    public function get($key){
        if (!cmsConfig::get('cache_enabled')) { return false; }
        ...
    }
 
    public function set($key, $value, $ttl=false){
        $config = cmsConfig::getInstance();
        if (!$config->cache_enabled) { return false; }
        ...
    }

И так далее. Во всех функциях класса кэша, везде проверки на разрешение его работы. Поэтому в getAllListeners() класс кэша вызывается, но реально ничего не делает, если в настройках разрешения нет. Если кэш запрещён, то в любом случае перечитываются все манифесты и собирается список евентов. Значит можно смело вынести проверку в getAllListeners() и вообще не вызывать (и даже не загружать) два класса кэша. Что я и сделал в этом хаке.

WebMan 10 лет назад #

Ещё предложение по поводу кэша исходя из темы про манифесты. Было бы разумно сделать более тонкие настройки кеширования: что именно и насколько кэшировать. Например, сделать галки "Разрешить кэширование ... (событий, виджетов и т.д.)" с полем для указания периода кеширования для каждого пункта. Тогда, например, можно спокойно включить кеширование списка хуков из манифестов на 24 часа и сократить загрузки файлов почти на 30 штук, оставив без кеширования виджеты и другие динамические вещи, которые должны собираться в реальном времени.

Хак: Оптимизация загрузки классов кэша в InstantCMS 2.0.1

Реклама #

lokanaft 10 лет назад #

WebMan 10 лет назад #

lokanaft 10 лет назад #

WebMan 10 лет назад #

WebMan 10 лет назад #

Еще от автора

Хуки-хухуки: Исключаем неактивных пользователей из списков

«Расширенная отладка» для InstantCMS 2.14.1 (v.14.1.2) – большое обновление для разработчиков

Использование расширенной отладки. Часть 11. Анализ ошибок 403/404 и редиректов

InstantCMS Team

О проекте

Поддержка

Дополнения