Форум dkLab и Denwer

[Дмитрий Котеров]

Вышла версия 2 библиотеки (поддержка MySQL, PosgreSQL, FireBird; макроподстановки; атрибуты).
dklab.ru/lib/DbSimple/

Полный список изменений в версиях (dklab.ru/wsvn/lib/DbSimple/trunk/?op=log&rev=0&sc=0&isdir=1)
Текущая разрабатываемая версия (внимание! может быть нестабильной!) (dklab.ru/wsvn/lib/DbSimple/trunk/)

Обсуждения старой версии 1.x (forum.dklab.ru/viewtopic.php?t=18395) (тема закрыта, это архив).
Обсуждения разработчиков библиотеки 2.x (forum.dklab.ru/viewtopic.php?t=23766) (если вы делаете драйвер для новой СУБД, пишите туда).

В текущем топике же - общие вопросы по библиотеке, баг-репорты, пожелания и т.д.

Модули для PostgreSQL и FireBird подготовлены Константином Жинько [tIT].

Last edited by Дмитрий Котеров on Tue Jan 09, 2007 1:17 am; edited 3 times in total

[Дмитрий Котеров]

Интересно погуглить по слову DbSimple.

Находятся всякие разные библиотеки; одна из них - www.greaterscope.net/project/dbSimple - на видо очень пожожа на DbSimple (особенно имена методов и идеология фетча). Однако, заглянув в код, становится ясно, что она крайне слабая. Там даже placeholder-ов нет.

Константин Жинько [tIT] wrote:

В DbSimple2 работать не будет, потому что ты поменял debug_backtrace_smart

Вроде поправил - надо было просто

$caller = $db->findLibraryCaller();

[Антон Макаренко]

Переходя на 2.x с 1.x, попался на глупое стечение обстоятельств.
К папке с библиотеками (скажем, ".../lib") был прописан include path. 1.x лежала в папке .../lib/DbSimple, а 2.x я сохранил как .../lib/DbSimple2
Таким образом, код оказался нерабочим:

Code (php):

скопировать код в буфер обмена

// include path if (!defined (www.php.net/defined)("PATH_SEPARATOR"))         define (www.php.net/define)("PATH_SEPARATOR", getenv (www.php.net/getenv)("COMSPEC")? ";" : ":"); ini_set (www.php.net/ini_set)("include_path", ini_get (www.php.net/ini_get)("include_path").PATH_SEPARATOR.$CFG['LIBS']); // ... // mysql connection require(CFG_LIBS.'/DbSimple2/Generic.php'); $DB = DbSimple_Generic::connect('mysql://'.CFG_MYSQL_USER.':'.CFG_MYSQL_PASSWORD.'@'.CFG_MYSQL_HOST.'/'.CFG_MYSQL_DATABASE);

Как показало исследование кода, если include path прописан, Generic.php ищет DbSimple/<бд>.php в include path... и находит первую версию :-/
Be careful!

[Maus]

Антон Макаренко wrote:

Что если при составлении запроса при передаче массивов в ?a и ?# анализировать числа и булевы и не окружать их кавычками?

а что, если кому-то другому нужно иногда число передать именно как строку? Мы приходим опять к тому же самому - типизированным массивным плейсхолдерам

[Антон Макаренко]

Maus wrote:

число передать именно как строку

Для этого в PHP есть приведение типов, функция settype(), а в DbSimple, соответственно is_integer(), is_float(). Сами плейсхолдеры ?a и #? не меняются, только убирают "лишние" кавычки при составлении, основываясь на типах элементов исходного массива.

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

[Константин Жинько [tIT]]

М

Ветка выделена в отдельную тему «MySQL - strict mode», расположенную в форуме Общие вопросы :: SQL (13 Декабря 2006, 15:40).

[symbix]

И вдогонку. Первая мысль - а зачем я сделал i алиасом a, ведь там могут быть и числовые ключи. Вторая - так ведь и логично: для запроса вида "insert into table values (?i)", array('foo','bar') как раз подходит :)

Тут же третья мысль - добавить константу DBSIMPLE_DEFAULT, которая трансформируется в неэкранированное DEFAULT. -INF (оно же log(0)) уже используется - зато просто INF свободно, можно для этих целей и попользовать.

[symbix]

...а еще есть NAN - например, результат acos(1.01). Сколько интересного узнаешь, направленно копая мануал :)

[Константин Жинько [tIT]]

symbix wrote:

Но в целом, вынос такого функционала в Generic мне кажется вполне оправданным, так как стандарт предусматривает только такую форму записи insert into.

А зачем дополнительный маркер? Можно ?a оставить. Пожалуй, полезная фича.

М

Вы бы зарегистрировались что ли.

[symbix]

Гм, если оставить "?a", то как отличить контекст insert от update? дополнительный парсинг запроса делать? слишком затратно это, мне кажется.

С DEFAULT все совсем просто получилось:
в Generic.php

Code (php):	скопировать код в буфер обмена
define (www.php.net/define)('DBSIMPLE_SKIP', INF);

и в Postgresql.php

Code (php):	скопировать код в буфер обмена
function _performEscape($s, $isIdent=false)     {         if (!$isIdent) !           return $s === DBSIMPLE_DEFAULT ? 'DEFAULT' : ("'" . str_replace (www.php.net/str_replace)("'", "''", $s) . "'");         else             return '"' . str_replace (www.php.net/str_replace)('"', '_', $s) . '"';     }

Возможно, по аналогии с обработкой NULL лучше это делать в Generic.php. Хотя, с другой стороны, я бы и обработку того же NULL вынес в _performEscape - чисто эстетически мне не нравятся частные случаи, дублирующиеся в нескольких местах.

Quote:

Вы бы зарегистрировались что ли.

Спасибо за предложение, но не думаю, что буду активным участником форума вне этой темы.

[symbix]

Ошибся, define('DBSIMPLE_DEFAULT', INF); разумеется.

[Phoebus]

symbix
А так?

Code (php):	скопировать код в буфер обмена
$DB->Query('         INSERT INTO tbl ( ?# )         VALUES ( ?a )',                 array_keys (www.php.net/array_keys)($array_data),                 array_values (www.php.net/array_values)($array_data));

И не надо доп. маркер совершенно. Такое нововведение как ?i -- это то же самое как введение метода Select($tables_to_fetch,$columns_to_fetch) который бы строил запрос самостоятельно.

Last edited by Phoebus on Sun Dec 17, 2006 11:35 pm; edited 1 time in total

[Константин Жинько [tIT]]

Phoebus wrote:

$DB->Query('         INSERT INTO tbl ( ?a )         VALUES ( ?a )',                 array_keys($array_data),                 array_values($array_data));

Вариант самый разумный, если не одно но: ключи - строки, поэтому будут заключены в апострофы. В случае MySQL это породит ошибку.

[Phoebus]

Константин Жинько [tIT] wrote:

Вариант самый разумный, если не одно но: ключи - строки, поэтому будут заключены в апострофы. В случае MySQL это породит ошибку.

Пардон, первый плейсходлер у меня используется так: ?#
Я уже давно себе переделал его, добавив проверку is_array: если массив, то квотим массив идентификаторов, разделяя запятой, если нет -- то стандартный квотинг строки.

[Константин Жинько [tIT]]

Дмитрий Котеров wrote:

И второй вопрос - приведите пример запроса, где DEFAULT требуется.

Code (SQL):	скопировать код в буфер обмена
CREATE TABLE test(   str VARCHAR(32) DEFAULT 'YOHOHO!' );

(-;

Перефразирую вопрос: приведите select/insert/update запрос, где используется ключевое слово DEFAULT. Лично мне пока такие конструкции не попадались.

Дмитрий Котеров wrote:

Я переделаю на case insensitive, если ты не против.

Объективных возражений пока нет.

[Дмитрий Котеров]

М

Ветка выделена в отдельную тему «Аспекты Subversion», расположенную в форуме Конструктор (18 Декабря 2006, 01:49).

[Александр Шустов]

Кстати, невозможность использования комбинации ?_?# - это фича или баг?

[Дмитрий Котеров]

?_?# - можете использовать, а можете и не использовать, в зависимости от контекста. Если в качестве значения для ?# передаете имя таблицы без префикса, то надо ?_?#, а если с префиксом, то ?#. Кроме того, в ?# ведь могут не только имена таблиц передаваться, но также и имена столбцов, а им префикс не нужен. Так что нельзя сказать, баг это или фича - так же, как нельзя сказать, баг или фича использование PHP-шного trim() в программе. :-)

Александр Шустов wrote:

теперь полностью отпала необходимость в DbSimple с собственными добавлениями

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

[symbix]

Quote:

Перефразирую вопрос: приведите select/insert/update запрос, где используется ключевое слово DEFAULT. Лично мне пока такие конструкции не попадались

Code (php):	скопировать код в буфер обмена
if ($id === false) {   $param = "DEFAULT"; } else {   $param = "'" . $db->escape($id) . "'"; } //... $db->query("insert into table (param, foo, bar) values ($id, $foo, $bar)");

Встречается нечто подобное. (Тут $db, разумеется, к DbSimple отношения не имеет).

[Дмитрий Котеров]

Наверное, Вы имели в виду не $id, а $param в query().

А каково значение default для таблицы table? Приведите полную структуру таблицы.
Кроме того, попробуйте использовать "insert into table(?#) values(?a)" (см. выше) и в случае default просто не передавать поле id вовсе.

[symbix]

Да, Вы правы, #? с поддержкой массивов, пожалуй, действительно решает сразу все вопросы.

[Guest]

Дмитрий Котеров wrote:

?_?# - можете использовать, а можете и не использовать

Дмитрий, вы меня неправильно поняли.
Например: $mysql->setIdentPrefix('prx_');
При попытке использования комбинации ?_?# получаем prx_`table_name`

[Константин Жинько [tIT]]

Гость wrote:

При попытке использования комбинации ?_?# получаем prx_`table_name`

Это баг.

[Дмитрий Котеров]

Нет, это не баг.

Вот правильное решение:
$DB->query("SELECT * FROM ?#", $DB->setIdentPrefix() . $tableName)

[Александр Шустов]

Дмитрий Котеров wrote:

Вот правильное решение: $DB->query("SELECT * FROM ?#", $DB->setIdentPrefix() . $tableName)

А вы не ошиблись?
$DB->setIdentPrefix() - возвращает старый вариант префикса

прошу прощения вы правы

[Константин Жинько [tIT]]

Дмитрий Котеров wrote:

$DB->query("SELECT * FROM ?#", $DB->setIdentPrefix() . $tableName)

Криво и коряво - каждый раз писать $DB->setIdentPrefix(), когда нужна таблица. Куда короче руками этот префикс записать (-;

[Александр Шустов]

Но все-таки, как-то некрасиво и если в ?# передать массив, все усложниться в разы
да и если и использовать приведенный вами вариант, то нужно
function setIdentPrefix($prx) заменить на function setIdentPrefix($prx = null)

да и вариант setIdentPrefix().$table_name это все-таки не ?_?#...

[Дмитрий Котеров]

Константин Жинько [tIT] wrote:

каждый ра

"Каждый раз" - это сильно. :-) Ситуации, когда это бывает нужно, случаются исчезающе редко и говорят скорее об ошибках в проектировании архитектуры, чем о недостатках метода.

Александр Шустов wrote:

если в ?# передать массив, все усложниться в разы

Префикс ?_ используется для имен таблиц, а не для полей. Зачем Вам передавать список имен таблиц в ?# ?

[Дмитрий Котеров]

М

Ветка выделена в отдельную тему «DbSilly: а давайте напишем собственный SQL-парсер!», расположенную в форуме Мусоропровод (20 Декабря 2006, 23:07).

[zuzmic]

Прошу прощения, если я написал не в ту ветку.
Хочу спросить DBSimple есть в варианте совместимом с PHP5 ?

[Константин Жинько [tIT]]

zuzmic wrote:

Хочу спросить DBSimple есть в варианте совместимом с PHP5 ?

Он совместим с PHP4, а следовательно совместим с PHP5. У меня на 5.1.6 работает превосходно!
С 5.2 могут быть (и скорее всего будут) проблемы, ибо обратная совместимость 5.2 во многом утеряна.

[Дмитрий Котеров]

М

Ветка выделена в отдельную тему «DbSimple: передача атрибутов через массив, а не в тексте запроса», расположенную в форуме Разное :: PHP (28 Декабря 2006, 16:29).

[chin]

Господа, ответьте, пожалуйста.

chin wrote:

Где мы явно указываем изменение таблицы tree.

Это важный вопрос для меня. Какие мысли по этому поводу?

[Дмитрий Котеров]

Кстати, в качестве кэш-хранилища можно использовать Cache_Lite из PEAR:
pear.php.net/manual/en/package.caching.cache-lite.intro.php

Вероятно, стоит сделать поддержку его прямо в библиотеке, т.е. дать возможность в setCacher() передавать прямо объект класса Cache_Light (method_exists($cacher, 'get') && method_exists($cacher, 'save')), а не только имя функции-кэшера. Так будет совсем универсально и благостно для буржуев. ;-)

[Дмитрий Котеров]

М

Ветка выделена в отдельную тему «DbSimple и кэширование.», расположенную в форуме Конструктор (13 Января 2007, 15:39).

[Дмитрий Котеров]

М

Ветка выделена в отдельную тему «DbSimple: устаревшие сообщения.», расположенную в форуме Мусоропровод (13 Января 2007, 15:43).

[Михаил Саяпин]

Доброго времени,

Замечательный враппер, единственное, что на моем неидеальном хостинге с выделением памяти под PHP всего 4 Мбайт - падает с превышением памяти из-за прямого выбора всего запроса. mysql_query + mysql_fetch_* решили проблему, но решение некрасивое. Очень бы хотелось какой-то модификатор для порядового выбора.

[Дмитрий Котеров]

Что же у Вас там за выборка такая огормная, что она в память не влезает... Может быть, неправильная логика скрипта? Например, выбираете ВСЕ записи, вместо того, чтобы выбрать 1 страницу?

[Guest]

Возможно не очень внимательно искал... А есть ли возможность получить имена и типы полей таблицы?

[Дмитрий Котеров]

dklab.ru/lib/DbSimple/manual.html

Обращаю особое внимание на то, что DbSimple намеренно не занимается «выравниваниием диалектов» SQL в различных СУБД (что частично пытаются делать PEAR DB и ADOdb). Она лишь позволяет обращаться к ним через единый стандартизированный и очень удобный интерфейс.
...
Также DbSimple не является слоем получения стандартизированных метаданных БД (информации о структуре таблиц, полей, индексов и т. д.).

Вообще, у меня есть такое убеждение, что необходимость получения метаданных базы свидетельствует об ошибке в архитектуре системы.