Как правильно создать SQL процедуру?

Question

[nkorobkov]

Добрейшее утречко. Имеется вот такая БД:

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

CREATE PROCEDURE get_cost(org_name, service_name)
BEGIN
    SELECT organizations.org_name, services_list.service_name, org_services.service_cost 
        FROM organizations, services_list, org_services WHERE organizations.ID = 
        org_services.org_id AND services_list.ID = org_services.service_id
        HAVING organizations.org_name = org_name AND services_list.service_name = service_name;
END

Подскажите пожалуйста: что не так, и как можно исправить или реализовать по-другому...

Answer 1

[Rsa97]

А зачем для этого процедура? Достаточно обычного запроса
И зачем возвращать из запроса названия организации и сервиса, если их туда только что передали?

SELECT `os`.`service_cost`
  FROM `Org_services` AS `os`
  JOIN `Organizations` AS `o` ON `o`.`ID` = `os`.`org_id` AND `o`.`org_name` = :orgName
  JOIN `Services_list` AS `s` ON `s`.`ID` = `os`.`service_id` AND `s`.`service_name` = :servName

Comments

[nkorobkov]

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

[Rsa97]

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

[d-stream]

Процедура позволяет избежать проблем при например изменении структуры данных.

[Rsa97]

d-stream: Это значит, что при изменении структуры базы переписывать придётся процедуры, а не код приложения, только и всего.

[d-stream]

Rsa97: да, а соответственно если процедура вызывается более чем из одного места - то переписывать придется во столько раз меньше, сколько раз процедура вызывалась в разных местах

[Rsa97]

d-stream: Дык оно и в коде крайне рекомендуется повторяющиеся фрагменты программы выносить в отдельные подпрограммы. Для этого не обязательно использовать процедуры SQL.

[d-stream]

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

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

[Rsa97]

d-stream: И опять таки, в чём разница, будет у вас в программе делаться запрос "CALL my_giganteous_procedure(param1, ..., param35)" или вызываться подпрограмма my_giganteous_subroutine(param1, ..., param35)?

А если ещё учесть, что MySQL не позволяет в хранимых процедурах захватывать таблицы (LOCK TABLE), то для корректной работы многопользовательской системы при изменении структуры базы вам придётся вносить правки и в хранимые процедуры, и в те места, откуда они вызываются. Иначе за время такого сложного анализа кто-то может просто продать этот товар в розницу, и оптовику товара уже просто не достанется.

[d-stream]

Rsa97: Вот как-то в повседневной практике так получается что профита от использования именно хп и триггеров я получаю гораздо больше, чем гимора от остатков прямых вызовов.
Тут сложно объяснить, примерно как в спорах "да движок на файлах в разы лучше и надежнее всяких там ваших баз данных".
Разве что скатываться в войны за то с какой стороны яйцо разбивать -)