Использование подстановки переменный напрямую в строку запроса - это очень плохая практика. С одной из проблем, порождаемых таким подходом, которая заключается в трудночитаемости запроса (и как следствие получение проблем с отладкой на пустом месте) вы столкнулись лично. Главная же проблема в том, что создавая запрос подобным образом, необходимо в параноидальном режиме проверять значения подставляя переменных, чтобы не получить как минимум неработоспособный запрос и как максимум SQL-инъекцию.
Гораздо более правильно и безопасно - это использовать подготовленные запросы.
В частности ваш код, по хорошему, должен выглядеть так:
$link = new mysqli('bla-bla-bla');
$sql = "UPDATE dialogs set last_u=?, last_m=?, `status`=1, `count`=`count`+1 WHERE id=?"
$stmt = $link->prepare($sql);
$stmt->bind_params('sss', $uid, $text, $id);
$dia = $stmt->execute();
Немного более многословно, зато:
- В подавляющем большинстве случаев гарантирует безопасность подстановки значений в запрос
- На порядок нагляднее
- Если вам в процессе работы скрипта потребуется запустить несколько таких запросов, отличающихся только значением параметров - будет достаточно один раз подготовить запрос, а потом просто менять забинденных значения переменных и вызывать
$stmt->execute()
. Ну и работать это будет, теоретически, быстрее.
Вообщем, самым простым решением была окантовка названий столбцов кавычками ``
В вашем случае достаточно было убрать в обратные кавычки поле count
, так как COUNT
является частью синтаксиса SQL. (Ну и, возможно, status
, но вряд ли)
$sql='Текст запроса'; $query=mysqli_query($link,$sql);
, и между операторами выведите полученный текст после подстановок. Сразу увидите, в чём проблема. Или показывайте тут текст запроса в том виде, в каком он отдаётся MySQL-серверу.var_dump($query, mysqli_errno);
после и в вопрос структуру таблицы добавьте