Нужно добавить или обновить запись при определенных условиях

[evshi]

Мне надо проверить, если в таблице user_history у uid (user id) более 10 записей, то найти самую старую по dt, и на место ее записать новую. Если же записей у uid < 10, то сделать insert.

Вот структура таблицы:
CREATE TABLE `user_history` (
`id` int(11) NOT NULL auto_increment,
`uid` int(11) default NULL,
`objId` int(11) default NULL,
`objType` int(11) default NULL,
`dt` datetime default NULL,
PRIMARY KEY (`id`)
) ENGINE=MyISAM DEFAULT CHARSET=cp1251;


Пытаюсь сделать через IF:
SELECT
IF(
((SELECT COUNT(*) FROM `user_history` WHERE `uid`=3)>10),
1,
(INSERT INTO `user_history` (`uid`) VALUES(99))
)
Но уже на этом шаге вываливается ошибка синтаксиса. Как правильно решить данную задачу?

 

[Gas]

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

 

[Dovg]

Задачка - обычный стек FIFO. Сделайте стандартно и будет вам счастье:
- добавить запись
- удалить первую, если записей больше 10

+ завернуть в транзакцию.

 

[evshi]

Чтобы удалить лишние, сначала надо посчитать кол-во записей. И если их >10, удалить, все равно IF напрашивается.

 

[Dovg]

Чтобы удалить лишние, сначала надо посчитать кол-во записей. И если их >10, удалить, все равно IF напрашивается.

delete ... where id not in (select id from ... limit 10);

 

[Dovg]

>Можно без подзапросов:
>Эээ, а разве MyISAM поддерживает транзакции?
Извините, привычка.

В альтернативной субд транзакции есть всегда и нельзя сделать delete с лимитом.

 

[evshi]

Можно без подзапросов:
delete from table where uid =... order by dt limit 1

В подзапросе limit не работает. Так как же быть-то

 

[evshi]

Какой ещё подзапрос?

Вот здесь delete ... where id not in (select id from ... limit 10); - поздрапрос
А вот как тут delete from table where uid =... order by dt limit 1, я что-то не совсем понял После = в данном случае разве можно select указывать?

 

[evshi]

Выше я написал:
- добавить
- удалить первую
Запрос "delete..." удаляет первую запись в стеке - что тут не понятно?

Прошу прощения, мне показалось, что в запросе не uid =..., а id=.... Отсюда и путаница вышла.
Спасибо.

 

[evshi]

Хотя секундочку...., мне же надо хранить последние 10 добавленных записей. Если я буду удалять, не проверяя кол-ва, то у меня все время будет одна запись.

И кстати, обнаружил сложность №2: мне надо у каждого uid хранить только уникальные сочетания objId + objType, т.е. для одного uid связка objId + objType повторяться не должна. Я сделал уникальный ключ на uid + objId + objType. Но в случае, если insert пытается добавить уже существующую связку, запись не добавляется, но mysql ругается. Как можно избежать появления ошибок?

 

[Вурдалак]

Господи, какое же говно советуют новичку.

 

[evshi]

А если нужно в стеке оставить только N записей после добавления, то без подзапроса не обойтись. Что-то вроде:
delete t1 from t1
INNER JOIN (select id from t1 order by id desc limit 10, 100) as t
where t1.id = t.id;

Спасибо, заработало, а с условием удаления старой и добавления новой ошибки, видимо, можно действительно избежать, но все же решение получилось не очень красивым по двум причинам:
1. При таком решении значения столбца id все время растут. При большом количестве запросов получится нехорошо
2. С точки зрения логики все же странно сначала добавить, а потом удалить. Правильнее было бы сделать update.

Но все равно спасибо - это лучше, чем ничего.

 

[grigori]

внутри базы update реализован как insert+delete

какие нафиг подзапросы, просто
1. insert ...
2. select count(*) where user_id=...
3. if ($count>10) select id ... order by dt asc limit $count-10
4. $ids = implode(',',$result);
5. "delete ... where id in (".$ids.")"

все эти хаки под mysql, подзапросы и т.п. надо юзать когда знаешь, что так надо

 

[evshi]

Какой-то mysql негибкий получается В итоге сделал кучу запросов но работает:

function updateUserHistory($objId, $objType, $uid){
	if ($objId AND $objType AND $uid) {
		mysql_query("SELECT `id` FROM `user_history` WHERE `uid`=$uid");
		$rowQnt=mysql_affected_rows();
		if ($rowQnt>9) {
			if ($rowQnt<11) {
				//надо проверить, встречается ли ключ $uid-$objId-$objType?
				$dblKeyId=mysql_fetch_array(mysql_query("SELECT `id` FROM `user_history` WHERE `uid`=$uid AND `objId`=$objId AND `objType`=$objType LIMIT 1"));
				if ($dblKeyId[id]) {//если нашел, то делаю update `dt`=NOW() у этой записи
					mysql_query("UPDATE `user_history` SET `dt`=NOW() WHERE `id`=".$dblKeyId[id]." LIMIT 1");
				}else{//если не нашел, делаю update самой старой записи
					$oldestId=mysql_fetch_array(mysql_query("SELECT `id` FROM `user_history` ORDER BY `dt` LIMIT 1"));
					mysql_query("UPDATE `user_history` SET `uid`=$uid, `objId`=$objId, `objType`=$objType, `dt`=NOW() WHERE `id`=".$oldestId[id]." LIMIT 1");
				}
			}else alertMe("udateUserHistory: rowQnt >10");
		}else{
			mysql_query("INSERT INTO `user_history`	(`uid`, `objId`, `objType`, `dt`) VALUES ($uid, $objId, $objType, NOW()) ON DUPLICATE KEY UPDATE `dt`=NOW()");
		}
	}
}

 

[grigori]

фигню ты написал, читай доку по mysql_affected_rows и mysql_num_rows

алгоритм у тебя хреновый, но 2й раз объяснять не буду

 

[Вурдалак]

Стеком называют LIFO. FIFO — очередь.

 

[Вурдалак]

Chusha, сам алгоритм уже расписал grigori.

Хотел бы донести до пионера в MySQL, что там есть INSERT IGNORE. Через «собаку» подавляют ошибку только мудаки.

 

[dr-sm]

В альтернативной субд транзакции есть всегда и нельзя сделать delete с лимитом.

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

 

[Вурдалак]

Chusha, я говорю не ему, а тебе.

 

[antson]

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

В предложенной постановке операция логирования становиться неоправданно тяжелой.
Я бы сделал простые инсерты и при просмотре просто добавил лимит 10 записей.
А чистку базы вынес на крон раз в сутки в 3 часа ночи сразу после бекапов .