Не отправляется MYSQL ЗАПРОС?

Question

[Андрей]

Суть запроса: если в столбце Id есть значение с $datatopush[0], то меняем текущее значение столбца Streetid на $datatopush[1]. Если нету - записываем айди и соответствующее значение.

if($_GET['pushuidstreetid']){
        $res= $_GET['pushuidstreetid'];
        $datatopush = explode(":", $res);
        echo $datatopush[1];
        $qpush = "IF ($datatopush[0] IN SELECT UserStreetid(Id)) THEN 
        INSERT INTO UserStreetid(Streetid) VALUES ($datatopush[1]); ELSE   
        INSERT INTO UserStreetid (Id, Streetid) VALUES ($datatopush[0], $datatopush[1]); END IF;";
        $result = $connection->query($qpush);
        echo"Error:".$qpush."<br>".mysqli_error($connection);
    };

Выдает ошибку:

pushError:IF SELECT Id FROM UserStreetid WHERE Id=5556 THEN INSERT INTO UserStreetid(Streetid) VALUES (push); ELSE INSERT INTO UserStreetid (Id, Streetid) VALUES (5556, push); END IF
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'IF SELECT Id FROM UserStreetid WHERE Id=5556 THEN INSERT INTO UserStree' at line 1

Comments

[Akina]

У MySQL есть свой собственный синтаксис. И придумывать ему ещё один совершенно бессмысленно - он будет проигнорирован. Что, собственно, и наблюдается.

Answer 1

[galaxy]

Смотреть страшно, что там в запросе понаписано... В общем, вот:

$stmt = $connection->prepare("INSERT INTO UserStreetid(Id, Streetid) VALUES (?, ?) ON DUPLICATE KEY UPDATE Streetid = ?");
$stmt->bind_param("iii", $datatopush[0], $datatopush[1], $datatopush[1]);
if (!$stmt->execute())
  echo"Error:".$qpush."<br>".mysqli_error($connection);

(нужен PRIMARY/UNIQUE ключ на id)

Comments

[Андрей]

Что это за ключь и для чего он нужен ? И еще вопрос к вашему коду. Разве он проверяет наличие значения в столбе? Если у меня будет два раза подаваться запрос с одним и тем же юзер айди но сразными streetid тогда надо поменять только streetid

[galaxy]

Андрей, https://www.mysqltutorial.org/mysql-primary-key/

[galaxy]

Андрей,

Если у меня будет два раза подаваться запрос с одним и тем же юзер айди но сразными streetid тогда надо поменять только streetid

так он и сделает (подробнее про такие запросы - тут).

[Андрей]

Вот я попробовал два раза поменять streetid с одним и тем же id,@galaxy,
Плюс к тему оно записует 0 в поле streetid вместо нужного текста на английском. Это тест Вашего кода galaxy

[galaxy]

Андрей, у вас нет ключа. Нужен первичный или уникальный ключ на id (или пару id, Streeid).
В phpmyadmin (это же он?) для этого должны быть галочки.

Ну и покажите запрос, который отправляете.

[Андрей]

https://website.online/bothourstate.php?pushuidstr...

[Андрей]

galaxy, Я включил ключ. Заработало. Только всеравно не пойму почему поле Streetid тип даных указан как TEXT шлю в него текст и выдает 0

[galaxy]

Андрей, покажите SQL запрос, который шлете (ссылка выше не работает, если что)

[Андрей]

$connection = new mysqli($servername,$username, $password, $database);


if($_GET['pushuidstreetid']){
        $res= $_GET['pushuidstreetid'];
        $datatopush = explode(":", $res);
        $datatopush[1] = strval($datatopush[1]);
        $stmt = $connection->prepare("INSERT INTO UserStreetid(Id, Streetid) VALUES (?, ?) ON DUPLICATE KEY UPDATE Streetid = ?");
$stmt->bind_param("iii", $datatopush[0], $datatopush[1], $datatopush[1]);
if (!$stmt->execute())
  echo"Error:".$qpush."<br>".mysqli_error($connection);
    };

Тоесть при постулении зароса с концом "/bothourstate.php?pushuidstreetid=555678:vel"
Streetid должен быть например vel

[galaxy]

Андрей, а, ясно. Исправьте строку:

$stmt->bind_param("iss", $datatopush[0], $datatopush[1], $datatopush[1]);

[Андрей]

Хорошо, спасибо большое , что помогли) Можете обьяснить мне как начинающему, что означает "iii" и "iss"?
Это сокращенная запись типа переменной которая будет использоваться ?

[galaxy]

Андрей, да - https://www.php.net/manual/ru/mysqli-stmt.bind-par...

[FanatPHP]

только if (!$stmt->execute()) - это адова ересь

[FanatPHP]

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

[Андрей]

FanatPHP, Вы можете спокойно выразить свое мнение более конструктивно и представить соотвецтвующий код Вашего варианта решения ?

[FanatPHP]

Могу, но эта реплика была не вам, а galaxy, который этот код видал 500 миллионов раз, но писать так и не научился

[Андрей]

FanatPHP, Я понял , что Вы обращались не ко мне . В любом случае мне , как начинающему программисту, полезно читать код написанный разными людьми , сравнивать и анализировать. Так Вы можете написать свое решение программы , что бы я его увидел ?

[FanatPHP]

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

[Андрей]

FanatPHP, Да , вполне. Я ознакомился с вашим решением .

Answer 2

[FanatPHP]

https://qna.habr.com/q/918033#answer_1847841
И в целом рекомендую прочитать ответы на этой странице, поскольку со всеми проблемами придется столнуться