Как увеличивать инкремент, если есть поле unique ?

Question

[VaVa_VooM]

Есть таблица page(id, url). Пишу в бд из массива, где сами элементы уникальны, но они могут быть не уникальными относительно бд. В бд поле url уникально. Но делая следующий запрос

$sql = "INSERT IGNORE page (id, url, title) VALUES ";
			foreach ($links as $value) {
				$sql .= "(".$id.", '".$value[0]."', '".addslashes($value[1])."'), ";
				++$id;
			}
			$sql = rtrim($sql, ", ");

он увеличивает счетчик даже тогда, когда не добавил элемент. Как можно это исправить ?

Comments

[issssrt]

перед запросом рекрусивно проверяйте есть ли такой Id в бд. А вообще неправильно так делать

[VaVa_VooM]

issssrt: Учусь. А как верно ?

Answer 1

[Армянское Радио]

Первая огромная проблема - вы строите запрос конкатенацией, а значит подвергаете себя риску SQL-инъекций. Используйте PDO.

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

Если вы столь заинтересованы в отсутствии утечки номеров (хоть это и не верно с точки зрения архитектуры), вам придется (серьезно потеряв производительность) проверять перед вставкой каждой строки ее возможность наличия в БД.

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

Comments

[VaVa_VooM]

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

[Армянское Радио]

VaVa_VooM: нужно учиться сразу делать правильно и без костылей. И ничего заоблачного в PDO нет.

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

У ignore кстати есть другие эффекты, помимо игнорирования ошибок уникальности при вставке. Это - костыль и использовать его - плохая практика.

[VaVa_VooM]

Армянское Радио: Я об этом читал вот только вчера. Что суррогатные ключи лучше не использовать. Но таки, подскажите, как можно в бд или в самом запросе указать ему то, что он должен делать автоинкремент только тогда, когда он добавил реально запись, а не в случае попытки.

[Армянское Радио]

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

[FanatPHP]

Армянское Радио: вот удивительно это стремление удовлетворить незнайку. И ради этого нагородить громоздкое и неудобное решение несуществующей проблемы. Откуда эта субмиссивность у отвечателей - загадка для меня. Неужели несчасная галочка принятого овета н а с т о л ь к о притягательна, что заставляет забыть и здравый смысл и профессиональную гордость?

[Армянское Радио]

FanatPHP: Профессионал отличается от любителя (фаната?) тем, что решает задачу, а не оценивает ее, пусть даже для этого приходится снять белые перчатки и натянуть болотные сапоги.

[FanatPHP]

Печально видеть такую деградацию. Слово "думать" во всем этом пафосном тексте не прозвучало ни разу. И то правда, "что тут думать - тут прыгать надо" - реплика такого же "прафесианала".

Answer 2

[FanatPHP]

Первое, что надо узнать - ЗАЧЕМ тебе это нужно - " делать автоинкремент только тогда, когда он добавил реально запись"? Какая тебе разница, какое значение у этого поля?

Отвечать подробно

Answer 3

[Назар Мокринский]

1) За addslashes() вместо mysql_real_escape_string() руки выкручивать надо
2) Что за $id? Используйте автоинкремент в таблице на уровне БД, БД будет сама его увеличивать при вставке