PHP.SU

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

PHP.SU Портал

Главная

Помощь

Поиск

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

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

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

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

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

Реализация класса User и статичной функции в нем

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

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

Без описания

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

daewoosh	Отправлено: 26 Марта, 2014 - 08:37:56
Новичок Покинул форум Сообщений всего: 5 Дата рег-ции: Март 2014 Помог: 0 раз(а)	Доброго времени суток. У меня есть таблица Users (id, name, age, city). Данные о городах хранятся в таблице cities,таблицы связаны между собой по полю city_id. Мне необходимо создать страницу, выводящую список пользователей в виде таблицы. Но проблема в том, что нужно реализовать класс User. Передавая в конструктор в качестве параметра id пользователя, мы должны на выходе получить объект пользователя с заполненными данными. Это я делаю следующим образом: PHP: скопировать код в буфер обмена <?PHP class User { public $id; public $username; public $age; public $city; protected $mysqli; function __construct($id){ $this->mysqli = new mysqli('localhost', 'root', '','users'); $query = "SELECT * FROM users where id = '$id'"; if ($result = $this->mysqli->query($query)){ // Выбор данных из ответа сервераMySQL $arr = $result->fetch_array(MYSQLI_ASSOC); // Удаляем результирующую выборку $result->close(); // Вывод ответа клиенту return $arr; } else return "данных нет"; } // Деструктор закрывает соединение с базой function __destruct() { $this->mysqli->close(); } } ?> Взято это из книги Бенкен "AJAX программирование для интернета" и переделано мною. Правильно ли я реализовал конструктор? Так же в этом классе должна быть реализована статичная функция GetAllUsers, которая извлекает всех пользователей из таблицы, причем страница со списком пользователей формируется на основе этих данных. Как сделать это слабо себе представляю. Должен ли я реализовать вывод полученных данных прямо в этой же функции или рисовать таблицу и заполнять ее нужно отдельно?

T1grOK	Отправлено: 26 Марта, 2014 - 12:50:43
Частый гость Покинул форум Сообщений всего: 129 Дата рег-ции: Июнь 2013 Помог: 7 раз(а)	Крайне неудачная реализация. А если мне потребуется создать 10 экземпляров User, тогда у меня будет 10 подключений к БД. ----- Mysql, Postgresql, Redis, Memcached, Unit Testing, CI, Kohana, Yii, Phalcon, Zend Framework, Joomla, Open Cart, Ymaps, VK Api

esterio	Отправлено: 26 Марта, 2014 - 13:06:14
Активный участник Покинул форум Сообщений всего: 5025 Дата рег-ции: Нояб. 2012 Откуда: Украина, Львов Помог: 127 раз(а)	Есть правило не создавать коннект к базе в конструкторе. запомните это. Используйте паттерн Regostry или Singleton

daewoosh	Отправлено: 26 Марта, 2014 - 13:16:50
Новичок Покинул форум Сообщений всего: 5 Дата рег-ции: Март 2014 Помог: 0 раз(а)	T1grOK пишет: Крайне неудачная реализация. А если мне потребуется создать 10 экземпляров User, тогда у меня будет 10 подключений к БД. Ну а если закрывать подключение в конце конструктора? или проверку делать на наличие подключения? или вообще подключать в другом месте? попробовал сам реализовать статическую функцию GetAllUsers. Вроде работает все, но хотелось бы узнать удачна ли такая реализация, и , если нет, то почему? PHP: скопировать код в буфер обмена <?PHP class users { static function getAllUsers(){ header( 'Content-Type: text/html; charset=utf-8' ); $db_host = 'localhost'; $db_name = 'users'; $db_username = 'root'; $db_password = ''; $db_table_to_show = 'users'; $connect_to_db = mysql_connect($db_host, $db_username, $db_password) or die("Could not connect: " . mysql_error()); mysql_query("SET NAMES UTF8") or die(mysql_error()); mysql_query("SET CHARACTER SET UTF8") or die(mysql_error()); mysql_select_db($db_name, $connect_to_db) or die("Could not select DB: " . mysql_error()); $qr_result = mysql_query("select * from " . $db_table_to_show) or die(mysql_error()); echo '<table border="1">'; echo '<thead>'; echo '<tr>'; echo '<th>Id</th>'; echo '<th>Фамилия</th>'; echo '<th>Возраст</th>'; echo '<th>Город</th>'; echo '</tr>'; echo '</thead>'; echo '<tbody>'; while($data = mysql_fetch_array($qr_result)){ echo '<tr>'; echo '<td>' . $data['users_id'] . '</td>'; echo '<td>' . $data['users_name'] . '</td>'; echo '<td>' . $data['users_age'] . '</td>'; echo '<td>' . $data['city'] . '</td>'; echo '</tr>'; } echo '</tbody>'; echo '</table>'; mysql_close($connect_to_db); } } ?> Здесь пока что без конструктора. Рационально ли такое создание таблицы?Возможно ли будет в дальнейшем реализовать редактирование такой таблицы?

DelphinPRO	Отправлено: 26 Марта, 2014 - 14:53:21
Активный участник Покинул форум Сообщений всего: 7187 Дата рег-ции: Февр. 2012 Помог: 353 раз(а)	по сути класс User вообще ничего не должен знать о коннектах. Пусть он скажет вашей абстракции над БД - "дай-ка мне вон те данные", получит их и использует. И пофигу классу откуда взялись эти данные. А БД-абстракция, в свою очередь, не будет вникать что там за данные, она просто выполнит если надо коннект, сделает запрос к мускулу и вернет набор данных тому, кто их попросил. ----- Чем больше узнаю, тем больше я не знаю.

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

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

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

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

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