class Mobile_Detect

[korshunov]

Класс Mobile_Detect как-то своеобразно используется. Он подключается в Design.php через
require_once 'Mobile_Detect.php';
требует дополнительную переменную и дополнительное действие:
$this->detect = new Mobile_Detect();

Непонятно, почему не использован стандартный механизм, как в аналогичных случаях для прочих классов. У меня работает так:
1. в Okay.php добавить строку
'detect' => 'Mobile_Detect',
2. И потом можно использовать в разных местах
$is_mobile = $this->detect->isMobile();
$is_tablet = $this->detect->isTablet();
Вместо этого сейчас имеем изобретенный велосипед с дополнительными строками, который делает то же самое, только хуже...

Зачем?

[makki]

Видимо разработчики посчитали, что класс Mobile_Detect будет вызываться только один раз, чтобы через smarty передать в пользовательский шаблон $is_mobile и $is_tablet

[korshunov]

Видимо, так.
Только возникает вопрос - а ЗАЧЕМ они так посчитали? Неужто не знают основы того, как устроена система и не знают, как работают классы сейчас? Классы и функции для того и придуманы, чтобы можно было свободно вызывать их в любых местах в нужных случаях. И искусственно ограничивать это совсем ни к чему.

[m1hasik]

А вы не ну думали, что это сделали специально, что бы как раз таки функции не вызывали где угодно. Может это специально для верстальщиков только делали.

[korshunov]

Скорее похоже совсем на другое - они не задумывались о том, почему и зачем и как лучше, а просто делали по аналогии.
Например, в api/Design.php увидели подключение стороннего файла
require_once(dirname(dirname(__FILE__)).'/Smarty/libs/Smarty.class.php');
и рядом приписали свое
require_once 'Mobile_Detect.php';
И далее аналогично.

[m1hasik]

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

Согласен что программно это ограничивает. Но с другой стороны, чисто для верстки пока хватает.
И то, я если честно, ими пользовался раза два всего

[korshunov]

Я думаю, что если бы у них действительно были стоящие причины, они бы тут написали...