Не срабатывает метод __callStatic

abler98 · 22 сен 2014

Почему в не срабатывает метод __callStatic?

Код (PHP):

<?php class DB { private static $_instance; private static $_pdo; private static $_statement = null; public function __construct() { $db_host = Config::get('database.host'); $db_name = Config::get('database.name'); $db_user = Config::get('database.user'); $db_pass = Config::get('database.pass'); try { self::$_pdo = new PDO('mysql:host='.$db_host.';dbname='.$db_name, $db_user, $db_pass); self::$_pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ); self::$_pdo->query('SET NAMES `utf8` COLLATE `utf8_unicode_ci`'); } catch (PDOException $e) {} } public function __clone() {} public static function __callStatic($method, $arguments) { if (!self::$_instance) self::$_instance = new DB; call_user_func_array(array(self::$_instance, $method), $arguments); } public static function quote($str) { return self::$_pdo->quote($str); } public static function bindParams($query, $params = array()) { if (is_array($params) && $count = count($params)) { for ($i = 0; $i < $count; ++$i) { $pos = strpos($query, '?'); $param = self::quote($params[$i]); $query = substr_replace($query, $param, $pos, 1); } } return $query; } public static function query($query) { return self::$_pdo->query($query); } public static function select($query, $params = array()) { self::$_statement = self::bindParams($query, $params); return self::$_instance; } public function get() { return self::query(self::$_statement)->fetch(); } }

mkramer · 22 сен 2014

Он сработает только при вызове несуществующего статического метода. И что-то непонятное внутри нарисовано

abler98 · 22 сен 2014

mkramer сказал(а):

Он сработает только при вызове несуществующего статического метода.
Нажмите, чтобы раскрыть...

Значит нужно вручную везде вписывать?

mkramer сказал(а):

И что-то непонятное внутри нарисовано
Нажмите, чтобы раскрыть...

Это Вы про что?

mkramer · 22 сен 2014

abler98 сказал(а):

Это Вы про что?
Нажмите, чтобы раскрыть...

ПРо

abler98 сказал(а):

Код (Text):

if (!self::$_instance) self::$_instance = new DB;

call_user_func_array(array(self::$_instance, $method), $arguments);

Нажмите, чтобы раскрыть...

Если метод статический, ему не нужен экземпляр

Добавлено спустя 1 минуту 27 секунд:
Синглтоны не так делаются. Закрывается конструктор, и делается статический метод типа instance(), который создаёт (если ещё не создано) и возвращает экземпляр класса. А все остальные методы делаются обычными - не статическими. Статические функции - это такие, которым не нужен экземпляр класса для работы, грубо говоря.

abler98 · 22 сен 2014

mkramer сказал(а):

Синглтоны не так делаются. Закрывается конструктор, и делается статический метод типа instance(), который создаёт (если ещё не создано) и возвращает экземпляр класса. А все остальные методы делаются обычными - не статическими. Статические функции - это такие, которым не нужен экземпляр класса для работы, грубо говоря.
Нажмите, чтобы раскрыть...

Я хочу сделать вроде этого:

Код (PHP):

$user = DB::select('SELECT * FROM `users` WHERE `id` = ? LIMIT 1', array(1))->get(); $user = DB::table('users')->where(array('id' => 1))->get();

Ну и так далее)

Вот доработал немного:

Код (PHP):

<?php class DB {     private static $_instance;     private static $_pdo;     private static $_statement = null;          private static $where_count = 0;          public function __construct() {         $db_host = Config::get('database.host');         $db_name = Config::get('database.name');         $db_user = Config::get('database.user');         $db_pass = Config::get('database.pass');              try {             self::$_pdo = new PDO('mysql:host='.$db_host.';dbname='.$db_name, $db_user, $db_pass);             self::$_pdo->setAttribute(PDO::ATTR_DEFAULT_FETCH_MODE, PDO::FETCH_OBJ);             self::$_pdo->query('SET NAMES `utf8` COLLATE `utf8_unicode_ci`');         } catch (PDOException $e) {}     }          public function __clone() {}          public static function quote($str) {         if (!self::$_instance) self::$_instance = new DB;         return self::$_pdo->quote($str);     }          public static function bindParams($query, $params = array()) {         if (!self::$_instance) self::$_instance = new DB;         if (is_array($params) && $count = count($params)) {             for ($i = 0; $i < $count; ++$i) {                 $pos = strpos($query, '?');                 $param = self::quote($params[$i]);                 $query = substr_replace($query, $param, $pos, 1);             }         }         return $query;     }          public static function query($query) {         if (!self::$_instance) self::$_instance = new DB;         return self::$_pdo->query($query);     }          public static function select($query, $params = array()) {         if (!self::$_instance) self::$_instance = new DB;         self::$_statement = self::bindParams($query, $params);         return self::$_instance;     }          public static function table($table) {         if (!self::$_instance) self::$_instance = new DB;         self::$_statement = 'SELECT * FROM `'.$table.'`';         return self::$_instance;     }          public function where($params = array()) {         if (self::$where_count == 0) self::$_statement .= ' WHERE ';         foreach ($params as $name => $value) {             self::$_statement .= (self::$where_count > 0 ? ' AND ' : null).'`'.$name.'` = '.self::quote($value);             self::$where_count++; }         return self::$_instance;     }          public function get() {         return self::query(self::$_statement)->fetch();     } }

mkramer · 22 сен 2014

Это порнография, поскольку кода дублируется дофига. Зачем вам статические методы? А если статические, зачем создавать экземпляр? Сделали бы статический экземпляр PDO и статическую функцию по его созданию. Но я считаю, что лучше нормальный экземпляр класса сделать. Вот синглтон базы данных схематично:

Код (PHP):

class DB {

   private $pdo;

   static private $inst = null;

   private function __construct() {

       $this->pdo = new PDO(/* параметры */);

   }



   static public function instance() {

      if (self::$instance === null)

          self::$instance = new self();

      return self::$instance;

   }



   public function query() {

   }



   /* И всё остальное */

}

$db = DB::instance();

$db->query("drop table users");

igordata · 22 сен 2014

а если засунуть проверку в каждый метод, то будет ещё проще =)

abler98 · 22 сен 2014

mkramer сказал(а):

Это порнография, поскольку кода дублируется дофига. Зачем вам статические методы? А если статические, зачем создавать экземпляр? Сделали бы статический экземпляр PDO и статическую функцию по его созданию. Но я считаю, что лучше нормальный экземпляр класса сделать. Вот синглтон базы данных схематично:
Нажмите, чтобы раскрыть...

Мне это не подходит)

mkramer · 22 сен 2014

А зачем тогда классы, если не создавать экземпляры? Ну фиг с вами. Тогда не нужно делать конструктор и создавать экземпляр - а то вы ничего не выигрываете вообще от своих статических методов (обычно так пишут те, кому жалко пару долей миллисекунды на создание экземпляра класса. Ну ещё аргумент - типа экземпляр много памяти занимает, тоже мне не очень верится)

igordata сказал(а):

а если засунуть проверку в каждый метод, то будет ещё проще =)
Нажмите, чтобы раскрыть...

Кому проще? И что проверять в каждом методе? Что-то такое имеете в виду?

Код (Text):

class DB {

static private $pdo = null;

static private funciton init() {

if (self::$pdo === null)

self::$pdo = new PDO(/* параметры */);

}

static public function query($q) {

self::init();

return self::$pdo->query($q);

}

}

А смысл в таком классе?

igordata · 22 сен 2014

Да.

mkramer сказал(а):

И что проверять в каждом методе?
Нажмите, чтобы раскрыть...

Думаешь это такие жестокие накладные расходы? Спорить не хочу =) мысль высказал. Кому привычно сначала делать ссылку на экземпляр - его дело.

mkramer · 22 сен 2014

ОК, не будем спорить. Я и создание объекта не считаю огромными накладными расходами. ТС, если хотите класс БД полностью на статических методах, возьмите мой последний пример за основу - чтоб не дублировать код создания объекта PDO в каждом классе - не красиво, и не создавать не нужный вам экземпляр класса.

igordata · 22 сен 2014

mkramer сказал(а):

Я и создание объекта не считаю огромными накладными расходами.
Нажмите, чтобы раскрыть...

объект-то единожды создаётся. Тут вопрос в том, что если ты строго через метод получаешь на него ссылку, то это удобно там, где методы стоит вызывать из объекта напрямую. В случае, если у тебя методы синглтона заранее известны и ограничены, и ты проверку и создание пихаешь в каждый метод, то объект тоже создаётся только один раз, но можно сразу дёргать какие-то методы синглтона, а если надо обратиться к объекту напрямую, то можно только тогда заюзать метод со ссылкой. Экономия. =)

runcore · 23 сен 2014

ТС явно хочет сделать синглтон.
по сути, синглтон - обычный класс, нестатический. но - конструктор заблокирован, клонирование заблокировано. экземпляр класса получаем через всего один статический метод.
если ТСу хочется краткости, то можно сделать короткую функцию(внутри тот же синглтон получать) алиас, и обращаться к БД так:
db()->select(...);
сразу, из любого места.

igordata · 23 сен 2014

runcore сказал(а):

по сути, синглтон - обычный класс, нестатический. но - конструктор заблокирован
Нажмите, чтобы раскрыть...

Это очень странная формулировка. Синглтон это класс, которы либо сам, либо оборачивает экземпляр некоего класса, который существует в единственном экземпляре. В вики так же написано "с глобальной точкой доступа". Мне кажется, что раз он в одном экземпляре, то статиковая оболочка лучше всего решает этот вопрос. Эстественно, что если это "экземпляр", то это заведомо не статическая штука. Но статика формирует как раз интерфейс. В PHP статика бесплатно глобальна.

Fell-x27 · 23 сен 2014

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

Для БД обертки, имхо, юзайте статику.

igordata · 23 сен 2014

брр. кто-то путает цель и метод.

runcore · 23 сен 2014

igordata сказал(а):

Это очень странная формулировка
Нажмите, чтобы раскрыть...

ничего странного. в той же вики приведен пример кода на пхп. реализующий синглтон. посмотри.
Это обычный класс. Из статики там только переменная - хранящая ЭКЗЕМПЛЯР класса и метод getInstance через который этот экземпляр можно получить. всё! все остальные свойства и методы нестатические и реализованы для их использования в контексте созданного объекта.

никакая "статиковаЯ" оболочка не требуется. класс сам реализует соответсвующий метод.

igordata · 23 сен 2014

Но это не аксиома. Я ж говорю, автоинит на мой взгляд удобнее в случае с заранее известными методами. Неужели никогда не замечали ленивую инициализацию?

runcore · 24 сен 2014

ну вот в getInstance и будет инициализация объекта. при первом же обращении. тоесть тогда когда уже нужны данные.
или ты про боле ленивую?) тогда поясни

igordata · 24 сен 2014

Ну ты проверку и вызов создания пихаешь в каждый метод как тут viewtopic.php?p=397218#p397218 и можешь без получения ссылки на объект сразу юзать методы статик-обёртки. Экономится целая строка!

runcore · 24 сен 2014

igordata сказал(а):

можешь без получения ссылки на объект сразу юзать методы
Нажмите, чтобы раскрыть...

но в методе init() то у тебя все равно есть рождение объекта? так?
а раз ВСЕ РАВНО объект создается, в чем профит от статики в данном случае? никакого.
наоборот. весь класс пронизан копипастом. в каждом методе вызов init(). и главное зачем? когда можно сделать классическую конструкцию(синглтон). которая будет проще в реализации и делать точно тоже самое по затратам на создание объекта.

igordata сказал(а):

Экономится целая строка!
Нажмите, чтобы раскрыть...

какая строка экономится?
всё делается и так одной строкой.
$rows = Db::getInstance()->getRows();

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

igordata · 24 сен 2014

Ну слово одно. Не вижу смысла печатать каждый раз одно и то же, если можно на автомате. Чего ты так боишься?

runcore · 24 сен 2014

igordata сказал(а):

Ну слово одно. Не вижу смысла печатать каждый раз одно и то же, если можно на автомате. Чего ты так боишься?
Нажмите, чтобы раскрыть...

экономия должна быть экономной. но всегда есть предел.
так рассуждая можно дойти до еще более короткого варианта:
$data = getRows(...);
а там внутри пусть всё втысячный раз описать, и проверки и инициализацию объекта...

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

igordata · 24 сен 2014

ну в любом направлении надо знать меру, конечно.

Не срабатывает метод __callStatic

abler98 Активный пользователь

mkramer Суперстар
Команда форума Модератор

abler98 Активный пользователь

mkramer Суперстар
Команда форума Модератор

abler98 Активный пользователь

mkramer Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

abler98 Активный пользователь

mkramer Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

mkramer Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

runcore Старожил

igordata Суперстар
Команда форума Модератор

Fell-x27 Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

runcore Старожил

igordata Суперстар
Команда форума Модератор

runcore Старожил

igordata Суперстар
Команда форума Модератор

runcore Старожил

igordata Суперстар
Команда форума Модератор

runcore Старожил

igordata Суперстар
Команда форума Модератор

Быстрый поиск

Не срабатывает метод __callStatic

abler98 Активный пользователь

mkramer Суперстар Команда форума Модератор

abler98 Активный пользователь

mkramer Суперстар Команда форума Модератор

abler98 Активный пользователь

mkramer Суперстар Команда форума Модератор

igordata Суперстар Команда форума Модератор

abler98 Активный пользователь

mkramer Суперстар Команда форума Модератор

igordata Суперстар Команда форума Модератор

mkramer Суперстар Команда форума Модератор

igordata Суперстар Команда форума Модератор

runcore Старожил

igordata Суперстар Команда форума Модератор

Fell-x27 Суперстар Команда форума Модератор

igordata Суперстар Команда форума Модератор

runcore Старожил

igordata Суперстар Команда форума Модератор

runcore Старожил

igordata Суперстар Команда форума Модератор

runcore Старожил

igordata Суперстар Команда форума Модератор

runcore Старожил

igordata Суперстар Команда форума Модератор

mkramer Суперстар
Команда форума Модератор

mkramer Суперстар
Команда форума Модератор

mkramer Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

mkramer Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

mkramer Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

Fell-x27 Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор

igordata Суперстар
Команда форума Модератор