Не срабатывает sql запрос. Как быть?

Question

[okha22]

не срабатывает SQL запрос на добавление нового пользователя в БД

$sql = "INSERT INTO `users` (`phone`,`name`,`fio`,`user_email`,`user_password`,`activity`,`twath`,
`actvml`,`vmlstr`,`code`,`group`,`city`,`DateReg`,`LastEnter`,`user_ip`,`user_login`,`skin`,`sex`,`money`) 
VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";
$stmt = $db->prepare($sql);
$stmt->bind_param("ssssssssssssssssssi", $phone,$name,'-',$mail,$pasw,1,0,0,1,$code,1,$city,
$dreg,$dreg,$ip,$login,0,$sex,0);
$res = $stmt->execute();

Comments

[Дмитрий]

И ошибку не выдаёт?

[okha22]

Дмитрий, нет, в том то и проблема

[Ипатьев]

Проблема не в этом, а в том что вы не настроили РНР, чтобы он вам сообщал об ошибках

[okha22]

Ипатьев, Окей, включил вывод, вот что говорит:

Fatal error: Uncaught Error: Cannot pass parameter 4 by reference in
/engine/modules/personal.php on line 56

собственно, вот 56 строка:

$stmt->bind_param("ssssssssssssssssssi", $phone,$name,'-',$mail,$pasw,1,0,0,1,$code,1,$city,$dreg,$dreg,$ip,$login,0,$sex,0);

[Ипатьев]

Вот, прямым текстом повторило то, что я написал в своем ответе.

[Дмитрий]

okha22, ну и внимательно схему глянуть, ошибки бывают разные, например поля не так названы, количество биндов может не совпадать или ещё чего. Без текста ошибок ни как.

Answer 1

[Ипатьев]

Не выполняется mysql запрос

Но в данном случае можно сократить путь "получение сообщения об ошибке - чтение сообщения об ошибке - гугление сообщения об ошибке".
Надо эти черточки и нолики перенести из bind_param в сам запрос.

Answer 2

[Slava Rozhnev]

Чтобы исправить используйте следующий трюк. Создаем массив значений, а потом используем деструктор

$sql = "INSERT INTO `users` (`phone`,`name`,`fio`,`user_email`,`user_password`,`activity`,`twath`,`actvml`,`vmlstr`,`code`,`group`,`city`,`DateReg`,`LastEnter`,`user_ip`,`user_login`,`skin`,`sex`,`money`) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";
$stmt = $mysqli->prepare($sql);

$params = [$phone, $name, '-', $mail, $pasw, 1, 0, 0, 1, $code, 1, $city, $dreg, $dreg, $ip, $login, 0, $sex, 0];

$stmt->bind_param("ssssssssssssssssssi", ...$params);
$res = $stmt->execute();

https://phpize.online/s/rW

Comments

[Ипатьев]

Во-первых, так не надо делать. Статическим элементам нечего делать в параметрах.
А по уму надо вообще настроить default значения в базе и совсем выкинуть эти поля из запроса.
Во-вторых, нет смысла делать одноразовую переменную, весь массив можно писать прямо в функцию.

[Ипатьев]

Да, и деструктор - это совсем другое.
А это можно описать как оператор распаковки массива