title
Description
Body
Перевел свои функции для работы с бд в стиль опп.
Прошу оценить 'говнястость и логичность кода', и дать дельные советы.
---
НоBый адрес файлов http://xwab.ru/forum/files/swomclass_626.txt и http://xwab.ru/forum/files/swomclass_973.zip
Отредактировано Morgan (2010.02.23 22:07)
Неактивен
Morgan написал:
// создание обьекта (аргументы аналогичны функции mysql_connect())
у mysql_connect больше параметров, поэтому аргументы не совсем аналогичны.
Не совсем понятно зачем переменная $con
|
|
mysql_free_result никогда не выполнится
if(is_integer($msg)) - лучше is_numeric
Могу еще придраться к форматированию кода, и еще в подобных прослойках желательно делать счетчик запросов к БД. Т.е. после успешного mysql_query инкрементить статическую переменную.
В целом, если учитывать что в теме упоминается Учу ООП, результаты, на мой взгляд отличные)
P.S. и еще попробуй использовать конструкцию try throw catch
Неактивен
В $con записываются все открытые соединения.
Почему, фри результ не выполнится?
try {...}
Перехват ошибок это?
Читал про это в пхп5 котерова, честно - мало что понял.
Отредактировано Morgan (2010.02.22 22:29)
Неактивен
Также , не могу понять причину использования расширении классов, если проще будет редактировать исходный класс.
В книжке, был пример - есть класс, который должен вырисовать на плоскости фигуру (не определенно какую), потом создается расширение исходного, для вырисовки фигуры - квадрата.
Это довольно хороший пример, но вот только больше я не нашел где можно применить эту систему.
Также не понял интерфейсы.
Неактивен
Интерфейс - каркас для классов. Простой набор свойств(вроде) и методов, которые обязательно должны быть созданы в норм классе, иначе ошибка вылезит. Клас может быть составлен из неск. интерфейсов.
Отредактировано Fuelen (2010.02.22 23:03)
Неактивен
когда пишется большое приложение, в разработке которого принимают участие куча народу, интерфейсы, абстрактные классы и прочие прелести ООП очень помогают структурировать и систематизировать код.
Morgan написал:
В $con записываются все открытые соединения.
но более эта переменная никак не ипользуется, и объявлена как приватная.
Неактивен