Версия для печати темы Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > PHP: Для профи > наследование и статические методы

Автор: MyDarkSide 9.12.2009, 12:59

Думаю сейчас над реализацией моделей в своем фреймворке MVC. Пришла в голову интересная идея: статические методы модели отвечают за выборку данных из множества.  А обычные члены класса реализуют сам объект модели (т.е. сущность)  Код <?php Class User {   protected static $table = 'users';   protected static $name  = 'User';         public static function getById($id) {     // код который загружает запись из базы и создает экземпляр класса User     // return object типа  User    }      public function isOnline() {     return ( time() - $this->lastActive ) < 3000;   }    } $user = User::getById(1); echo $user->name; echo $user->isOnline() ? "online" : "offline" ;      ?> следующий шаг естественно вынести методы типа getById() в базовый класс Код <?php Abstract Class AbstractModel {   protected static $table = '';   protected static $name  = '';      public static function getById($id) {     // self::$table;  - содержит всегда '' даже если вызов идет через класс User     // get_called_class() есть только в php >= 5.3     // в debug_backtrace() нет даже упоминания о классе User не смотря на то что вызов метода я сделал как-бы через него          // код который загружает запись из базы и создает экземпляр класса User     // return object типа  User (если вызов User::getById() )    }    }  Class User extends AbstractModel {   protected static $table = 'users';   protected static $name  = 'User';      public function isOnline() {     return ( time() - $this->lastActive ) < 3000;   }    } ?> и тут я познал дзен - комменты в AbstractModel::getById() получается как в старом анекдоте: ни денег, ни топора и вроде все правильно.

Автор: awdev 9.12.2009, 14:51

в чем вопрос?

Автор: NewDima 9.12.2009, 15:00

Пространства имен абстрактных членов классов разных уровней наследования не совпадают

Автор: NewDima 9.12.2009, 16:47

я бы сделал как-то так Код abstract class abstractModel {     static protected $__prefix = 'tbl_';     static public function __callStatic($method, $params) {         if (!preg_match('#^get(\w+)By(\w+)$#', $method, $matches) || !count($params)) {             throw new exception('dwad');         }         $tb_name = self::$__prefix . ($name = strtolower($matches[1])) . 's';         $tb_by   = strtolower($matches[2]);         $data    = self::loadData($tb_name, $tb_by, $params[0]);         $class   = $name.'Model';         return new $class($data);     }     static protected function loadData($table, $domain, $where) {         return array($domain => $where);     } } class userModel extends abstractModel {     protected $data = null;     public function __construct($data) {         $this->data = $data;     } } $user = abstractModel::getUserById();

Автор: awdev 9.12.2009, 20:22

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

Автор: NewDima 10.12.2009, 15:09

awdev, комментируя чей-то код, выкладывай аргументы. Голые твои слова стоят ровным счетом ничего. Это на тему ### кода. Холеварить на тему "стоит ли писать самостоятельно велосипеды" не буду, потому что для меня здесь ответ очевиден. MyDarkSide, Зачем создавать тему, в которой потом не отписываешься? Нравится, когда пинают репутацию?

Автор: sTa1kEr 10.12.2009, 15:16

Магический метод __callStatic() тоже только в PHP 5.3 доступен. А вообще это избитый вопрос. Ответ: никак нельзя, только переходить на PHP 5.3

Автор: awdev 10.12.2009, 16:22

Цитата(NewDima @ 9.12.2009,  16:47) я бы сделал как-то так Код abstract class abstractModel {     static protected $__prefix = 'tbl_';     static public function __callStatic($method, $params) {         if (!preg_match('#^get(\w+)By(\w+)$#', $method, $matches) || !count($params)) {             throw new exception('dwad');         }         $tb_name = self::$__prefix . ($name = strtolower($matches[1])) . 's';         $tb_by   = strtolower($matches[2]);         $data    = self::loadData($tb_name, $tb_by, $params[0]);         $class   = $name.'Model';         return new $class($data);     }     static protected function loadData($table, $domain, $where) {         return array($domain => $where);     } } class userModel extends abstractModel {     protected $data = null;     public function __construct($data) {         $this->data = $data;     } } $user = abstractModel::getUserById(); Вся бизнес логика должна быть спрятана в модели. Моделями управляет контроллер, какой когда запустить, собрать данные и отправить в отображение. Отсюда следует что конструктор у моделей может быть разный.  Во вторых этот способ не абстрагируется.  Сегодня мне нужно из БД, завтра считать с XML файла послезавтра вообще передумаю и буду брать через SOAP, в классической постановке мы меняем одну модель на другую в контроллере. Или вместо new News(new XMLClient) написали new News(new SOAPClient) и все пашет.   Все остальное - остается неизменным, ваш же код будет вызывать один и тот же класс. Например мне нужно получить Новость по ID: Будет вызвано: return NewsModel(array("ID"=>10)); Или вообще захочу игнорировать все запросы. Изменить в контролере вызов NewsModel на NewsModelBlackHole - все. А у вас надо будет модель перезаписывать. То есть если я передумаю использовать эту модель, и захочу другую - мне придется перезаписывать файл NewsModel. Что, как мне кажется, не очень красиво и правильно.

Автор: NewDima 10.12.2009, 17:18

awdev, а ты не заметил, что перед кодом я написал "как то так"? Это значит что это только смысл кода, почти псевдокод. Это одно. Другое: я ответил на вопрос топикстартера так, как он был поставлен, остальное - его проблемы. sTa1kEr, а вы пишете для версий ниже?! Добавлено через 4 минуты и 25 секунд Вообще, создается впечатление, что большинство программистов пишут под четверку и даже горядтся этим...

Автор: awdev 10.12.2009, 17:36

NewDima, я не понимаю почему Вы пишите сообщение на "ты"? Во вторых я не хотел доказывать или что-либо еще, я лишь написал видимые для меня недостатки Вашего кода.  Если вы хотели написать это как псевдокод, то это не прочиталось. В любое случае, я надеюсь ответ ТС был дан.

Автор: sTa1kEr 10.12.2009, 18:05

Цитата(NewDima @  10.12.2009,  18:18 ) sTa1kEr, а вы пишете для версий ниже?! Ну вопрос задал не я, а ТС и судя по Цитата(MyDarkSide @  9.12.2009,  13:59 )     // get_called_class() есть только в php >= 5.3 он пишет на версии ниже. Добавлено через 1 минуту и 8 секунд Но в этом нет ничего плохого, т.к. версия 5.3 еще очень сырая.

Автор: MyDarkSide 10.12.2009, 18:12

Цитата(NewDima @  10.12.2009,  15:09 ) MyDarkSide, Зачем создавать тему, в которой потом не отписываешься? Нравится, когда пинают репутацию?  вообще я работаю 8 часовой день, и отвечать ежечасно не могу, только вечером, днем если - только если свободная минута. к тому же обдумывал  свои варианты как извратиться. Пнуть ума большого не надо. Цитата(NewDima @  10.12.2009,  17:18 ) awdev, а ты не заметил, что перед кодом я написал "как то так"? Это значит что это только смысл кода, почти псевдокод. Это одно. Другое: я ответил на вопрос топикстартера так, как он был поставлен, остальное - его проблемы. sTa1kEr, а вы пишете для версий ниже?! Добавлено через 4 минуты и 25 секунд Вообще, создается впечатление, что большинство программистов пишут под четверку и даже горядтся этим...  Господа холиваров не надо, пост серьезный. Принципиально пишу рамку только под пятерку, но затачивать совсем уж под самую последнюю, кажется тоже перебор. Цитата(awdev @  9.12.2009,  14:51 ) в чем вопрос?  Кто виноват и что делать ?  Виноват я (может нарушаю какие то концепции ООП) или ZendCo (не продумали)-на это наталкивает то что в 5.3 появилась функция get_called_class() ? что делать: изначальная идея плохая и допиливать нет смысла или идея хорошая надо допиливать реализацию ? Добавлено через 10 минут и 29 секунд Цитата(awdev @  9.12.2009,  20:22 ) Вот так и создаются ### фрейморки   именно такие рамки и сподвигли написать своё.  последнее время приходится много работать с cakePHP и больше всего бесит что, весь фреймворк типа ОО а данные из моделей возвращаются в виде ассоциативных массивов. И в итоге такой отстой получается:    Код $user = $this->User->getById(5); $this->User->is_online($user['User']['id']); //вместо очевидного $user->isOnline();  вообще в чем основная фишка, из тех рамок что я сомтрел, никто не разделяет по сути два разных класса: класс связанный с каким-то  единственным объектом реальности  (User, Post etc)  -  собственно модель, этот класс моделирует поведение объекта (метод isOnline()  )  ,    и класс-коллекция однородных классов-моделей, который занимается выборкой, сохранением, ассоциированием ит.д.  (метод getById() )

Автор: NewDima 11.12.2009, 07:49

Цитата(MyDarkSide @ 11.12.2009,  01:12) вообще я работаю 8 часовой день, и отвечать ежечасно не могу, только вечером, днем если - только если свободная минута. к тому же обдумывал  свои варианты как извратиться. это при том, что в других ветках ты участвуешь, а в своей нет?

Автор: MyDarkSide 11.12.2009, 10:54

по большому в данном случае версия 5.3 не сильно спасет потому что, использование get_called_class все равно похоже на какой-то костыль.  Т.е. когда речь идет о простых классах и членах, то класс-потомок переопределяет одноименные члены класса-родителя,  другие же наследует т.е. включает в себя. А со статическими членами - поведение почему то другое. И тут возникает вопрос - поведение от Zend'a (когда механизм наследования по сути не работает в статических членах ) - правильное и соответствует каким-то догмам ООП (может быть в других языках такая же ситуация), или это все-таки не доработки

Автор: awdev 11.12.2009, 14:56

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

Автор: MyDarkSide 11.12.2009, 17:40

awdev,  если в базовом классе сделать debug_backtrace() то про класс-потомок там нет ни слова, даже если вызов метода идет из класса потомка.

Автор: awdev 11.12.2009, 18:50

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

Автор: nerezus 13.12.2009, 17:28

Тему не читал, но почти наверняка подойдет фабрика ) Судя по другим прочитанным темам с таким вопросом.

Автор: MyDarkSide 14.12.2009, 10:43

nerezus,  сейчас именно так и сделал. завтра UML прикреплю, а в коде так: есть два класса User extends AbstractModel  и UserCollection extends AbstractCollection (здесь и далее User для примера) есть функция-фабрика Collection($name) которая производит коллекции. А коллекции в результате поиска возвращают один или массив объектов User.  Код function Collection($name) {   return new $name . 'Collection'; // упрощенно } $user = Collection('User')->getById(10); echo  $user->isOnline();

Автор: nerezus 14.12.2009, 12:40

Как сказал один уважаемый человек, static и наследование не совместимы. И при программировании стоит это помнить. Я не знал это и потратил 2 дня на казалось бы "мегакрутую" архитектуру.  Переделал за полчаса потом за действительно норальную, т.к. не мог исправить многие косяки без кучи костылей.

Автор: MyDarkSide 14.12.2009, 14:09

nerezus, именно ! та же х...я самое главное перечитал кучу манов и букварей, и нигде даже намека не было на такую "особенность" статики

Автор: NewDima 14.12.2009, 18:57

MyDarkSide, блин, да это логично вытекает из самой природы статичных методов. А по поводу фабрики, так мой код - тоже фабрика, если не заметили. Какая разница, Код $user = Collection('User')->getById(10); или Код $user = Collection::getUserById(10); Кто скажет? Только не нужно говорить о непроизводительности магических методов и прочей чепухе. А, ну да, мы же пишем ниже 5.3.2, это главная причина

Автор: MyDarkSide 15.12.2009, 11:03

Цитата(NewDima @  14.12.2009,  18:57 ) MyDarkSide, блин, да это логично вытекает из самой природы статичных методов. Вот когда разобъешь об это лоб - то да подумав, впринципе логично, но не мешало бы и хоть словом обмолвиться, про менее важные вещт трубят и разжёвывают, а тут какое то белое пятно, у того же Котерова про 5PHP половина книги посвящена XML, а тут хоть бы строчку...   Цитата(NewDima @  14.12.2009,  18:57 ) 1:      $user = Collection::getUserById(10); разница есть, в каждом классе модели могут быть определены свои специфические методы выборки, например для выборки сегодняшних именниников, в моем коде вызов останется прозрачным Код $user = Collection('User')->getTodayIsBirthday(); в твоем коде Код $user = User::getTodayIsBirthday();

Автор: nerezus 15.12.2009, 11:30

Цитата у того же Котерова про 5PHP половина книги посвящена XML  Да ему просто влом было писать - вот мануал и пересказывает. Так многие авторы делают, тот же Шилдт.

Автор: NewDima 16.12.2009, 07:58

Цитата(MyDarkSide @ 15.12.2009,  18:03) Код $user = Collection('User')->getTodayIsBirthday(); Интересно, что возвращает метод getTodayIsBirthday? булево значение (судя по названию)? Что за пример ты привел? Бред какой-то Мой код будет примерно таким: Код $user = Collection::getUserByBirthday($data) Еще раз говорю, всмотрись внимательно, между этим Код $user = Collection('User')->getById(10); и этим Код $user = Collection::getUserById(10); разница минимальна, можно сказать, что только в записи. Первый способ: Collection('User') возвращает объект типа User, метод getById вызывается в контексте его же. Второй способ: Все намного прозрачнее

Автор: MyDarkSide 17.12.2009, 01:31

Цитата(NewDima @  16.12.2009,  07:58 ) Интересно, что возвращает метод getTodayIsBirthday? булево значение (судя по названию)? Что за пример ты привел? Бред какой-то  прежде чем хамить надо внимательно прочитать пост:    Цитата(MyDarkSide @  15.12.2009,  11:03 ) например для выборки сегодняшних именниников во-вторых это тема НЕ про названия методов, каждый волен называть как ему нравится. Цитата(NewDima @  16.12.2009,  07:58 ) разница минимальна, можно сказать, что только в записи. Цитата(NewDima @  16.12.2009,  07:58 ) Мой код будет примерно таким: $user = Collection::getUserByBirthday($data) т.е. у тебя метод getUserByBirthday() - будет определен в классе Collection ? и ещё 100 других методов  типа: выбрать не прочтенные комментарии, посты за сегодня, не прочтенные ПМ, кто пинал мою карму и т.д. ? как-то не очень..

Автор: NewDima 17.12.2009, 11:43

Цитата(MyDarkSide @ 17.12.2009,  08:31) т.е. у тебя метод getUserByBirthday() - будет определен в классе Collection ? и ещё 100 других методов  типа: выбрать не прочтенные комментарии, посты за сегодня, не прочтенные ПМ, кто пинал мою карму и т.д. ? как-то не очень.. явное непонимание азов... читать азы http://php.net/manual/en/language.oop5.overloading.php. Если ты бы хоть взглянул на код, который я дал, то не ответил так, как ты это сделал. Присмотрись внимательней, можешь открыть экранную клавиатуру, если тебе без нее проблема прочитать код. В общем сделай что-нибудь, прежде чем говорить про код, который ты не прочитал. В нем даже НЕТ метода getUserById, представляешь? а он будет вызываться, вот так чудо!

Автор: MyDarkSide 18.12.2009, 12:10

Цитата(NewDima @  17.12.2009,  11:43 ) явное непонимание азов... читать азы здесь. судя по твоему коду азам могут поучить тебя даже первокурсники. Аргументирую, твой код: Код abstract class abstractModel {     static protected $__prefix = 'tbl_';     static public function __callStatic($method, $params) {         if (!preg_match('#^get(\w+)By(\w+)$#', $method, $matches) || !count($params)) {             throw new exception('dwad');         }         $tb_name = self::$__prefix . ($name = strtolower($matches[1])) . 's';         $tb_by   = strtolower($matches[2]);         $data    = self::loadData($tb_name, $tb_by, $params[0]);         $class   = $name.'Model';         return new $class($data);     }     static protected function loadData($table, $domain, $where) {         return array($domain => $where);     } } class userModel extends abstractModel {     protected $data = null;     public function __construct($data) {         $this->data = $data;     } } $user = abstractModel::getUserById(); 1. Регулярка в методе, которая определяет какой метод "перевызвать",  необходимость её применения надо обосновать. 2. Вы Очень не внимательно читаете топик-старты:     2.1 Речь идет о PHP5, но более ранем чем 5.3, соответственно функции там __callStatic() - там нет, иначе бы и мой код можно было заставить работать более прозрачным способом.     2.2 Речь идет о нормальном механизме наследования, в твоем коде, вы решаете совсем другую задачу,а именно сделать так, чтобы вызов метода типа МодельByПоле - выбирало запись из таблицы по определенному полю и всё. А что вы будете делать если надо будет добавить пагинаторы сюда, сортировки, выборки из 2 или более связанных таблиц - лепить новые регулярки ? а если выборка вообще не с полем связана а например, нужно получить время последней модификации таблицы модели, как выкручиваться будете ? наверно опять регуляркой. 3. Как работает нормальное наследование: есть базовый класс, часто объявленный как абстрактный, содержащий общее поведение для объектов своего класса,  и есть его потомки, конкретизирующие поведение для своего подкласса, создается экземпляр потомка и вызов происходит методов потомка, который имеет доступ к общим методом объявленным в базовом классе, ты предлагаешь вызывать метод базового класса, который делегирует его своему потомку - наследование здесь вообще не при чем, в классе userModel можно вообще убрать "extends abstractModel".  это по вашему "прозрачном" коду.  Посмотрел ваш профиль  Цитата Увлечения  Вэбкод          "Вэбкод" - это не увлечение (для меня по-крайней мере), это ежедневная работа по 8 часов, в т.ч. и с заказчиками которые хотят получить, то что они хотят, а не ссылки на умные статьи. Если вы где-то работаете, я бы на месте начальства по внимательнее пригляделся к вам, если не работаете - не надо строить из себя супер-гуру, которым вы не являетесь, хамом - да, гуру - нет, это разные вещи.

Автор: Simpliest 18.12.2009, 15:02

Безотносительно ко всему обсуждавшемуся в топике, а почему именно метод должен быть статическим? И работает такое даже в PHP<5.3 Код <?php abstract class ParentClass {     public function method1() {         $this->method2();     }     public function method2() {         echo 'Hey, i\'m Parent';     } } class Child extends ParentClass {     public function method2() {         echo 'Hey, i\'m Child';     } } $c = new Child(); $c->method1();

Автор: NewDima 18.12.2009, 15:08

Цитата(MyDarkSide @ 18.12.2009,  19:10) в твоем коде, вы решаете совсем другую задачу,а именно сделать так, чтобы вызов метода типа МодельByПоле - выбирало запись из таблицы по определенному полю и всё.  Ну это уже твое восприятие кода, я тебе говорил только про саму идею. Реализовывай как хочешь. Добавлено через 3 минуты и 31 секунду Цитата(MyDarkSide @ 18.12.2009,  19:10) "Вэбкод" - это не увлечение (для меня по-крайней мере), это ежедневная работа по 8 часов, в т.ч. и с заказчиками которые хотят получить, то что они хотят, а не ссылки на умные статьи. А ты не смотрел, когда я поставил такой текст? Время идет. И у меня тоже есть заказчики, которые довольны результатами моей работы. И я работаю на СЕБЯ. Больше не переходи на личности Добавлено через 4 минуты и 26 секунд О, кстати, я действительно хам, можешь отметить это в карме

Автор: MyDarkSide 18.12.2009, 16:15

Цитата(NewDima @  18.12.2009,  15:08 ) О, кстати, я действительно хам, можешь отметить это в карме  я смотрел твой кармалог - это не только мое мнение. По сколько тема перешла во флуд предлагаю её закрыть.

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)