MYSQL — Почему update обновляет два значения вместо одного?

Question

[w_b_x]

Доброго времени суток!

Есть у меня PHP-код. Он находит в БД самую "старую" запись, а именно запись у которой в столбце "last" типа datetime самая старая дата стоит в сравнении с другими. Берёт ID этой записи и обновляет её дату на нынешнюю.

Однако как я не крутил этот код и не пытал, он меняет дату не у одной записи, а у всех что подходят под описание status=1. Как это исправить и почему это происходит?

//Получаем из БД самую старую запись
$sql1 = mysql_query("SELECT `id` FROM advert WHERE status=1 ORDER BY last LIMIT 1"); 
//Проверяем есть ли вообще записи подходящие под описание
if(mysql_num_rows($sql1) != 0){
//Создаем ассоциативный массив найденной записи
$adata = mysql_fetch_assoc($sql1);
//Задаем переменную с нынешним временем
$now = date("Y-m-d H:i:s");
//Меняем дату у найденной записи
mysql_query("UPDATE advert SET last = '".$now."' WHERE id=".$adata['id']);
}

P.S. Если выполняю запрос "SELECT `id` FROM advert WHERE status=1 ORDER BY last LIMIT 1" через PhpMyAdmin то получаю одну единственную запись, которая мне и нужна.

Comments

[Макс]

id уникально? SELECT * FROM advert WHERE id= что говорит?

[ldv]

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

Answer 1

[w_b_x]

Выполнил ваш запрос через SQL - всё правильно работает. Как только делаю через php:

mysql_query("UPDATE `advert` 
    SET `last` = NOW() 
    WHERE `status` = 1 
    ORDER BY `last` 
    LIMIT 1");

Меняет у двух сразу 0_о. Что не так?

Comments

[ldv]

w_b_x: как вызываете скрипт? Что вокруг приведенного блока? Код явно выполняется несколько раз.

[w_b_x]

Да, разобрался, никак не мог заподозрить один элемент скрипта из-за которого скрипт выполнялся два раза. в .htaccess у меня прописано, что если путь не найден, то выполнять вот этот самый скрипт. А в самом скрипте я вставляю картинку, которой нету (в базе данных image) из-за чего скрипт выполнялся два раза в скрытом режиме.

Answer 2

[Поликарп Занудский]

Это делается в один SQL-запрос:

UPDATE advert
SET last = NOW()
WHERE id = (
    SELECT id
    FROM advert
    WHERE status = 1
    ORDER BY last
    LIMIT 1
)

Comments

[Макс]

если id не уникально - не прокатит

[Поликарп Занудский]

Макс: А в чем тогда смысл "Id"entificator-а? Он, априори, должен быть уникален.

[Макс]

Поликарп Занудский: к сожалению БД об этом законе природы не знает. И позволяет обозвать полем id что угодно. И сделать его не уникальным, если такова воля DBA.

Answer 3

[Rsa97]

А просто сделать?

UPDATE `advert` 
    SET `last` = NOW() 
    WHERE `status` = 1 
    ORDER BY `last` 
    LIMIT 1;

Comments

[w_b_x]

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