Форум программистов, компьютерный форум, киберфорум
PHP: ООП
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.63/8: Рейтинг темы: голосов - 8, средняя оценка - 4.63
 Аватар для Raisin Zn
101 / 100 / 51
Регистрация: 19.04.2011
Сообщений: 965

Оцените код

04.01.2016, 08:16. Показов 1530. Ответов 4
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Здравствуйте. В этой теме Mysqli_set_charset в ООП пошла речь о том какой ужасный код.
Я как бы знаю о его "уродстве", так как последнее время приходится с ним плотно работать, но хотелось бы услышать еще мнение других людей, которые в php понимают больше меня.
Код в посте не поместился, пришлось заархивить...

Для перехода на mysqli используется массив из другого файла:
PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
<?php
defined('PHPFOX') or exit('NO DICE!');
 
Phpfox::getLibClass('phpfox.database.driver.mysql');
 
class Phpfox_Database_Driver_Mysqli extends Phpfox_Database_Driver_Mysql
{
    /**
     * Array of all the MySQLi functions we use. This 
     * variable overwrites the parent MySQL variable.
     *
     * @see parent::$_aCmd
     * @var array
     */
    protected $_aCmd = array(
        'mysql_query'              => 'mysqli_query',
        'mysql_connect'            => 'mysqli_connect',
        'mysql_pconnect'           => 'mysqli_pconnect',
        'mysql_select_db'          => 'mysqli_select_db',
        'mysql_num_rows'           => 'mysqli_num_rows',
        'mysql_fetch_array'        => 'mysqli_fetch_array',
        'mysql_real_escape_string' => 'mysqli_real_escape_string',
        'mysql_insert_id'          => 'mysqli_insert_id',
        'mysql_fetch_assoc'        => 'mysqli_fetch_assoc',
        'mysql_free_result'        => 'mysqli_free_result',
        'mysql_error'              => 'mysqli_error',
        'mysql_affected_rows'      => 'mysqli_affected_rows',
        'mysql_get_server_info'    => 'mysqli_get_server_info',
        'mysql_set_charset'        => 'mysqli_set_charset',        
        'mysql_close'              => 'mysqli_close'        
    );
}
Спасибо за критику.
Вложения
Тип файла: rar mysql.class.rar (4.9 Кб, 5 просмотров)
0
Лучшие ответы (1)
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
04.01.2016, 08:16
Ответы с готовыми решениями:

Оцените код пожалуйста - правильно ли я понял ООП и MVC?
Здравствуйте! php изучаю уже три месяца, когда есть возможность. С ООП в php познакомился две неделии назад. До этого с ним немного имел...

Оцените код
Доброго времени. Выполняю задание, необходимо собрать пользователей сайта habr, и у каждого по пять записей, сайт должен быть в рамках MVC....

Оцените код
Всем привет. Такая ситуация, пару недель назад пытался устроиться на работы на вакансию &quot;PHP Разработчик&quot;. По требованиям я им...

4
 Аватар для Drygba
512 / 454 / 119
Регистрация: 17.02.2012
Сообщений: 1,032
Записей в блоге: 1
05.01.2016, 12:41
Лучший ответ Сообщение было отмечено Raisin Zn как решение

Решение

Жуть.

Сначала давайте о структуре и стиле.
1. Откуда в классах взялись глобальные константы?
2. Название переменных. Догадайся когда читаешь код что это - _bIsSlave, _aCmd.
3. Название методов. При виде метода is... интуитивно ожидаешь получить true или false но не то что у вас в isNull/isNotNull.
4. Какой смысл для переменных добавлять приставку s - sSql, sHost, sPass... . Зачем?
5. Зачем глушите ошибки @?

О самом классе.
1. Зачем делать коннект сразу и к мастеру и к слейву? Делайте коннекты только когда будет нужно.
2. Что это за выбор случайного коннекта к слейву?
3. Объекты/ресурсы передаются по ссылке, запись =& бессмысленна.
4. Объект который отвечает за коннект к базе и работу с ней - не должен не чего логировать, это не его обязанности.

Добавлено через 20 минут
О самом классе.
5. Смысл в массиве protected $_aCmd?
6. Зачем нужен mysqli? Чем не устраивает PDO?
2
 Аватар для Raisin Zn
101 / 100 / 51
Регистрация: 19.04.2011
Сообщений: 965
05.01.2016, 13:32  [ТС]
Drygba, спасибо, что уделили время на просмотр этого кода...
На все Ваши вопросы, ответить не могу, так как этот код я не писал (phpFOX 3), но мне с ним приходится работать...
0
Фрилансер
Эксперт PythonЭксперт JSЭксперт PHP
 Аватар для Azdeman
1871 / 1362 / 604
Регистрация: 12.01.2011
Сообщений: 5,470
08.01.2016, 21:36
Цитата Сообщение от Drygba Посмотреть сообщение
Зачем нужен mysqli? Чем не устраивает PDO?
а чем не устраивает Mysqli?
0
Software Engineer
 Аватар для Custos
332 / 336 / 55
Регистрация: 23.09.2014
Сообщений: 997
09.01.2016, 01:10
Цитата Сообщение от Drygba Посмотреть сообщение
3. Объекты/ресурсы передаются по ссылке, запись =& бессмысленна.
Не совсем
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
09.01.2016, 01:10
Помогаю со студенческими работами здесь

Оцените мой код. Знания в Веб технологиях у меня не маленькие и опыт тоже!
Ребят, нет времени объяснять, вообщем оцените мой код. Есть база данных с таблицей аккаунтов, в ней 10.000 записей, вот я короче делаю...

Оцените от 1 до 10 код
Всем привет!Меня зовут Макс.Мне 14 лет и я хотел бы получить вашу оценку о коде который написал 2 дня назад.Нормально для 14 лет? ...

Оцените код
Собственно вот условие задачи: Дано целое число m&gt;1.Получить наибольшее целое k,при котором 4k&lt;m. Вот код программы: var m,k :...

Оцените код
В общем, пилю я тут свою &quot;мега,кликер,супер&quot; игру, скажите пожалуйста, что не так? Может есть какие-то очевидные ошибки и недочёты? Буду...

Оцените код
Короче первый код который я забацл сам считай без помощи интернета! Теперь пришло время его показать хоть кому-то. Если посмотреть по...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Новые блоги и статьи
Кому нужен AOT?
DevAlt 26.03.2026
Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .
Отправка уведомления на почту при изменении наименования справочника
Maks 24.03.2026
Программная отправка письма электронной почты на примере изменения наименования типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной. . .
модель ЗдравоСохранения 5. Меньше увольнений- больше дохода!
anaschu 24.03.2026
Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .
Midnight Chicago Blues
kumehtar 24.03.2026
Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .
SDL3 для Desktop (MinGW): Вывод текста со шрифтом TTF с помощью библиотеки SDL3_ttf на Си и C++
8Observer8 24.03.2026
Содержание блога Финальные проекты на Си и на C++: finish-text-sdl3-c. zip finish-text-sdl3-cpp. zip
Жизнь в неопределённости
kumehtar 23.03.2026
Жизнь — это постоянное существование в неопределённости. Например, даже если у тебя есть список дел, невозможно дойти до точки, где всё окончательно завершено и больше ничего не осталось. В принципе,. . .
Модель здравоСохранения: работники работают быстрее после её введения.
anaschu 23.03.2026
geJalZw1fLo Корпорация до введения программа здравоохранения имела много невыполненных работниками заданий, после введения программы количество заданий выросло. Но на выплатах по больничным это. . .
Контроль уникальности заводского номера
Maks 23.03.2026
Алгоритм контроля уникальности заводского (или серийного) номера на примере нетипового документа выдачи шин для спецтехники с табличной частью, разработанного в конфигурации КА2. Номеклатура. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru