Awilum


_____________
Обучаю веб-программированию качественно и не дорого: http://school-php.com
Фрилансер, принимаю заказы: PHP, JS, AS (видео-чаты). Писать в ЛС (Личные сообщения на phpforum).

Fansoro 2.0.4 Release

• Добавлена возможность создавать settings.yml - файл настроек плагина отдельно от файла манифеста плагина.
• Добавлена возможность устанавливать значение по умолчанию в Config::get()
• Добавлены значения для Memcache и Redis
• Добавлен метод для очистки кеша Cache::clear()
• Исправлен баг в Pages::getPages() при работе с кешем.

http://fansoro.org/blog/fansoro-2.0.4

_____________
https://github.com/Awilum - Open Source Evangelist

chee, можешь обосновать чем Синглтоны плохи ? в моем случае, их вполне достаточно. Я конечно мог пойти нестатичным путем и прикрутить потом http://pimple.sensiolabs.org например но я не вижу зачем.

_____________
https://github.com/Awilum - Open Source Evangelist

Сами по себе синглтоны не плохи как паттерн, но в твоём случае они бесполезны, просто излишняя конструкция.

Вот к примеру, есть метод Cache::init, и есть конструктор

Можно смело переносить код из конструктора в init, получим

    public static function init()
    {
        if (!empty(static::$processed)) {
            return;
        }

        // Set current time
        static::$now = time();
        // Cache key allows us to invalidate all cache on configuration changes.
        static::$key = (Config::get('system.cache.prefix') ? Config::get('system.cache.prefix') : 'fansoro') . '-' . md5(ROOT_DIR . Fansoro::VERSION);
        // Get Cache Driver
        static::$driver = static::getCacheDriver();
        // Set the cache namespace to our unique key
        static::$driver->setNamespace(static::$key);
        self::$processed = true;
    }

Ах да, комментарии в теле данного метода, я бы полностью вырезал, ты занимаешься тавтологией, комментирую настолько банальный код.

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

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


static::$key = (Config::get('system.cache.prefix') ? Config::get('system.cache.prefix') : 'fansoro') . '-' . md5(ROOT_DIR . Fansoro::VERSION);


это должно быть инициализировано в контейнере и переданы через конструктор в твой объект, а так получается сильная зависимость от Config

bestxp, если честно, то там надо все переписать. Я уже выше писал, можно придраться практически к каждой строчке. ИМХО

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

На другом форуме уже задали вопросы, но ТС их проигнорировал. Я позволю себе продублировать их здесь:

bestxp, почему кеш не должен зависить от конфига ? в какой системе такое сделано иначе, так что бы она работала без конфигурации ?

_____________
https://github.com/Awilum - Open Source Evangelist

Awilum, ты его не правильно понял, он говорил что у тебя указан явный статический обьект для забора конфигурации, а не подменяемый объект с определенным интерфейсом. Короче говоря тебе намикнули, что статика не гуд и нужно юзать внедрение зависимостей

_____________
Люди, имеющие низкий уровень квалификации, делают ошибочные выводы, принимают неудачные решения и при этом неспособны осознавать свои ошибки в силу низкого уровня своей квалификации

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

_____________
https://github.com/Awilum - Open Source Evangelist

Awilum а выше предложений и пожеланий мало? какой смысл спрашивать предложения если они в игноре?

TMake, о каких предложениях вы говорите ? о совете "стрельнуть себе в ногу" ? оставьте это для себя.

_____________
https://github.com/Awilum - Open Source Evangelist