PHP.SU

Программирование на PHP, MySQL и другие веб-технологии

PHP.SU Портал

Главная

Помощь

Поиск

Поиск Яндекс Вакансии

Пользователи

Здравствуйте, Гость

( Вход · Регистрация · Правила форума )

Забыли пароль?

тестовый класс ООП

Форумы портала PHP.SU » PHP » Программирование на PHP » Объектно-ориентированное программирование (Модераторы: OrmaJever, Саныч)

Страниц (2): [1] 2 »

Без описания

Поиск в теме | Версия для печати

and_07	Отправлено: 20 Сентября, 2012 - 20:40:29
Гость Покинул форум Сообщений всего: 91 Дата рег-ции: Сент. 2012 Помог: 3 раз(а)	Прошу указать фрагменты которые можно улучшить в данном коде PHP: скопировать код в буфер обмена <?PHP define('DBHOST','localhost'); define('DBUSER',''); define('DBPASS',''); define('DBNAME',''); ?> <?PHP interface BaseData { function query($sql); function insert( $table, $data ); function update( $table, $changes, $condition ); function delete( $table, $condition, $limit ); } ?> <?PHP class DataClass implements BaseData { private static $_instance; private $_connection; private $results; public function __construct() { $this->_connection = mysql_connect(DBHOST, DBUSER, DBPASS) or die(mysql_error()); mysql_select_db(DBNAME) or die(mysql_error()); } public function __destruct() { @mysql_close($this->_connection); } public function delete( $table, $condition, $limit) { $limit = ( $limit == '' ) ? '' : ' LIMIT ' . $limit; $delete = "DELETE FROM {$table} WHERE {$condition} {$limit}"; $this->query( $delete ); } public function update( $table, $changes, $condition ) { $update = "UPDATE " . $table . " SET "; foreach( $changes as $field => $value ) { $update .= "`" . $field . "`='{$value}',"; } $update = substr($update, 0, -1); if( $condition != '' ) { $update .= "WHERE " . $condition; } $this->query( $update ); return true; } public function insert( $table, $data ) { $fields = ""; $values = ""; foreach ($data as $f => $v) { $fields .= "`$f`,"; $values .= ( is_numeric( $v ) && ( intval( $v ) == $v ) ) ? $v."," : "'$v',"; } $fields = substr($fields, 0, -1); $values = substr($values, 0, -1); $insert = "INSERT INTO $table ({$fields}) VALUES({$values})"; $this->query( $insert ); return true; } public function query( $queryStr ) { if ($this->_connection === false) { die('No Database Connection Found.'); } $result = @mysql_query($queryStr,$this->_connection); if ($result === false) { die(mysql_error()); } return $result; } public function fetch($sql) { $result = $this->query($sql); $ret = array(); while ($row = mysql_fetch_assoc($result)) { $ret []= $row; } return ($ret); } } ?> <?PHP class GuestBook { private $db; private $table = "guestbook"; public function __construct( BaseData $db) { $this->db = $db; } public function checkunique($email) { $this->db->query("SELECT * FROM `".$this->table."` WHERE email='$email'"); } public function add($data) { $this->db->insert( $this->table, $data ); } public function delete($condition, $limit) { $this->db->delete( $this->table, $condition, $limit ); } public function update( $changes, $condition ) { $this->db->update($this->table, $changes, $condition ); } public function query($sql) { $this->db->query($sql); } public function create() { $sql =" CREATE TABLE IF NOT EXISTS `".$this->table."` ( `id` int(10) unsigned NOT NULL auto_increment, `parent_id` int(10) , `name` varchar(255) default '', `email` varchar(255) default '', `description` varchar(255) default '', `when` int(11) NOT NULL default '0', `ip` varchar(20) default NULL, PRIMARY KEY (`id`) ) ENGINE=MyISAM DEFAULT CHARSET=utf8"; $this->db->query($sql); } public function fetch($sql) { return $this->db->fetch($sql); } public function get_ip() { $ip = "0.0.0.0"; if( ( isset( $_SERVER['HTTP_X_FORWARDED_FOR'] ) ) && ( !empty( $_SERVER['HTTP_X_FORWARDED_FOR'] ) ) ) { $ip = $_SERVER['HTTP_X_FORWARDED_FOR']; } elseif( ( isset( $_SERVER['HTTP_CLIENT_IP'])) && (!empty($_SERVER['HTTP_CLIENT_IP'] ) ) ) { $ip = explode(".",$_SERVER['HTTP_CLIENT_IP']); $ip = $ip[3].".".$ip[2].".".$ip[1].".".$ip[0]; } elseif((!isset( $_SERVER['HTTP_X_FORWARDED_FOR'])) \|\| (empty($_SERVER['HTTP_X_FORWARDED_FOR']))) { if ((!isset( $_SERVER['HTTP_CLIENT_IP'])) && (empty($_SERVER['HTTP_CLIENT_IP']))) { $ip = $_SERVER['REMOTE_ADDR']; } } return $ip; } } ?> <?PHP $GB = new GuestBook( new DataClass()) ; //$GB->create(); /* $arr=array(); $arr["name"] = 'test'; $arr["email"] = 'test@te.te'; $arr["description"]= 'bla bla bla'; $arr["when"] =123; $arr["ip"] ='127.0.1'; $arr1=array(); $arr1["name"] = 'test2'; $arr1["email"] = 'test@te.te'; $arr1["description"]= 'bla bla bla'; $arr1["when"] =2; $arr1["ip"] = $GB->get_ip(); $GB->add( $arr); $GB->update( $arr1,'id=4'); $GB->delete('name ="test" ' ,2);/ $sql ="select from guestbook "; $res =$GB->fetch($sql) ; $ip = $GB->get_ip(); ?>

Мелкий	Отправлено: 20 Сентября, 2012 - 21:28:47
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	0) Глобальные константы 1) использование mysql_*, вместо mysqli/PDO 2) неиспользуемые свойства, как минимум $_instance 3) безальтернативные die 4) нет защиты от инъекций даже там, где она легко была бы реализована ----- PostgreSQL DBA

OrmaJever	Отправлено: 20 Сентября, 2012 - 21:32:53
Активный участник Покинул форум Сообщений всего: 7540 Дата рег-ции: Янв. 2010 Откуда: Чернигов Помог: 299 раз(а)	5) частое использование "@", что очень плохо. 6) некоторые циклы можно заменить на join() 7) для работы с базой даных можно и даже нужно использовать singleton. ----- Если вы хотя бы 3-4 раза не решите всё выкинуть и начать заново - вы явно что-то делаете не так.

Bio man	Отправлено: 20 Сентября, 2012 - 21:39:02
Постоянный участник Покинул форум Сообщений всего: 2751 Дата рег-ции: Июль 2010 Откуда: Даугавпилс, Латвия Помог: 52 раз(а)	8) Нет обработки ошибок в случае неудачного запроса. Как исправишь первые 7 пунктов, пиши, напишем еще 5 пунктов а то и больше. Для учебного проекта 8 пункт не столь важен, но лучше привыкать сразу. (Отредактировано автором: 20 Сентября, 2012 - 21:39:25)

armancho7777777	Отправлено: 20 Сентября, 2012 - 22:07:08
Активный участник Покинул форум Сообщений всего: 4526 Дата рег-ции: Февр. 2011 Откуда: Москва Помог: 221 раз(а)	Мелкий, можно прокомментировать: Мелкий пишет: безальтернативные die Не совсем понял что Вы имели в виду

DlTA	Отправлено: 20 Сентября, 2012 - 22:21:48
Постоянный участник Покинул форум Сообщений всего: 2952 Дата рег-ции: Окт. 2010 Помог: 53 раз(а)	armancho7777777 пишет: Не совсем понял что Вы имели в виду and_07 пишет: mysql_select_db(DBNAME) or die(mysql_error()); если не подключились, то отвалилось ВСЕ. (Добавление) кстати тоже очень плохой учатсок, или справильней сказать очень хороший для злоумышленников, им сразу видно что и куда идет, ведь ошибка то возвращается в браузер

Мелкий	Отправлено: 20 Сентября, 2012 - 22:29:47
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	armancho7777777 пишет: можно прокомментировать: DlTA верно пояснил мою мысль. ----- PostgreSQL DBA

armancho7777777	Отправлено: 20 Сентября, 2012 - 22:51:13
Активный участник Покинул форум Сообщений всего: 4526 Дата рег-ции: Февр. 2011 Откуда: Москва Помог: 221 раз(а)	Мелкий пишет: DlTA верно пояснил мою мысль. А. Я думал, Вы что - то ещё имели в виду ) (Отредактировано автором: 21 Сентября, 2012 - 04:09:52)

digi	Отправлено: 21 Сентября, 2012 - 09:01:52
Посетитель Покинул форум Сообщений всего: 406 Дата рег-ции: Янв. 2012 Помог: 4 раз(а)	OrmaJever пишет: 7) для работы с базой даных можно и даже нужно использовать singleton. factory - т.к. коннектов может быть много.

snikers987	Отправлено: 21 Сентября, 2012 - 09:07:16
Участник Покинул форум Сообщений всего: 1239 Дата рег-ции: Сент. 2011 Откуда: Крым Помог: 25 раз(а)	digi пишет: OrmaJever пишет: 7) для работы с базой даных можно и даже нужно использовать singleton. factory - т.к. коннектов может быть много. Ну и зачем плодить подключения к бд? ----- Когда всматриваешься в тёмную бездну, учти, что кто-то может смотреть на тебя из неё...

armancho7777777	Отправлено: 21 Сентября, 2012 - 09:35:07
Активный участник Покинул форум Сообщений всего: 4526 Дата рег-ции: Февр. 2011 Откуда: Москва Помог: 221 раз(а)	digi пишет: коннектов может быть много. Только в случае к разным БД. Но в этом случае переменная getInstance может быть массивом, где ключи - названия БД. Получится что-то типа этого: PHP: скопировать код в буфер обмена class DB { private static $_getInstance = array(); // Данные для коннектов баз данных private $_data_connects = array( 'default_db_name' => array( 'host' => '', 'user' => '', 'pass' => '' ), 'myDatabase2' => array( 'host' => '', 'user' => '', 'pass' => '' ), 'myDatabase3' => array( 'host' => '', 'user' => '', 'pass' => '' ) ); public static function getInstance($db_name = 'default_db_name') { if(!isset(self::$_getInstance[$db_name])) self::$_getInstance[$db_name] = new self($db_name); return self::$_getInstance[$db_name]; } private function __construct($db_name) { // Connect $this->_data_connects[$db_name]['host']; $this->_data_connects[$db_name]['user']; $this->_data_connects[$db_name]['pass']; } } DB::getInstance(); DB::getInstance('myDatabase2'); DB::getInstance('myDatabase3'); (Отредактировано автором: 21 Сентября, 2012 - 09:44:47)

Мелкий	Отправлено: 21 Сентября, 2012 - 09:41:03
Активный участник Покинул форум Сообщений всего: 11926 Дата рег-ции: Июль 2009 Откуда: Россия, Санкт-Петербург Помог: 618 раз(а)	snikers987 пишет: Ну и зачем плодить подключения к бд? Например, при репликации. Классическая схема пишем на мастер, читаем со слейва. Хотя тут больше вопрос по названиям синглтона и фабрики. Класс, содержащий статическими несколько экземпляров самого себя и выдающего оные на основании переданного параметра (как раз armancho7777777 пример показал) - уже не синглтон, т.к. объектов класса уже несколько, но ещё и не фабрика, т.к. должен не только породить экземпляр, но и хранить его, как одиночку. ----- PostgreSQL DBA

armancho7777777	Отправлено: 21 Сентября, 2012 - 09:45:22
Активный участник Покинул форум Сообщений всего: 4526 Дата рег-ции: Февр. 2011 Откуда: Москва Помог: 221 раз(а)	Дописал ещё немного для наглядности )) (Добавление) Мелкий пишет: уже не синглтон, т.к. объектов класса уже несколько, но ещё и не фабрика, т.к. должен не только породить экземпляр, но и хранить его, как одиночку. single factory (фабрика одиночек) (Отредактировано автором: 21 Сентября, 2012 - 10:27:08)

digi	Отправлено: 21 Сентября, 2012 - 11:51:41
Посетитель Покинул форум Сообщений всего: 406 Дата рег-ции: Янв. 2012 Помог: 4 раз(а)	armancho7777777 пишет: single factory Улыбка (фабрика одиночек) registry

and_07	Отправлено: 21 Сентября, 2012 - 13:05:01
Гость Покинул форум Сообщений всего: 91 Дата рег-ции: Сент. 2012 Помог: 3 раз(а)	Всем доброго времени суток спасибо за комментарии учту но хотелось бы услышать что нибудь о реализации композиции или наследования и вообще если я реализую singleton разве такая конструкция будет нормально работать PHP: скопировать код в буфер обмена $GB = new GuestBook( DataClass::getInstance()) ; (Добавление) что в данном случае уместнее использовать композицию или наследование

Поиск в теме | Версия для печати

Страниц (2): [1] 2 »

Сейчас эту тему просматривают: 0 (гостей: 0, зарегистрированных: 0)

« Объектно-ориентированное программирование »

Все гости форума могут просматривать этот раздел.
Только зарегистрированные пользователи могут создавать новые темы в этом разделе.
Только зарегистрированные пользователи могут отвечать на сообщения в этом разделе.