Не работает php код?

Question

[stasbombit]

Пишу небольшой код(соцсеть) и делая регистрацию столкнулся с проблемой

<?php

require_once __DIR__.'/boot.php';

// Проверим, не занято ли имя пользователя
$stmt = pdo()->prepare("SELECT * FROM `accounts` WHERE `login` = :username");
$stmt->execute(['username' => $_POST['login']]);
if ($stmt->rowCount() > 0) {
    flash('Это имя пользователя уже занято.');
    echo('Это имя пользователя уже занято.');
    header('Location: register.php'); // Возврат на форму регистрации
    die; // Остановка выполнения скрипта
}

$name = $_POST["name"];
$login = $_POST["login"];
$pass = $_POST["password"];
$phone = $_POST["phone"];
$email = $_POST["email"];
$bio = $_POST["bio"];

// Добавим пользователя в базу
$stmt = pdo()->prepare("INSERT INTO `accounts` (`login`, `password`, `name`, `bio`, `email`, `phone`, `rating`) VALUES (:login, :password, :name, :bio, :email, :phone, '1')");
$stmt->execute([
    'login' => $login,
    'password' => $pass,
    'phone' => $phone,
    'email' => $email,
    'bio' => $bio,
    'name' => $name,
]);

header('Location: login.php');

Но когда я выполнял код то был просто белый экран и всё
Делал с https://habr.com/ru/articles/665602/
изменил только index.php

Comments

[stasbombit]

Я знаю что уже писал недавно мне дали инструкцию и она не работает

[Сергей delphinpro]

stasbombit, необходимо включить вывод ошибок или посмотреть их в логах.
По белому экрану угадать что случилось невозможно.

[runapa]

$stmt->execute(['username' => $_POST['login']]);

Возможно тут собака порылась.

Answer 1

[Ипатьев]

Как включить вывод ошибок в PHP?

Comments

[stasbombit]

Я включил вывод ошибок и вот что есть
"Warning: require_once(/home/z/z92616gy/z92616gy.beget.tech/public_html/account/boot.php): failed to open stream: No such file or directory in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 7 Fatal error: require_once(): Failed opening required '/home/z/z92616gy/z92616gy.beget.tech/public_html/account/boot.php' (include_path='.:/usr/share/php') in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 7 "

Я прочитал только про "require_once(/home/z/z92616gy/z92616gy.beget.tech/public_html/account/boot.php): failed to open stream: No such file or directory in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 7 Fatal error: require_once(): Failed opening required "
но в коде require_once __DIR__.'/boot.php';
т.е boot.php находиться в корневой директории а php ищет его в /accounts/

[Ипатьев]

Правильно, boot.php находиТся в корневой директории.
А вы его вызываете из директории /account/

Команда __DIR__ означает совсем не "корневая директория".
Корневая директория - это $_SERVER['DOCUMENT_ROOT']

[stasbombit]

Ипатьев, так в итоге как писать
вместо require_once __DIR__.'/boot.php';
писать

require_once $_SERVER['DOCUMENT_ROOT'].'/boot.php';

я просто в php недавно

[Ипатьев]

К РНР это не имеет отношения, это базовая компьютерная грамотность.

https://phpfaq.ru/newbie/paths

[stasbombit]

Ипатьев, я исправил теперь новая проблема(просто суточный лимит вопросов исчерпал)
Ошибка:Parse error: syntax error, unexpected ':', expecting '{' in /home/z/z92616gy/z92616gy.beget.tech/public_html/boot.php on line 7
файл boot.php

<?php

// Инициализируем сессию
session_start();

// Простой способ сделать глобально доступным подключение в БД
function pdo(): PDO
{
    static $pdo;

    if (!$pdo) {
        $config = include __DIR__.'/config.php';
        // Подключение к БД
        $dsn = 'mysql:dbname='.$config['db_name'].';host='.$config['db_host'];
        $pdo = new PDO($dsn, $config['db_user'], $config['db_pass']);
        $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        error_reporting(E_ALL);
        ini_set('display_startup_errors', 1);
        ini_set('display_errors', '1');
    }

    return $pdo;
}
function flash(?string $message = null)
{
    if ($message) {
        $_SESSION['flash'] = $message;
    } else {
        if (!empty($_SESSION['flash'])) { ?>
          <div class="alert alert-danger mb-3">
              <?=$_SESSION['flash']?>
          </div>
        <?php }
        unset($_SESSION['flash']);
    }
}

[Ипатьев]

Это старая версия РНР. уберите : PDO или обновите версию.

[stasbombit]

Ипатьев, 1)обновить php я не могу тк это хостинг
2)Когда я убрал : PDO появилась новая ошибка:
Parse error: syntax error, unexpected '?', expecting variable (T_VARIABLE) in /home/z/z92616gy/z92616gy.beget.tech/public_html/boot.php on line 24

[stasbombit]

Ипатьев, а всё спасибо обновил php на хостинге

[stasbombit]

Ипатьев, Я понимаю что уже надоел но новая ошибка
Notice: Undefined index: login in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 10 Notice: Undefined index: login in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 23 Notice: Undefined index: password in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 24 Notice: Undefined index: phone in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 25 Notice: Undefined index: email in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 26 Notice: Undefined index: bio in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 27 Notice: Undefined index: name in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 28 Fatal error: Uncaught PDOException: SQLSTATE[23000]: Integrity constraint violation: 1048 Column 'login' cannot be null in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php:28 Stack trace: #0 /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php(28): PDOStatement->execute(Array) #1 {main} thrown in /home/z/z92616gy/z92616gy.beget.tech/public_html/account/regscr.php on line 28

файл регистрации

<?php
error_reporting(E_ALL);
ini_set('display_startup_errors', 1);
ini_set('display_errors', '1');

require_once '../boot.php';

// Проверим, не занято ли имя пользователя
$stmt = pdo()->prepare("SELECT * FROM `accounts` WHERE `login` = :login");
$stmt->execute(['login' => $_POST['login']]);
if ($stmt->rowCount() > 0) {
    flash('Это имя пользователя уже занято.');
    echo('Это имя пользователя уже занято.');
    header('Location: register.php'); // Возврат на форму регистрации
    die; // Остановка выполнения скрипта
}



// Добавим пользователя в базу
$stmt = pdo()->prepare("INSERT INTO `accounts` (`login`, `password`, `name`, `bio`, `email`, `phone`, `rating`) VALUES (:login, :password, :name, :bio, :email, :phone, '1')");
$stmt->execute([
    'login' => $_POST["login"],
    'password' => $_POST["password"],
    'phone' => $_POST["phone"],
    'email' => $_POST["email"],
    'bio' => $_POST["bio"],
    'name' => $_POST["name"],
]);

header('Location: login.php');

[Ипатьев]

а как вы это файл запускаете? напрямую к нему что ли обращаетесь?

[stasbombit]

Ипатьев, через форму естественно

[Ипатьев]

ну так смотрите свою форму. почему там нет ни логина, ни пароля, ни телефона

[stasbombit]

они в url есть(z92616gy.beget.tech/account/do_register.php?name=s...)
ps я сменил имя файла regscr.php на do_register.php

[stasbombit]

stasbombit, я разобрался в форме и в php были разные методы
спасибо

[Ипатьев]

Ну правильно, если они в url, значит это метод GET

[Ипатьев]

Но вообще это все конечно говнокод
Нет валидации, нет нормального взаимодействия с пользователем
нет хэширования пароля почему-то
эта функция pdo() дурацкая.
Лучше бы вы оставались со своим кодом, чем бездумно копипастили у delphinpro

Answer 2

[ThunderCat]

Логи смотри, ну или включи вывод ошибок.

echo('Это имя пользователя уже занято.');
header('Location: register.php'); // Возврат на форму регистрации

Здесь стопудоф будет лажа, так как хеадер так не отправляется. Почему - учим матчасть. Собсно в логах это будет отражено.

Comments

[Ипатьев]

ну это видно же что отладочный вывод

[ThunderCat]

Ипатьев, А разница? Работать то не будет...

Answer 3

[Артём Дайнов]

Перед session_start, а так же перед header() - не должно ничего выводиться. Если вы будете что-то выводить, это сразу выводит ошибку. Подумайте над другим вариантом, как пользователю сообщить, что логин занят.

Comments

[Ипатьев]

Вам надо научиться читать код, чтобы не оставлять бессмысленные комментарии.

[Артём Дайнов]

Ипатьев, Возможно у него по какой-то причине возвращается true при поиске пользователя.
У меня при использование echo или echo() всегда ошибка, если после этого header() будет.

[Ипатьев]

Если говорить про заголовки вообще, то вы пишете правильно.
Но для конкретно этого кода ваше замечание не имеет смысла.

Во-первых, у автора вообще нет никаких проблем с заголовками.
Автор пишет, что видит "белый экран". Это значит, что у него проблема не с заголовками, а с тем, что вообще ничего не работает

Во-вторых, сам по себе совет тоже не в тему.
Как можно увидеть, echo дублирует сообщение, которое пишется в сессию строчкой выше.
то есть это отладочный вывод, временный, чтобы разобраться с проблемой.