Почему неправильно идет запрос в mysql?

Question

[vellitonrus]

Написал простенький скрипт для апдейта определенной колонки базы на единицу меньше от того числа которое было.

<?php

$sql = mysql_query("select * from item");
      
   while($item = mysql_fetch_array($sql)){ 
	    echo $item['updays'];
		$day=$item['updays']-1;
		echo $day;
		mysql_query("UPDATE `item` SET `updays` = '".$day."' where id = '".$item['id']."'");

		
		}
?>

Но в базу, заносится число не на 1 меньше а на 2, тоесть если число $item['updays']=10, то в базу после операции удаления заносится число 8, а не 9. В чём может быть подвох?! P.S. Принт переменной $day после операции удаления показывает, что чисто стало меньше на 1, тоесть было 10 стало 9, однако в базу забивается 8. В базе 4 товара)

Answer 1

[Rsa97]

UPDATE `items` SET `updays` = `updays`-1
и всё, не надо никаких циклов

Comments

[Filipp Shcherbanich]

выше уже написал :)

[vellitonrus]

@Rsa97 это то конечно хорошо, но есть еще один нюанс, если чисто становиться меньше 5 к примеру, то нужно добавлять данные в другую колонку, то есть всё равно нужен цикл для проверки переменной $item['updays']

[Filipp Shcherbanich]

без цикла можно обойтись и в этом случае

[vellitonrus]

@Rsa97 и даже без цикла удаляет не 1, а 2. и оставляет после числа 100,число 98, а не 99.

[Rsa97]

@vellitonrus Выдайте запрос не из скрипта, а из консоли mysql. Если всё равно уменьшится на 2, то возможно есть триггер на изменение строк. Если нет - то скорее всего скрипт запускается два раза.

Answer 2

[Петруша Укропов]

mysql_query("UPDATE item SET updays = updays-1 where id = '".$item['id']."'");

Comments

[vellitonrus]

синтаксис нарушен!

[vellitonrus]

и всё равно убирает по 2 а не по 1

[vellitonrus]

@artishok разобрался, просто запустил скрипт напрямую а не через редирект движка сайта.

Answer 3

[Filipp Shcherbanich]

не надо в цикле писать запросы, лучше сформировать его тело, а потом 1 раз выполнить

Comments

[vellitonrus]

а как это оформить кодом? просто не так силён в php. ведь мне нужно чтобы каждая строка делала апдейт.

[Filipp Shcherbanich]

$sql = mysql_query("select * from item");

while($item = mysql_fetch_array($sql))
$query[] = '('.($item['updays']-1).','.$item['id'].')';

mysql_query("
INSERT INTO
`item`
(
`updays`,
`id`
)
VALUES ".implode(',',$query)."
ON DUPLICATE KEY
UPDATE
`updays` = VALUES(updays);
");

[vellitonrus]

допустим у меня 1000 записей и мне нужно у всей 1000 изменить колонку updays на значения на единицу меньше. не понимаю, как это оформить.

[Filipp Shcherbanich]

ну сделай выборку не всех записей, а только нужных

[Filipp Shcherbanich]

ещё можно условие в UPDATE написать, чтобы был 1 запрос

[vellitonrus]

Вы не поверите, но всё равно делает удаление 2х единиц. Эффект тот же.

[Filipp Shcherbanich]

mysql_query("UPDATE item SET updays = updays-1");
в твоём случае всё вообще вот так

[vellitonrus]

С вашим скриптом...

[Filipp Shcherbanich]

@vellitonrus запрос абсолютно верный, предположу что это всё находится в цикле, поэтому несколько раз одно и то же действие выполняется, внимательнее посмотрите код.

[vellitonrus]

На второй скрипт тот же эффект, не помогает, может нужно какой то определенный тип таблицы в базе установить у меня стоит int(2) ну тоесть на 2 символа.

[vellitonrus]

@scherbanich ну так вы сами видите код, там впринцепе не может быть нигде ошибки.

[Filipp Shcherbanich]

@vellitonrus ну этот код верный, может там триггер где-то стоит или ещё что, может этот скрипт подключается куда несколько раз

[Rsa97]

@vellitonrus Если int(2), то 100 в базу не запишется, обрежется до максимально возможного - 99.

Answer 4

[Melkij]

Скрипт каким образом запускается?
Удостоверьтесь в том, что знаете, сколько именно раз он запускается.
Это можно сделать, например, так:

file_put_contents('dbupd.txt', date('Y-m-d H:i:s').' run'.PHP_EOL, FILE_APPEND);