В чем ошибка SQL запроса?

Question

[Maxim Plotnikov]

Доброго всем дня. Написал sql запрос для обновления в базе данных. Сам он выглядит так:

DECLARE coins_take int(11);
SET coins_take = 1;
UPDATE users 
SET users.coins_ready_to_collect = CASE 
        WHEN users.farmboost_buy='Yes' THEN users.coins_ready_to_collect + coins_take * 2
        WHEN users.farmboost_buy='No' THEN users.coins_ready_to_collect + coins_take
      END 
WHERE users.id > 0;

Но при его добавлении база данных ругается и ничего не происходит. Где ошибка, подскажите.
Заранее спасибо!
P.S. скриншот ошибки

Comments

[Adamos]

В запросе зачем-то использован CASE вместо банального IF и указана таблица перед каждым полем, хотя работа идет с одной таблицей.

[Maxim Plotnikov]

Adamos, с CASE все работает стабильно, но я добавил переменную, и вот из за неё мне выдает ошибку. Почему?

[Adamos]

Maxim Plotnikov, потому что переменные задаются с собакой, полагаю.

[Maxim Plotnikov]

Adamos, и с собакой тоже ошибка.

DECLARE @coins_take int(11);
SET @coins_take = 1;
UPDATE users 
SET users.coins_ready_to_collect = CASE 
        WHEN users.farmboost_buy='Yes' THEN users.coins_ready_to_collect + @coins_take * 2
        WHEN users.farmboost_buy='No' THEN users.coins_ready_to_collect + @coins_take
      END 
WHERE users.id > 0;

[Adamos]

Maxim Plotnikov, а в DECLARE - без. Эта строка вообще зачем-то здесь нужна?

[Maxim Plotnikov]

Adamos, это нужный параметр. Убрать не могу.

Answer 1

[Rsa97]

DECLARE is permitted only inside a BEGIN ... END compound statement and must be at its start, before any other statements.

https://dev.mysql.com/doc/refman/8.0/en/declare.html
Ну и не мешало бы привести текст ошибки.

Comments

[Maxim Plotnikov]

обновил вопрос, посмотрите.

[Rsa97]

Maxim Plotnikov, Дык блока BEGIN ... END у вас нету, а вне такого блока DECLARE использовать нельзя.

[Maxim Plotnikov]

Rsa97, вот добавил, но ошибка повторяется.

BEGIN
DECLARE @coins_take int;
SET @coins_take = 1;
UPDATE users 
SET users.coins_ready_to_collect = CASE 
        WHEN users.farmboost_buy='Yes' THEN users.coins_ready_to_collect + @coins_take * 2
        WHEN users.farmboost_buy='No' THEN users.coins_ready_to_collect + @coins_take
      END 
WHERE users.id > 0
END;

[Adamos]

"Пацак думает на языках, окончания которых не знает".

[Maxim Plotnikov]

Adamos, спасибо за помощь.

[Rsa97]

Maxim Plotnikov, Уберите @ у переменной.

[Maxim Plotnikov]

Rsa97, почему то ругается на END, хотя я сделал по примеру из документации.

BEGIN
DECLARE coins_take int;
SET coins_take = 1;
UPDATE users 
SET users.coins_ready_to_collect = CASE 
        WHEN users.farmboost_buy='Yes' THEN users.coins_ready_to_collect + coins_take * 2
        WHEN users.farmboost_buy='No' THEN users.coins_ready_to_collect + coins_take
      END 
WHERE users.id > 0
END;

[Rsa97]

Maxim Plotnikov, Потому что у вас UPDATE не завершается точкой с запятой.

[Adamos]

Maxim Plotnikov, а заодно BEGIN, DECLARE и END на хрен не нужны, а в переменной (НО НЕ В DECLARE!!! - так видно?) таки должна быть собачка.

[Maxim Plotnikov]

Rsa97, оооооо, заработало. Спасибо, очень благодарен.

[Vitsliputsli]

Maxim Plotnikov, так ведь coins_take нигде не изменяется в приведенном коде, зачем ее объявлять переменной?
По-сути это тоже самое, что и:

UPDATE users 
SET users.coins_ready_to_collect = CASE 
        WHEN users.farmboost_buy='Yes' THEN users.coins_ready_to_collect + 2
        WHEN users.farmboost_buy='No' THEN users.coins_ready_to_collect + 1
      END 
WHERE users.id > 0

[Adamos]

Vitsliputsli, да весь этот CASE заменяется однострочным IF, но раз уж человеку приспичило поучиться именно этой конструкции...