Создал свою функцию вывода SQL запроса с БД. Напишите минусы моего метода

@Tovenaar · Регистрация: 08.09.2015

Студворк — интернет-сервис помощи студентам

Первый файл sql.php (для ввода логина, пароля и название БД):

PHP
1
2
3
if( !$link = mysql_connect('localhost', 'root', 'root') ) return false;
mysql_set_charset('utf8');
$db = 'localhost';

Второй - собственна сама страница с фукцией:

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
function query(){
    if( func_num_args() < 1 or !is_file('sql.php') ) return false;  // если нет запроса или нет файла sql.php (с помощью которого мы соединяемся с БД), возвращаем ошибку
    include 'sql.php';
    $query = func_get_arg(0);
    
    if( func_num_args() > 1 ) $db = func_get_arg(0);    // если обращатся к другой базе, вводим второй аргумент
    if( !mysql_select_db( $db, $link) ) return false;
    
    $q = mysql_query( $query );
    if( gettype( $q ) == 'boolean' ){   // запросы типа UPDATE, DELETE и т.п
        if( $q ){ mysql_close($link); return true; }
        else{ mysql_close($link); return false; }
    }
    if( gettype( $q ) == 'resource' ){  // запросы типа SELECT
        while( $r = mysql_fetch_assoc($q) ){
            if( count($r) == 1 ) foreach( $r as $v ) $res[] = $v;
            else if( count($r) > 1 ) $res[] = $r;
        }
        
        if( isset($res) ){
            if( count($res) == 0 ) return false;
            else if( substr($query, strpos($query, 'LIMIT 1'), strlen($query)) == 'LIMIT 1' ) $res = $res[0];   // если нам нужена только одина строка и т.п
            return $res;
        }
    }
    
    mysql_close($link);
}

К примеру, есть таблица клиентов и обращаемся к нему:

PHP
1
2
3
4
print_r(query("SELECT * FROM `clients`"));  // запрашиваем весь список клиентов
    // вывод array(['id'=>1, 'name'=>'Павел', 'level'=>10],['id'=>2, 'name'=>'Воля', 'level'=>5])
 
echo query("SELECT `name` FROM `clients` WHERE `id`=2 LIMIT 1");    // вывод: Воля

Хотелось бы узнать Ваше мнение по этому методу и где я могу проверить скорость работы запросов?

KOPOJI · 08.09.2015, 08:37

Минусов много, в общем-то.
Использование устаревшего расширения (mysql_*), использование "автоматических" аргументов без надобности, мелкие недочеты..
Например, это

Сообщение от Tovenaar

PHP
1
2
3
4
if( gettype( $q ) == 'boolean' ){   // запросы типа UPDATE, DELETE и т.п
        if( $q ){ mysql_close($link); return true; }
        else{ mysql_close($link); return false; }
    }

Так и просится преобразоваться в это

PHP
1
2
3
4
if( gettype( $q ) == 'boolean' ){   // запросы типа UPDATE, DELETE и т.п
    mysql_close($link);
    return $q;
}

А это, в свою очередь, просит избавиться от лишнего цикла и проверок с count, когда есть empty

Сообщение от Tovenaar

PHP
1
2
3
4
5
6
7
8
9
10
11
if( gettype( $q ) == 'resource' ){* // запросы типа SELECT
* * * * while( $r = mysql_fetch_assoc($q) ){
* * * * * * if( count($r) == 1 ) foreach( $r as $v ) $res[] = $v;
* * * * * * else if( count($r) > 1 ) $res[] = $r;
* * * * }
if( isset($res) ){
* * * * * * if( count($res) == 0 ) return false;
* * * * * * else if( substr($query, strpos($query, 'LIMIT 1'), strlen($query)) == 'LIMIT 1' ) $res = $res[0]; * // если нам нужена только одина строка и т.п
* * * * * * return $res;
* * * * }
* * }

PHP
1
2
3
4
5
6
7
8
9
10
11
if( gettype( $q ) == 'resource' ){  // запросы типа SELECT
        while( $r = mysql_fetch_assoc($q) ){
            if( count($r) == 1 ) $res[] = current($v);
            else if( count($r) > 1 ) $res[] = $r;
        }
        
    if( empty($res) )
        return false;
    if( preg_match('~LIMIT 1$~i', $query) ) // если нам нужена только одина строка и т.п
            return current($res);
    }

Но все равно, код совершенно непонятный и не дает практически никаких преимуществ. Параметры необходимо писать в тексте запроса, вжесткую (хотя логичнее было бы дать возможность передавать параметры отдельно, а внутри уже формировать текст запроса). А возвращения mixed value и вовсе вносит путаницу: я еще и должен проверять, что мне вернулось: булево, строка, или же массив массивов. Булево ладно, понятно. Но насчет одного-многих значений - лучше уж добавить флаг (а-ля $getAll = true), раз не хотите отдельные функции писать.

Сообщение от Tovenaar

и где я могу проверить скорость работы запросов?

в мускуле, EXPLAIN

Сообщение от Tovenaar

PHP
1
if( gettype( $q ) == 'boolean' ){ * // запросы типа UPDATE, DELETE и т.п

Кстати, насчет этого я бы еще поспорил. Независимо от того, что "снаружи" при ошибочном запросе результат будет верным - false, в самом коде комментарий неверный, в общем-то. Ну и успех запросов а-ля UPDATE надо проверять при помощи mysql_affected_rows().

@Tovenaar · 08.09.2015, 10:46 **[ТС]**

Этот метод я использую, что бы сократить время кодинга. Частенько я использую соединени с БД и все время писать типа mysql_connect, mysql_query, mysql_fetch_array, и т.п, отнимает много времени.

Благодарю за Ваши замечания!

в мускуле, EXPLAIN

Я имел ввиду - время загрузки php скрипта с запросом на БД.

Добавлено через 1 час 13 минут
Исправил.
sql.php:

PHP
1
2
3
4
5
6
$sql = array(
    'host'      => 'localhost'
    'name'      => 'root',
    'pass'      => '123456'
    'datebase'  => 'mybd'
);

и сама функция:

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
function query(){
    if( func_num_args() < 1 ) return false;
    
    include 'assets/sql.php';
    if( !$link = mysql_connect($sql['host'], $sql['name'], $sql['pass']) ) return false;
    mysql_set_charset('utf8');
    $query = func_get_arg(0);
    
    if( func_num_args() > 1 ) $sql['datebase'] = func_get_arg(1);
    if( !mysql_select_db( $sql['datebase'], $link) ) return false;
    
    $q = mysql_query( $query );
    switch( gettype($q) ){
        case 'boolean':
            if( mysql_affected_rows() == 0 ) $q = false;
            mysql_close($link);
            return $q;
        case 'resource':
            while( $r = mysql_fetch_assoc($q) ){
                if( count($r) == 1 ) $res[] = current($v);
                else if( count($r) > 1 ) $res[] = $r;
            }
            
            mysql_close($link);
            
            if( isset($res) ){
                if( empty($res) ) return false;
                if( preg_match('~LIMIT 1$~i', $query) ) return current($res);
                return $res;
            }
    }
}

KOPOJI · 08.09.2015, 22:12

Сообщение от Tovenaar

PHP
1
2
if( isset($res) ){
* * * * * * * * if( empty($res) ) return false;

лишнее это, говорил же

PHP
1
2
if(empty($res))
    return false;

Проверка на isset не имеет смысла, ибо:
1) empty включает в себя ту же самую проверку на isset (то, что значение существует и не равно NULL)
2) если проверка выполнится (empty вернет true), то функция вернет значение false и завершит свое выполнение
3) исходя из предыдущих пунктов, достаточно кода, который я написал выше:

PHP
1
2
3
4
5
if( empty($res) )
    return false;
if( preg_match('~LIMIT 1$~i', $query) ) // если нам нужена только одина строка и т.п
    return current($res);
return $res;

по поводу остального - все равно вы предпочитаете использовать func_get_arg, хотя это неудобно: необходимо смотреть код функции, чтобы узнать, что ожидает функция получить (при использовании "обычного" способа передачи аргументов большинство IDE отобразит вам прототип функции); assets - это общепринятое название для статических файлов - css, js, картинки, а вовсе не для конфигурационных файлов с настройками соединения.
Ну и да, вы в курсе, что при каждом вызове вашей функции вы будете заново пересоздавать соединение с БД? Лучше один раз создайте подключение и передавайте в функцию идентификатор, ну или, хотя бы, используйте статическую переменную

Добавлено через 2 минуты

Сообщение от Tovenaar

время загрузки php скрипта с запросом на БД

если по-простому, то можно таким скриптом https://www.cyberforum.ru/post7709681.html

@Tovenaar · 10.09.2015, 09:43 **[ТС]**

Насчет func_get_arg, в сысле обычную? Вы имеете ввиду типа такого?:

PHP
1
2
3
function name($arg1, $arg2){
    if( empty($arg1) ) return false;
}

Если да, тогда необходимо аргумент вводить, ибо PHP начинает жаловаться.
Если же нет, какие ещё методы есть?

Насчет подключения, решил сделать так:

PHP
1
2
3
4
5
6
7
8
function query(){
    ...
    global $mysql;
    if( empty($mysql) and !empty($mysql = mysql_connect($sql['host'], $sql['name'], $sql['pass'])) ) return false;
    ...
}
...
if( isset($mysql) ) mysql_close($mysql);

Что Вы думаете об этом?
Насчет последнего if( isset($mysql) ), есть ли функция, которая проверяет - подключен ли он к БД?
Что если не закрывать подключение?

KOPOJI · 10.09.2015, 20:30

Сообщение от Tovenaar

Вы имеете ввиду типа такого?

Именно. Только называть соответственно.

Сообщение от Tovenaar

Если да, тогда необходимо аргумент вводить, ибо PHP начинает жаловаться

ничего не понял из этой фразы

Сообщение от Tovenaar

global $mysql;

использование global считается не лучшим способом, с передачей аргумента, пожалуй, было бы получше.. Но и так сойдет, в принципе.

Сообщение от Tovenaar

Что Вы думаете об этом?

мне не нравится использование empty для проверки присваивания установки соединения

Ну да ладно

Сообщение от Tovenaar

Насчет последнего if( isset($mysql) ), есть ли функция, которая проверяет - подключен ли он к БД?

смысла закрывать соединения обычно нет - пых самостоятельно закроет соединение после завершения работы скрипта. Ну а по поводу функции для проверки, установлено ли соединение с БД - empty подойдет, пожалуй, как нельзя лучше.

Новые блоги и статьи Все статьи Все блоги /
Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит токи на L и напряжения на C в установ. режимах до и. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .