Запрос-то правильный. Может правда поспать? :)

UPD
Тут можно сэкономить. Конечно зависит от структуры твоей БД, но обычно можно определить некое уникальное поле (или составной уникальный индекс), и просто делать запрос на вставку.
Потом можно проверить так:

if(mysql_query("INSERT INTO ... SET ...")) {
     echo 'запись добавлена';
} else {
     if(mysql_errno() == 1062) {
          echo 'запись уже существует';
     } else {
          exit('Произошла ошибка: ' . mysql_error());
     }
}
// или так:
mysql_query("INSERT INTO ... SET ...");
$status = mysql_errno();
if($status == 0) {
     echo 'запись добавлена';
} elseif($status == 1062) {
     echo 'запись уже существует';
} else {
     exit('Произошла ошибка: ' . mysql_errno());
}

Про коды ошибок можно почитать тут.

SlavaFr
Нет, так не пойдёт. $myrow используется мной и дальше в скрипте. Там такое не подходит.
Без очков, special for you

И всё равно не работает и всё равно не понимаю почему. Ррррр

Как бы я дебужил (от слова debug) или повесть о программисте.

Итак, для начала вывести результирующий запрос, примерно так:

$query = "SELECT * FROM `bookmarks` WHERE `book_id` = '" . $_GET['book'] . "' AND `user_id` = '" . $_SESSION['id']."'";
echo $query;

После этого выполнить такой запрос через шел или phpmyadmin (или что у Вас под рукой)

Затем, зачем берете в одинарные кавычки значения для полей book_id и user_id? Это ведь наверняка числа?

Потом, надо остерегаться злобных хацкеров и проверять все, что приходит через $_GET (и другие)

ну тогда зделай так

$myrow = mysql_query($zapros="SELECT * FROM `bookmarks` WHERE `book_id` = '".$_GET['book']."' AND `user_id` = '".$_SESSION['id']."'") or die(mysql_error());
echo $zapros;
		if (mysql_num_rows($myrow) == 0) {
			$ins = mysql_query($insert="INSERT INTO `bookmarks` (`book_id`,`user_id`) VALUES ('".$_GET['book']."', '".$_SESSION['id']."')");
echo $insert;
		}

и теперь покажи запрос который выдает "echo $zapros" и результаты которые в этот запрос по твоему мнению не вписываются.

SQL запрос выдаёт такую вот замарочку:
#1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INSERT INTO `bookmarks` (`book_id`,`user_id`) VALUES ('2', '47') LIMIT 0, 30' at line 2

Сказать по правде, я до сих пор не очень понимаю, в чём загвоздка. Значения принимает правильные...

waldicom
Всё верно, а числа нельзя в одинаковые кавычки брать?

Ну, в плане защиты от подлецов у меня пока полная прорва. Радует, что защищать по сути ещё и нечего. А вообще надо, конечно.

SlavaFr

waldicom
Вообще я лимит не вводил, но это не помешало ему появиться в окончательном запросе почему-то.
Но дело не в этом. Дело в том, что для отдельно взятого значения user_id может быть море записей. А вот для отдельно взятой book_id - одна. Поэтому стоит мне удалить запись, как пользователь, который ещё недавно никак не мог вставить запись, тут же без проблем это делает. Короче говоря book_id судя по всему у меня уникальный. Хотя вроде бы я его таким не делал. Пересоздал таблицу - помогло... мне точно надо больше спать. Всем спасибо!