Как оставить значение в базе без изменений если null?

Question

[Виктор]

Как оставить значение в базе без изменений если null?

$query = "UPDATE base SET  test=:test ";
$database->bindValue(':test', (Session::user_role == 'admin') ? $data['test'] : null);

Если null и test например условно имеет значение 100, то оставить 100
В данном примере значение меняется на null

Comments

[Adamos]

Дичь какая-то. Сам в коде определяет значение null, зачем-то пишет его в базу и удивленно спрашивает, как этого не делать.

[Виктор]

Adamos, Код для примера, вот вопрос Если null и test например условно имеет значение 100, то оставить 100
тоесть если Если null и test например условно имеет значение 100 то оставить 100

[Akina]

UPDATE base 
SET test = CASE WHEN test IS NULL
                THEN NULL
                ELSE :test
                END

Если надо оставить не NULL, а определённое значение или список значений, то

UPDATE base 
SET test = CASE WHEN test IN ( {значение или список} )
                THEN test
                ELSE :test
                END

[Михаил]

Akina, ему надо не так, ему надо если параметр имеет значение NULL, то оставить значение поля
CASE WHEN :test IS NULL THEN test ELSE :test END
Или, проще:
UPDATE base SET test = IFNULL(:test, test)

[Adamos]

Если в коде уже известно, что поле не нужно будет менять - логично в коде же и предусмотреть ветку, не вставляющую в запрос это изменение. Или не делающее запрос вовсе, если в этом нет смысла. А сбегать в базу, чтобы ничего не сделать - это говнокод.

[Akina]

Михаил,

ему надо если параметр имеет значение NULL, то оставить значение поля

Тогда всё проще - SET test = COALESCE(:test, test). Ну или как у тебя, IFNULL (не люблю эту функцию - и не стандарт, и всё равно маппится на COALESCE).

Adamos,

сбегать в базу, чтобы ничего не сделать - это говнокод.

+100500

[Виктор]

Adamos, Некоторые поля меняются в зависимости кто вводит данные, тоесть в этом примере поле тест может менять только админ, остальные поля и пользователи и админ

[Adamos]

Виктор, это ничего не меняет.
Логично СНАЧАЛА собрать данные, а ПОТОМ из них уже делать запрос, ЕСЛИ он вообще имеет смысл. Это позволит через год, разбираясь в этом коде, не крыть трехэтажно того, кто такого наворотил. Стыдливо вспоминая, что сам же и обосрался.

[Виктор]

Akina,

сбегать в базу, чтобы ничего не сделать - это говнокод.

SET test = COALESCE(:test, test) - это одна строчка, а менять и ставить условия в коде (прием, валидация), форме, js, и тд не знаю я не спец, это форма редактирования

Answer 1

[Михаил]

UPDATE base SET test = IFNULL(:test, test)

Comments

[Slava Rozhnev]

Просто онлайн-тест