Класс GRLAB+шаблонизатор

Последняя активность
Файлы топика
1.94K
.
Simba
15 мая 2015 г., 13:16
Кадило крутится, лавэха мутится
# Koenig (15.05.2015 / 13:12)
IdKoder, чтоб не парится с глобализацией переменных, так как не везде удается проследить все вхождения, например page id и прочее из ядра
такой вот код в конструктор шаблонизатора

foreach ($GLOBA
По мне, так лучше подавать в шаблон 1 массив со всеми данными. Проще смотреть что есть и не приходится думать какие переменные есть.
.
Rakovskiy
15 мая 2015 г., 13:43
Изменено
The Fast, Secure and Professional - Yii2
Детский сад, вторая четверть
.
L!MP
15 мая 2015 г., 13:47
Да тут вообще не понятно что это такое и какие советы давать.

Пока я тут вижу Registry, сделанный синглтоном, с сеттером у которого ключ и значение зачем-то перепутаны местами, без единой проверки на то, есть ли запрашиваемое значение в реестре.

Ну а последний код "модели" я вообще х.з.
.
L!MP
15 мая 2015 г., 13:49
# Koenig (15.05.2015 / 13:12)
foreach ($GLOBALS as $k => $v) {
global $$k;
$$k = $v;
}
Глядя на это хочется убивать, гг.
.
Koenig
15 мая 2015 г., 13:54
(\/)____o_O____(\/)
L!MP, а мне нравится, в случае где в шаблоны глобалятся отдельные переменные, а их там от шаблона к шаблону разное количество, это норм решение
.
Folour
15 мая 2015 г., 13:57
Think different
.
L!MP
15 мая 2015 г., 14:01
IdKoder, а в первом посте - это рабочий код? Просто там два класса-синглтона и ни в одном метод получения объекта правильно не написан.

Вот, например:

public static function getInstance ()
{
$class = __CLASS__ ;
self :: $instance =new $class ;
return self :: $instance ;
}


Оно же каждый раз будет возвращать новый объект.

И зачем все эти пляски с получение имени класса из __CLASS__ если уже лет сто как можно писать так:

protected static $instance;

public static function getInstance()
{
    return static::$instance ?: static::$instance = new static;
}
.
L!MP
15 мая 2015 г., 14:09
Koenig,
$GLOBALS

Хочется сильно ударить.
global

Хочется убить, быстро и без страданий.
$$k

Несите паяльник.
.
Koenig
15 мая 2015 г., 14:16
(\/)____o_O____(\/)
L!MP,
.
Folour
15 мая 2015 г., 14:35
Think different
L!MP, согласен)
Всего: 82
© 2024, JohnCMS