Почему возникает ошибка (T_VARIBLE)?

Question

[TochkaSonya]

Код в котором возникает ошибка:

if ($row["api_key"] == null) {
	$new_api_key = api_key_generator()
	$sql = "UPDATE `Users` SET `api_key`=".parse_str($new_api_key)." WHERE id=".$row["id"]; //58 строка тут
	$result = mysqli_query($conn, $sql);
}

Функция которая там используется:

function api_key_generator($l = 8){
  $better_token = md5(uniqid(rand(), true));
      $rem = strlen($better_token)-$l;
  $unique_code = substr($better_token, 0, -$rem);
  $uniqueid = $unique_code;
  return $uniqueid;
}

Логи:

[*** May ** **:22:04 2025] [emerg] [client **.**.**.***] PHP Parse error:  syntax error, unexpected '$sql' (T_VARIABLE) in /____/_/______/public_html/login.php on line 58, referer: http://______/index.php

Оказывается ошибка заключается в отсутствии точки с запятой в строке 57

Comments

[Everything_is_bad]

из каждого утюга вещают, что sql запросы не нужно собирать через конкатенацию, но всегда найдет очередной

[TochkaSonya]

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

[Ипатьев]

Это сейчас не может. А потом логика поменяется, и вдруг сможет.
Зачем вообще писать небезопасные запросы? Какая проблема написать

$sql = "UPDATE `Users` SET `api_key`=? WHERE id=?";
$conn->execute_query($sql, [$new_api_key, $row["id"]]);

Так и код аккуратнее, и не надо сидеть думать над каждой переменной, может её пользователь изменить, или нет.

[Ипатьев]

У меня ещё свой вопрос есть. Что, по-вашему, делает функция parse_str, если вы её вдруг решили здесь написать?

[Ипатьев]

function api_key_generator($length = 8){
    return bin2hex(random_bytes(ceil($length/2)));
}

а не вот этот вот ужас

[Everything_is_bad]

TochkaSonya,

но обе вставляемые переменные хранятся на сервере и их содержимое пользователь изменить не может

да тут даже не это, подстановка через плейсхолдер еще и правильное форматирование данных делает

[TochkaSonya]

Ипатьев,

Это сейчас не может. А потом логика поменяется, и вдруг сможет.
Зачем вообще писать небезопасные запросы? Какая проблема написать

В этом месте генерируется апи ключ для пользователя тут нужен только id пользователя которые получается из sql базы и сам случайный апи ключ

Answer 1

[Ипатьев]

Когда пишет unexpected '$sql' (T_VARIABLE) это значит, что проблема в конце предыдущего выражения.
Поднимаем глаза, смотрим на конец предыдущего выражения, и видим, что с ним не так.