Russian Qt Forum

Добрый день!  :)

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

SQL
ERROR: NULL value IN COLUMN "column_name" violates not-NULL constraint

psql-драйвер, к сожалению, не возвращает коды ошибок. Я вижу два варианта.
1) Изменять данные в таблицах не напрямую (INSERT table ...), а с помощью хранимых процедур, в которых генерить исключение в случае ошибки с "красивым" текстом.
2) Парсить пришедший текст ошибки, на основании ключевых слов строить правильный текст, который и показывать пользователю.
В первом случае получается много-много функций в БД, второй - не очень надежен, т.к. текст может и измениться, со сменой версии СУБД, например.

Помогите определиться! Или, может, есть способы лучше? Как вы решаете такие задачи?

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

Первый вариант определенно лучше для внятных сообщений об ошибках. Тем более в этом случае все будет на сервере БД, клиенты будут дергать только хранимые процедуры. В случае любых изменений, Вам необходимо будет поправить процедуру, и не думать о том сколько у Вас клиентов и все ли обновили версию ПО для корректной работы с изменившейся структурой, например.

Добрый день!
Вообще, вот тут про ошибки
http://www.prog.org.ru/index.php?topic=22903.msg162092#msg162092 (http://www.prog.org.ru/index.php?topic=22903.msg162092#msg162092)
Исправленный драйвер для 5.3 в архиве

Я правильно понимаю, что для каждой таблицы должно быть создано как минимум 3 хранимки - на добавление новой записи, на редактирование и на удаление? Не получается ли изобилие хранимок в таком случае?

С одной стороны, я тоже об этом думал. Действительно, вся работа с БД собрана в одном месте, в код идет самый минимум. С другой стороны - уже сталкивался с такой проблемой, что функций в БД становилось чересчур много. И опять же - при любом изменении в структуре таблиц следует помнить еще про необходимость изменить все связанные с ней функции.

Вот править драйвер мне кажется не очень хорошим и "правильным" делом. Или я не прав?

У нас данные,в основном, добавляются сразу в несколько таблиц.

Может лучше не допускать подобных ошибок.
На уровне UI контролировать заполнение обязательных полей и не давать завершать редактирование, пока пользователь их не введет.

Можете посмотреть diff от оригинала. Ничего существенного там не добавлено.
Драйвер тоже люди писали. И там тоже могут быть ошибки и отсутствовать некий нужный функционал.
В багтрекер qt этот код просили добавить, но это там никому не нужно.
Я себе драйвер компилирую сам и им пользуюсь. Проблем нет.
А так... каждый для себя решает сам.

Вот код который я первоначально использовал.
http://www.dfndr.pp.ua/qt4-qpsql-%D0%B4%D1%80%D0%B0%D0%B9%D0%B2%D0%B5%D1%80-%D0%BA%D0%BE%D0%B4%D1%8B-%D0%BE%D1%88%D0%B8%D0%B1%D0%BE%D0%BA.html (http://www.dfndr.pp.ua/qt4-qpsql-%D0%B4%D1%80%D0%B0%D0%B9%D0%B2%D0%B5%D1%80-%D0%BA%D0%BE%D0%B4%D1%8B-%D0%BE%D1%88%D0%B8%D0%B1%D0%BE%D0%BA.html)
Вот еще об этой же проблеме.
http://www.qtcentre.org/threads/17495-patch-QPSQL-set-an-error-code-so-QSqlError-number()-gives-a-useful-value (http://www.qtcentre.org/threads/17495-patch-QPSQL-set-an-error-code-so-QSqlError-number()-gives-a-useful-value)

Заполнение обязательных полей контролировать - это да. А вот что делать с попыткой удалить запись, на которую есть ссылки в других таблицах? Это уже не проконтролируешь, это только получать сообщение от БД, что нельзя удалять эту строку...

Спасибо за ссылки, изучу. Я все варианты рассматриваю :)

Почему, можно пробежаться по таблицам, где могут быть ссылки на удаляемую запись и проверить есть такие записи или нет.
Делается это одним простым запросом на каждую таблицу.
Я так и делаю и если эту запись удалить нельзя, по причине зависимостей, то у меня даже кнопка удаления дизаблица.

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

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

Делай проверки не при открытии, а при выполнении пользователем определенных действий: попытке удаления, попытке изменения.

В посте выше речь шла о том, что кнопка удаления заблокирована сразу. Если же проверять в момент выполнения действия, то не вижу большой разницы - самому осуществить проверку и выдать ответ пользователю о невозможности удаления или попытаться удалить без проверку и получить ответ от БД о невозможности удаления.

Насколько я понял, вас не устраивает текст ответа, так смысл тогда от БД получать ответ?

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

Я уж грешным делом подумал, что все давно решилось с помощью кода, который я прикреплял.
Давно с ним работаю. Коды ошибок получаю. Проблем пока нет.
Я лично для себя решил, что ждать когда включат этот код в драйвер смысла нет.