Не передает переменную в запрос?

Question

[Soolberns]

Есть функция:

function regis():bool
{
    global $pdo;
    $login = $_POST['login'];
    $pass = $_POST['pass'];

    if (empty($login) || empty($pass)) {
        $_SESSION['errors'] = "Все поля должны быть заполнены!";
        return false;
    }

    $pass = password_hash($pass, PASSWORD_DEFAULT);
    $res = $pdo->prepare("INSERT INTO users(login, password) VALUES (?,?)");
    if ($res->execute([$login, $pass])) {
        $_SESSION['success'] = 'Успешная регистрация';
        return true;
    } else {
        $_SESSION['errors'] = 'Ошибка регистрации';
        return false;
    }
}

Хочу в

$res = $pdo->prepare("INSERT INTO users(login, password) VALUES (?,?)");

заменить вопросы переменными:

$res = $pdo->prepare("INSERT INTO users(login, password) VALUES ($login,$pass)");

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

Invalid parameter number: number of bound variables does not match number of tokens in

Что делать?

Comments

[JhaoDa]

PhpStorm тут с какого боку?

Answer 1

[Ипатьев]

Я думаю, имелось в виду что-то вроде этого

function regis($pdo, $login, $pass ): bool
{
    $res = $pdo->prepare("INSERT INTO users(login, password) VALUES (:login,:pass)");
    $res->execute(["login" => $login, "pass" => $pass]);
    return true;
}

На будущее: функция должна всегда делать что-то одно. Все проверки, работа с НТТР, сессии - должно быть снаружи.

Comments

[Ипатьев]

Fedor Vlasenko, не будет. Будет исключение.

[Ипатьев]

Fedor Vlasenko, https://www.php.net/manual/ru/pdo.constants.php#pd...
Предписание выбрасывать исключение PDOException в случае возникновения ошибки. Подробнее об этом атрибуте смотрите в разделе Ошибки и их обработка.
Начиная с PHP 8.0.0 является режимом по умолчанию. Помимо задания кода ошибки PDO будет выбрасывать исключение PDOException, свойства которого будут отражать код ошибки и её описание.