#1 2010.02.22 15:11

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Учу ООП. Результаты

Перевел свои функции для работы с бд в стиль опп.
Прошу оценить 'говнястость и логичность кода', и дать дельные советы.
---

НоBый адрес файлов http://xwab.ru/forum/files/swomclass_626.txt и http://xwab.ru/forum/files/swomclass_973.zip

Отредактировано Morgan (2010.02.23 22:07)


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#2 2010.02.22 18:19

Fuelen
Участник
Зарегистрирован: 2009.06.22
Сообщений: 229
Карма: 5
Профиль

Re: Учу ООП. Результаты

И здесь ты big_smile


на хую вас вертів

Неактивен

#3 2010.02.22 20:34

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Re: Учу ООП. Результаты

Fuelen написал:

И здесь ты big_smile

я везде, и нигде :-D


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#4 2010.02.22 21:41

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6593
Карма: 107
Профиль Веб-сайт

Re: Учу ООП. Результаты

Morgan написал:

// создание обьекта (аргументы аналогичны функции mysql_connect())

у mysql_connect больше параметров, поэтому аргументы не совсем аналогичны.
Не совсем понятно зачем переменная $con

Код:

1
2
return $result;
mysql_free_result($result);

mysql_free_result никогда не выполнится
if(is_integer($msg)) - лучше is_numeric
Могу еще придраться к форматированию кода, и еще в подобных прослойках желательно делать счетчик запросов к БД. Т.е. после успешного mysql_query инкрементить статическую переменную.
В целом, если учитывать что в теме упоминается Учу ООП, результаты, на мой взгляд отличные)
P.S. и еще попробуй использовать конструкцию try throw catch

Неактивен

#5 2010.02.22 22:28

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Re: Учу ООП. Результаты

В $con записываются все открытые соединения.
Почему, фри результ не выполнится?
try {...}
Перехват ошибок это?
Читал про это в пхп5 котерова, честно - мало что понял.

Отредактировано Morgan (2010.02.22 22:29)


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#6 2010.02.22 22:33

Fuelen
Участник
Зарегистрирован: 2009.06.22
Сообщений: 229
Карма: 5
Профиль

Re: Учу ООП. Результаты

После return код дальше в функции/методе не выполняется


на хую вас вертів

Неактивен

#7 2010.02.22 22:36

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Re: Учу ООП. Результаты

%)
забыл, надо мозг менять


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#8 2010.02.22 22:48

Morgan
Участник
Зарегистрирован: 2009.03.31
Сообщений: 333
Карма: 10
Профиль Веб-сайт

Re: Учу ООП. Результаты

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

Также не понял интерфейсы.
sad


Натан ротшильд :
Кто владеет информацией, тот владеет миром

Неактивен

#9 2010.02.22 23:01

Fuelen
Участник
Зарегистрирован: 2009.06.22
Сообщений: 229
Карма: 5
Профиль

Re: Учу ООП. Результаты

Интерфейс - каркас для классов. Простой набор свойств(вроде) и методов, которые обязательно должны быть созданы в норм классе, иначе ошибка вылезит. Клас может быть составлен из неск. интерфейсов.

Отредактировано Fuelen (2010.02.22 23:03)


на хую вас вертів

Неактивен

#10 2010.02.22 23:12

Gemorroj
Administrator
Откуда: Белоруссия
Зарегистрирован: 2007.11.03
Сообщений: 6593
Карма: 107
Профиль Веб-сайт

Re: Учу ООП. Результаты

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

Morgan написал:

В $con записываются все открытые соединения.

но более эта переменная никак не ипользуется, и объявлена как приватная.

Неактивен

Дополнительно

forum.wapinet.ru

PunBB Mod v0.6.2
0.011 s