MySQL. Хранимая процедура зависает при выполнении

[lawliet93]

Приует. Работаю в phpmyadmin, написал процедуру, она проверяет некоторые поля некоторой таблицы на заянятость. если первое поле свободно. то в него заносятся данные и все, если это поле зайнято. то проверяется следующий столбец. Так вот, когда выполняю эту процедуру в phpmyadmin, то все зависает, никаких сообщений об ошибке или о удачном выполнении нету, просто надпись - выполняется запрос, или типа того (у меня на украинском все), так вот оно выполняется очень долго, а в поля в таблицах никак не изменяются. Что в этой процедуре не так?

DROP PROCEDURE `insert_new_char`;
CREATE DEFINER=`root`@`localhost` PROCEDURE `insert_new_char`(IN `nick` VARCHAR(20), IN `idd` INT(11)) NOT DETERMINISTIC NO SQL SQL SECURITY DEFINER BEGIN
SELECT @n:=num FROM id_and_char_id WHERE id=idd;
IF @n<5 THEN
SELECT @var1:=ch1 FROM id_and_char_id WHERE id=idd;
SELECT @var2:=ch2 FROM id_and_char_id WHERE id=idd;
SELECT @var3:=ch3 FROM id_and_char_id WHERE id=idd;
SELECT @var4:=ch4 FROM id_and_char_id WHERE id=idd;
SELECT @var5:=ch5 FROM id_and_char_id WHERE id=idd;
IF @var1!=0 THEN
UPDATE id_and_char_id SET ch1=nick WHERE id=idd;
UPDATE id_and_char_id SET num=@n+1 WHERE id=idd;
#INSERT INTO chars (nick,spawn_x,spawn_y,_spawn_z) VALUES (nick,0,0,0);
ELSEIF @var2 !=0 THEN
UPDATE id_and_char_id SET ch2=nick WHERE id=idd;
UPDATE id_and_char_id SET num=@n+1 WHERE id=idd;
#INSERT INTO chars (nick,spawn_x,spawn_y,_spawn_z) VALUES (nick,0,0,0);
ELSEIF @var3 !=0 THEN
UPDATE id_and_char_id SET ch3=nick WHERE id=idd;
UPDATE id_and_char_id SET num=@n+1 WHERE id=idd;
#INSERT INTO chars (nick,spawn_x,spawn_y,_spawn_z) VALUES (nick,0,0,0);
ELSEIF @var4 !=0 THEN
UPDATE id_and_char_id SET ch4=nick WHERE id=idd;
UPDATE id_and_char_id SET num=@n+1 WHERE id=idd;
#INSERT INTO chars (nick,spawn_x,spawn_y,_spawn_z) VALUES (nick,0,0,0);
ELSEIF @var5 !=0 THEN
UPDATE id_and_char_id SET ch5=nick WHERE id=idd;
UPDATE id_and_char_id SET num=@n+1 WHERE id=idd;
#INSERT INTO chars (nick,spawn_x,spawn_y,_spawn_z) VALUES (nick,0,0,0);
END IF;
END IF;
END

[Stilet]

то все зависает

Может на запись ставится блокировка?
Разве нельзя вместо множества select сделать один типа:

SELECT 
@var1:=ch1, 
@var2:=ch2, 
@var3:=ch3, 
@var4:=ch4, 
@var5:=ch5 
FROM id_and_char_id WHERE id=idd;

?

UPDATE id_and_char_id SET num=@n+1 WHERE id=idd;

Зачем это в каждом условии? Убери. Вынеси в конец первого IF

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

[lawliet93]

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

Может на запись ставится блокировка?
Разве нельзя вместо множества select сделать один типа:

SELECT 
@var1:=ch1, 
@var2:=ch2, 
@var3:=ch3, 
@var4:=ch4, 
@var5:=ch5 
FROM id_and_char_id WHERE id=idd;

?

Зачем это в каждом условии? Убери. Вынеси в конец первого IF

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

Структура чего?

[Аватар]

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

Согласен с Виталием - проблема определенно есть со структурой таблиц

[lawliet93]

Сообщение от Аватар

Согласен с Виталием - проблема определенно есть со структурой таблиц

тогда помогите составить более подходящую структуру. Сейчас у меня такая структура.
Есть таблицы logins, id_and_char_id, chars. При регистрации в таблицу logins заносятся логин и пароль, и добавляется id который с автоинкрементом, этот же id сразу же заносится в таблицу id_and_char_id. Если пользователь хочет создать персонажа, вызывается не работающая процедура, которая проверяет счетчик персонажей в таблице id_and_char_id (макс - 5), если он меньше пяти, то в первое свободное поле (char1,char2...) заносится ник нового перса, после этого этот же ник заносится в таблицу chars. Я наваял недоструктуру с недоблоксхемой. (картинка во вложениях)
Я-то новичек в проэктировании БД, как не трудно догадатся. По-этому насоветуйте советов, как лучше составить БД для решения вышеописанных задач? Спасибо заранее!

[Stilet]

Ну так сделай таблицу
[idЮзера] [Номер перса]
Туда заноси нового персонажа, проверяя

Если (select count(Номер перса) from этатаблица) <5 то
insert нового персонажа
inset [idЮзера] [Номер перса] в эту таблицу

И вместо пяти полей у тебя будет пять записей, отбираемых по idЮзера

[lawliet93]

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

Ну так сделай таблицу
[idЮзера] [Номер перса]
Туда заноси нового персонажа, проверяя

Если (select count(Номер перса) from этатаблица) <5 то
insert нового персонажа
inset [idЮзера] [Номер перса] в эту таблицу

И вместо пяти полей у тебя будет пять записей, отбираемых по idЮзера

а для ников надо еще одну таблицу? [idЮзера] [НикПерса] ?

[Stilet]

Я думаю достаточно будет переделать твою id_and_char_id, поскольку от нее все равно не много толку.

[lawliet93]

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

Я думаю достаточно будет переделать твою id_and_char_id, поскольку от нее все равно не много толку.

тогда я не допер, куда мне ники пихать? вы имели ввиду таблицу [idЮзера] [Номер перса] [Ник] или [idЮзера] [Номер перса] ??

[Stilet]

Я всего лишь предлагал тебе на каждый новый персонаж заводить не новое поле а новую запись. А как ты уже это заложишь в программу тебе решать.