PHP OOП статический метод

Да, так будет правильно. Но зачем вызывать статический метод, если он всё равно инициализирует объект ?

 

У тебя view - это уже объект, при условии использования нормального фреймворка. В этот объект ты можешь засунуть функцию получения объекта хелпера.
Причем хэлпер тоже можно объектом хранить.
В шаблоне приведенный выше пример можно использовать так:

PHP:

1. <? /** $var View $this **/?>
2. <?= $this->getShorteningHelper()->asphaltConcreteShort($service) ?>

А сам ShorteningHelper получать либо ленивой загрузкой, либо в конструкторе.

 

А что в этом такого? Статику вообще вредно использовать.

 

А в самом хэлпере у вас уже не помойка? Если вы в довольно абстрактном, судя по названию, классе, работаете с довольно конкретными списками. Т.е. это дело подхода - если разработчик позволит себе делать помойку, то она будет в любом случае.

Хотя с данным моментом согласен.

 

я как раз на днях переделывал класс для работы с БД в моем велосипеде))

PHP:

1. <?php
2.  
3. /*
4. * To change this license header, choose License Headers in Project Properties.
5. * To change this template file, choose Tools | Templates
6. * and open the template in the editor.
7. */
8.  
9. namespace natCMF\core;
10.  
11. /**
12. * Description of Db
13. *
14. * @author 27087
15. */
16. class Db {
17.  
18.     /**
19.      * Храним экземпляр текущего класса
20.      * @var object Db
21.      */
22.     static private $_instance = null;
23.  
24.     /**
25.      * Храним объекс работы с БД - имплементированный от интерфейса
26.      * DbInterface
27.      * @var object - implement DbInterface. На данный момент это может быть
28.      * DbMysqli или DbSqlite3
29.      */
30.     private $_db = null;
31.  
32.     /**
33.      * Создаем подключенте БД
34.      * @return object Db - возвращает экземпляр себя хранимый в _instance
35.      */
36.     static public function getInstance() {
37.         if (is_null(self::$_instance)) {
38.             self::$_instance = new self;
39.  
40.             $dbParams = Config::get('dbParams');
41.             if ($dbParams['dbType'] == 'Sqlite3') {
42.                 self::$_instance->_db = new mysql\DbSqlite3($dbParams);
43.             } elseif ($dbParams['dbType'] == 'mysqli') {
44.                 self::$_instance->_db = new mysql\DbMysqli($dbParams);
45.             } else {
46.                 self::$_instance->_db = false;
47.             }
48.         }
49.  
50.         return self::$_instance;
51.     }
52.  
53.     /**
54.      * Запрос к БД
55.      * @param string $query
56.      * @param array $values
57.      * @return object
58.      */
59.     static function query(string $query, array $values = []) {
60.         return self::getInstance()->_db->query($query, $values);
61.     }
62.  
63.     /**
64.      * Возвращает количество строк полученых в результате запроса к БД
65.      * @param type $result
66.      * @return int
67.      */
68.     static function numRows($result) {
69.         return self::getInstance()->_db->numRows($result);
70.     }
71.  
72.     /**
73.      * Очищает объект БД
74.      * @param type $result
75.      * @return boolean
76.      */
77.     static function freeResult(&$result) {
78.         return self::getInstance()->_db->freeResult($result);
79.     }
80.  
81.     /**
82.      * Перебирая объект $result возвращает строки в виде ассоциативного
83.      * массива
84.      * @param type $result
85.      * @return array
86.      */
87.     static function fetchAssoc($result) {
88.         return self::getInstance()->_db->fetchAssoc($result);
89.     }
90.  
91.     /**
92.      * Перебирая объект $result возвращает строки в виде индексированного
93.      * массива
94.      * @param type $result
95.      * @return array
96.      */
97.     static function fetchRow($result) {
98.         return self::getInstance()->_db->fetchRow($result);
99.     }
100.  
101.     /**
102.      * Получаем информацию о модели/таблице
103.      * @param type $table
104.      * @return array
105.      */
106.     static function tableInfo($table) {
107.         return self::getInstance()->_db->tableInfo($table);
108.     }
109.  
110.     /**
111.      * Возвращает количество запросов сделанных к базе
112.      * @return int
113.      */
114.     static function getCountQuery() {
115.         return self::getInstance()->_db->getCountQuery();
116.     }
117.  
118. }

 

@Алекс8
1) self::$_instance не должен быть false
2) почему сразу не вызывать Db::getInstance()->query(), кроме того, что запись короче?
3) почему класс бд (mysql\DbSqlite3 или другой) не вынести в конфиг?

PHP:

1. $dbConfig = Config::get('dbConfig');
2. $dbClass = $dbConfig['class'];
3. $params = $dbConfig['params'];
4. $this->db = new $dbClass($params);

 

1. Согласен.. там вообще планируется добавить исключения)) сейчас читаю про них)) просто у велосипеда из деталей пока только спицы да и то не все))
2. Потому что короче запись)) мне так больше нравится)) плюс мой NetBeans не будет методам подсказки выводить)
3. ХЗ))) может и надо)) это уже у меня 5 или 6 мой велосипед)) я дохожу до определенного момента когда мне велосипед перестется нравится, и я делаю новый))

 

выводить будет.. только мне нравится когда запись короткая))
--- Добавлено ---
я для того что бы короткая запись была добавляют в родителя моих моделей метода типа getRow getRows getCount insert update
что бы модели были более лаконичные когда работаешь с одной таблицей без связей с другими таблицами. .
ну а если связи там у меня уже начинаются джоины и классические записи запросов к мускулу.. только плейсхолдеры немного доработанные))

 

@Алекс8
1) да, именно исключения надо добавлять.
2) переходи на phpstorm. Аннотации у getInstance помогут с подсказками. @return $this работает.
3) отказывайся от getInstance вообще.
во первых - это объект подключения к БД. БД у тебя может быть несколько, при чем вторая появится значительно позже. И ты не сможешь переиспользовать данный класс просто создав новый объект.
во вторых - данный класс не должен ничего знать о таблицах, об их полях и тем более о том, какие там джойны. Для каждой таблицы отдельный класс, где и реализуется логика запросов к БД.
Хранить объект(ы) БД можно в каком-нибудь синглтоне (App::getInstanse())
Например

PHP:

1. App::getInstance()->get('db')->query($params);
2. Yii::$app->db->query($params);
3. Yii::$app->db2->query($params);

Либо в сервис локаторе и пр.

Т.е. в итоге у тебя будет только одна единственная глобальная переменная - App (Yii)
К тому же всё это на много проще тестируется

 

Так такой багофичей обращаются к методу базы из другого класса.

 

а почему это багофича?))

 

Да так субъективно, мне эти прокидывания классами своих инстансов не нужны

 

т.е. вы всякий раз, когда хотите сделать запрос к БД делаете new?

 

Да и в конструкторе создавать подключение а не в конфиге.

 

Так в конструкторе и надо создавать соединение. Но не столько раз, сколько нужно сделать запросов к БД, а один раз за всё время выполнение скрипта.
Но я писал, что лучше не кучу синглтонов хранить, а один - в котором уже хранятся нормальные объекты.