Форум пользователей MySQL

Ovenvan · 15.05.2016 09:30:15

Добрый день, опять MySQL вводит меня в тупик, или я уже сам просто крайне невнимателен.

Хочу реализовать процедуру на добавление нового договора:

create procedure SigningContract(new_id int, new_sum int, new_begin date, new_individ int, 

new_comp int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

declare check1plus int;

set check1 = (select max(id_contract) from insurance_contract);

set check1plus = check1++;

if (select count(*) from insurance_contract where id_contract = new_id) != 0 then new_id = check1plus;

else if (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

else if (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

else if (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

else if (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

else if (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

end if;

end if;

end if;

end if;

end if;

end if;

insert into insurance_contract values(new_id, new_sum, new_begin, AddPeriod(new_begin), new_individ, new_comp, new_agent, new_object, new_categ);

end

Добавление со все возможными проверками (правильность ввода id, существования клиента и т. д.), также есть функция AddPeriod(new_begin), возвращающая окончание срока договора.
Но почему-то эта процедура не компилируется, выдавая ошибку синтаксиса рядом с первой проверкой IF. Неужели я умудрился допустить ошибку в таком месте?

klow · 15.05.2016 09:46:57

Впервые слышу об операторе ++ (check1++) в MySQL!

Ovenvan · 15.05.2016 09:51:45

Вы думаете это не будет работать? Просто на set check1plus = check1 + 1 тоже ругался

klow · 15.05.2016 09:56:00

думаю не будет.
Кроме того
new_id = check1plus; нужно SET new_id = check1plus;
else if нужно elseif

Отредактированно klow (15.05.2016 10:00:14)

Ovenvan · 15.05.2016 10:00:21

klow написал:
думаю не будет.
Кроме того ELSE IF! http://dev.mysql.com/doc/refman/5.7/en/ … unction_if
рекомендую использовать CASE http://dev.mysql.com/doc/refman/5.7/en/ … rator_case

Хорошо, case - вероятно хорошее решение, но мне бы хотелось всё-таки ещё разобраться у себя с IF, что не нравится в моей конструкции

klow · 15.05.2016 10:01:06

Ovenvan написал:
klow написал:
думаю не будет.
Кроме того ELSE IF! http://dev.mysql.com/doc/refman/5.7/en/ … unction_if
рекомендую использовать CASE http://dev.mysql.com/doc/refman/5.7/en/ … rator_case
Хорошо, case - вероятно хорошее решение, но мне бы хотелось всё-таки ещё разобраться у себя с IF, что не нравится в моей конструкции

это я немного погорячился. Исправил. см. мое предыдущее сообщение

Отредактированно klow (15.05.2016 10:02:44)

Ovenvan · 15.05.2016 10:05:02

Действительно нельзя использовать else if? хм.. не знал
Попробовал исправить по Вашим советам:

create procedure SigningContract(new_id int, new_sum int, new_begin date, new_individ int, 

new_comp int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

declare check1plus int;

set check1 = (select max(id_contract) from insurance_contract);

set check1plus = check1++;

if (select count(*) from insurance_contract where id_contract = new_id) != 0 then set new_id = check1plus;

elseif (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else insert into insurance_contract values(new_id, new_sum, new_begin, AddPeriod(new_begin), new_individ, new_comp, new_agent, new_object, new_categ);

end if;

end if;

end if;

end if;

end if;

end if;

end

Ошибка в том же первом условии(

klow · 15.05.2016 10:06:41

set check1plus = check1++;
Нужно
set check1plus = check1+1;
Кроме того, нужен только один end if;

klow · 15.05.2016 10:07:49

create procedure SigningContract(new_id int, new_sum int, new_begin date, new_individ int, 

new_comp int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

declare check1plus int;

set check1 = (select max(id_contract) from insurance_contract);

set check1plus = check1+1;

if (select count(*) from insurance_contract where id_contract = new_id) != 0 then set new_id = check1plus;

elseif (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else insert into insurance_contract values(new_id, new_sum, new_begin, AddPeriod(new_begin), new_individ, new_comp, new_agent, new_object, new_categ);

end if;

end

klow · 15.05.2016 10:15:30

Можно еще немного упростить

...

declare check1 int;

-- declare check1plus int;

set check1 = (select max(id_contract) from insurance_contract);

-- set check1plus = check1+1;

if (select count(*) from insurance_contract where id_contract = new_id) != 0 then set new_id = check1 + 1;

...

Отредактированно klow (15.05.2016 10:15:57)

Ovenvan · 15.05.2016 11:06:09

Благодарю Вас, удалил лишние END IF, всё скомпилировалось.

Но теперь возникла проблема алгоритмическая, в таблице контрактов у меня в полях Individual и Company могут быть значения null, в зависимости с кем контракт, а при моей проверке постоянно выводит "Такого ... не существует ", как работать с null проверкой?

Например с таким вызовом: call SigningContract(11, 500, '2016-05-15', 11,
null, 7, 11, 5)
Ошибка не существования клиента, а он есть

Отредактированно Ovenvan (15.05.2016 11:11:21)

vasya · 15.05.2016 11:10:18

is null

Ovenvan · 15.05.2016 11:17:04

Хм... я чего-то совсем не понимаю, вызов процедуры выполняется, но в таблицу ничего не добавляется

create procedure SigningContract(new_id int, new_sum int, new_begin date, new_individ int, 

new_comp int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

set check1 = (select max(id_contract) from insurance_contract);

if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1;

elseif (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else insert into insurance_contract(id_contract, sum_insured, beginning, ending, Individual, Company,  Agent,  Object, Category)

values(new_id, new_sum, new_begin, AddPeriod(new_begin), new_individ, new_comp, new_agent, new_object, new_categ);

end if;

end

call SigningContract(11, 500, '2016-05-15', 11, null, 7, 11, 5)

Отредактированно Ovenvan (15.05.2016 11:17:26)

vasya · 15.05.2016 11:27:15

if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1; -- присвоили значение
...
end if;
end -- закончили процедуру. Почему что-то должно добавиться?

klow · 15.05.2016 11:32:16

Посмотрите на условия. Например, смысла в set new_id = check1 + 1 первом IF вообще никакого нет. Оно все равно не будет использовано.
Возможно

if (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else 

  if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1 end if;

  insert into insurance_contract(id_contract, sum_insured, beginning, ending, Individual, Company,  Agent,  Object, Category)

  values(new_id, new_sum, new_begin, AddPeriod(new_begin), new_individ, new_comp, new_agent, new_object, new_categ);

end if;

Не успел.

Отредактированно klow (15.05.2016 11:33:19)

vasya · 15.05.2016 11:42:59

klow написал:
Не успел.

зато я только указал на проблему, а вы и решение предложили

Ovenvan · 15.05.2016 12:55:16

То есть процедура должна иметь вот такой вид?

create procedure SigningContract(new_id int, new_sum int, new_begin date, new_individ int, 

new_comp int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

declare periodplus int;

set check1 = (select max(id_contract) from insurance_contract);

set periodplus = (select Period from guide_services where id_guide = (select Category from insurance_contract where id_contract = new_id));

if (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else

  if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1 end if;

  insert into insurance_contract(id_contract, Sum_insured, Beginning, Ending, Individual, Company,  Agent,  Object, Category)

  values(new_id, new_sum, new_begin, AddPeriod(new_begin, periodplus), new_individ, new_comp, new_agent, new_object, new_categ);

end if;

end

Просто не будет ложью, если скажу, что запутался, вроде всё понял, а ошибку синтаксиса всё рано выдаёт

Отредактированно Ovenvan (15.05.2016 12:57:06)

klow · 15.05.2016 13:04:37

Мне сложно судить не зная деталей задачи, но думаю нужно немного подправить

create procedure SigningContract(new_id int, new_sum int, new_begin date, new_individ int, 

new_comp int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

declare periodplus int;

set check1 = (select max(id_contract) from insurance_contract);

if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1; end if; -- Перенес и исправил

set periodplus = (select Period from guide_services where id_guide = (select Category from insurance_contract where id_contract = new_id));

if (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from company where id_company = new_comp) = 0 then signal sqlstate '45000' set message_text = 'Такой компании не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from insurance_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else

--  if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1 end if;

  insert into insurance_contract(id_contract, Sum_insured, Beginning, Ending, Individual, Company,  Agent,  Object, Category)

  values(new_id, new_sum, new_begin, AddPeriod(new_begin, periodplus), new_individ, new_comp, new_agent, new_object, new_categ);

end if;

end

Отредактированно klow (15.05.2016 13:06:20)

Ovenvan · 15.05.2016 21:59:58

Спасибо Вам всем за помощь, немного переписал процедуру, теперь не знаю как решить ещё одну проблемку.

create procedure SigningContractClient(new_id int, new_sum int, new_begin date, new_individ int, new_agent int, new_object int, new_categ int)

begin

declare check1 int;

declare periodplus int;

declare ending date;

set check1 = (select max(id_contract) from insurance_contract);

set ending = AddPeriod(new_begin, periodplus);

if (select count(*) from insurance_contract where id_contract = new_id) = 0 then set new_id = check1 + 1; end if;

if (select count(*) from individual where id_individual = new_individ) = 0 then signal sqlstate '45000' set message_text = 'Такого клиента не существует';

elseif (select count(*) from insurance_agent where id_agent = new_agent) = 0 then signal sqlstate '45000' set message_text = 'Такого агента в нашем агентстве нет';

elseif (select count(*) from object_insurance where id_object = new_object) = 0 then signal sqlstate '45000' set message_text = 'Такой объект страхования не зарегистрирован';

elseif (select count(*) from guide_services where id_guide = new_categ) = 0 then signal sqlstate '45000' set message_text = 'Такая услуга невозможна';

else

insert into insurance_contract(id_contract, Sum_insured, Beginning, Ending, Individual, Company,  Agent,  Object, Category)

values(new_id, new_sum, new_begin, ending, new_individ, null, new_agent, new_object, new_categ);

end if;

set periodplus = (select Period from guide_services where id_guide = (select Category from insurance_contract where id_contract = new_id));

set ending = AddPeriod(new_begin, periodplus);

if (select Ending from insurance_contract where id_contract = new_id) is null then update insurance_contract set Ending = ending where id_contract = new_id; end if;

end

Есть функция AddPeriod(new_begin, periodplus), на входе у неё дата и количество лет для увеличение даты, на выходе новая увеличенная дата. Только вместо даты он вставляет null

rgbeast · 15.05.2016 22:05:32

А зачем функция?

 set ending = new_begin + INTERVAL periodplus YEAR;

Ovenvan · 15.05.2016 22:12:21

rgbeast написал:
А зачем функция?
set ending = new_begin + INTERVAL periodplus YEAR;

Понимаю Вашу заинтересованность, просто уже придумать что-то из функций более толковое не смог. Кстати, функция это и делает

create function AddPeriod(FirstDate date, newperiod int)

returns date

begin

declare newdate date;

set newdate = (select adddate(FirstDate, interval newperiod year));

return(newdate);

end

rgbeast · 15.05.2016 22:15:33

У Вас первый вызов

set ending = AddPeriod(new_begin, periodplus);

происходит до того, как переменной periodplus присвоено какое-либо значение. Поэтому результат NULL

Ovenvan · 15.05.2016 22:18:31

rgbeast написал:
У Вас первый вызов
set ending = AddPeriod(new_begin, periodplus);
происходит до того, как переменной periodplus присвоено какое-либо значение. Поэтому результат NULL

Да, соглашусь, и поэтому у меня поле Ending может быть null. Возможно ли в этой процедуре после INSERT перезаписать это значение?

Я вот попробовал в конце

set periodplus = (select Period from guide_services where id_guide = (select Category from insurance_contract where id_contract = new_id));

set ending = AddPeriod(new_begin, periodplus);

if (select Ending from insurance_contract where id_contract = new_id) is null then update insurance_contract set Ending = ending where id_contract = new_id; end if;

Но боюсь это уж слишком тупо)

rgbeast · 15.05.2016 22:26:12

Отлаживайте вручную по шагам.
В guide_services есть необходимая запись?
Что попадает в переменную periodplus?

Ovenvan · 15.05.2016 22:34:37

rgbeast написал:
Отлаживайте вручную по шагам.
В guide_services есть необходимая запись?
Что попадает в переменную periodplus?

Необходимая запись есть.
В guide_services содержится наименование услуги и её период страхования в виде обычного числа (количество лет), а в periodplus попадает как раз это число, которое используется в новом контракте.

Форум пользователей MySQL

#1 15.05.2016 09:30:15

Ошибка синтаксиса

#2 15.05.2016 09:46:57

Re: Ошибка синтаксиса

#3 15.05.2016 09:51:45

Re: Ошибка синтаксиса

#4 15.05.2016 09:56:00

Re: Ошибка синтаксиса

#5 15.05.2016 10:00:21

Re: Ошибка синтаксиса

klow написал:

#6 15.05.2016 10:01:06

Re: Ошибка синтаксиса

Ovenvan написал:

klow написал:

#7 15.05.2016 10:05:02

Re: Ошибка синтаксиса

#8 15.05.2016 10:06:41

Re: Ошибка синтаксиса

#9 15.05.2016 10:07:49

Re: Ошибка синтаксиса

#10 15.05.2016 10:15:30

Re: Ошибка синтаксиса

#11 15.05.2016 11:06:09

Re: Ошибка синтаксиса

#12 15.05.2016 11:10:18

Re: Ошибка синтаксиса

#13 15.05.2016 11:17:04

Re: Ошибка синтаксиса

#14 15.05.2016 11:27:15

Re: Ошибка синтаксиса

#15 15.05.2016 11:32:16

Re: Ошибка синтаксиса

#16 15.05.2016 11:42:59

Re: Ошибка синтаксиса

klow написал:

#17 15.05.2016 12:55:16

Re: Ошибка синтаксиса

#18 15.05.2016 13:04:37

Re: Ошибка синтаксиса

#19 15.05.2016 21:59:58

Re: Ошибка синтаксиса

#20 15.05.2016 22:05:32

Re: Ошибка синтаксиса

#21 15.05.2016 22:12:21

Re: Ошибка синтаксиса

rgbeast написал:

#22 15.05.2016 22:15:33

Re: Ошибка синтаксиса

#23 15.05.2016 22:18:31

Re: Ошибка синтаксиса

rgbeast написал:

#24 15.05.2016 22:26:12

Re: Ошибка синтаксиса

#25 15.05.2016 22:34:37

Re: Ошибка синтаксиса

rgbeast написал:

Board footer