Почему не работает UPDATE php?

Question

[js_123]

Вечер добрый
Начал изучать работу с sql посредством php и столкнулся с проблемой
Код ниже работает исправно (передаю значение через ajax)

$sql = "UPDATE main_test1 SET name = 1234 WHERE id = 2;";

А вот код ниже не работает

$test = $_POST["test"];
$sql = "UPDATE main_test1 SET name = $test  WHERE id = 2;";

В чём может быть проблема?

Answer 1

[Игорь]

Коллега, ваш код относительно правильный и в некоторых случаях он будет работать, в некоторых нет.
Нельзя сказать что это не рабочий вариант.

Когда вы составляете такой запрос вы подвергаете себя риску.
Ответьте себе на вопрос, а что произойдёт, если в переменной $test будет строка?

Опасно!

$test = $_POST["test"];
$sql = "UPDATE main_test1 SET name = $test  WHERE id = 2;";

В вашем случае во так

$test = $_POST["test"];
$sql = "UPDATE `main_test1` SET `name` = '$test'  WHERE id = 2;";

Comments

[FanatPHP]

вот всегда придет такой дурачок и все испортит
и, главное, ведь сам не понимает, что пишет.
но лезет отвечать. как же - надо поделиться тайными познаниями

[Игорь]

FanatPHP, Ой, вас не позвали, какая жалость.
В следующий раз вас позовём.

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

Не обижайтесь, теперь будем вас звать.

[Игорь]

FanatPHP, Ооо, батенька, да вы хейтер, обходите его стороной.
Сейчас посыпется срач.

Answer 2

[FanatPHP]

проблема в неправильном запросе.
Переменную надо не прямо в запрос пихать, а передавать отдельно.

$sql = "UPDATE main_test1 SET name = ?  WHERE id = 2";
$stmt= $conn->prepare($sql);
$stmt->bind_param("s", $name);
$stmt->execute();

Answer 3

[Кирилл Горелов]

Напиши хотя бы
$sql = "UPDATE main_test1 SET name = ".$test." WHERE id = 2;";

или

$sql = "UPDATE main_test1 SET name = {$test} WHERE id = 2;";

А вообще используй pdo

Comments

[FanatPHP]

господи
ну откуда вы такие советчики-то беретесь?
даже собственный ответ понять не можете