Не работает условие проверки уже зарегистрированных пользователя?

Question

[Алексей]

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

<?php
  require('connect.php');
  if ($conn->connect_error) {
    die("Connection failed: " . $conn->connect_error);
  }
  if(isset($_POST['username']) && isset($_POST['password'])){
        $username = $_POST['username'];
        $email = $_POST['email'];
        $password = $_POST['password'];


    $sql = 'SELECT count(id) as count FROM users WHERE username=?'; 
    $query = $conn->prepare($sql); 
    $query->execute([$username]);
    $count_users = $query->fetch(); 
    if ((int)$count_users['count'] === 0) { 
      $sql = "INSERT INTO users (`username`, `password`, `email`)
      VALUES ('$username', '$password', '$email')";
    } else {
      echo "<script>alert('Логин уже занят')</script>"; 
    }
        
  if ($conn->query($sql) === TRUE) {
    header("Location: /");
  } else {
    echo "Error: " . $sql . "<br>" . $conn->error;
  }

  $conn->close();
 
    }
?>

Comments

[Алексей Уколов]

Как конкретно оно «не работает»?

[Алексей]

Алексей Уколов, добрый день, мне нужно проверка на уже зарегистрированных пользователей, по условие счетчик, если пользователь уникален он должен быть зарегистрирован и переход в админку (самописную), если нет, а повторяется то должно появляться сообщение, что логин занят и блокировать переход. Сейчас при дублировании логина, идёт просто переход не срабатывает условие.

[Ипатьев]

Не воспроизводится.
Это означает, что в $username лежит имя, которого нет в базе. или запускается какой-то другой код, а не этот.

В целом код нормальный, за исключением того что подготовленные выражения надо использовать для всех запросов, а не только для select. Плюс два бессмысленных if-а:

if ($conn->query($sql) === TRUE) { - это ненужный мусор, это никогда не надо писать. Сразу header без всяких условий.

if ($conn->connect_error) вообще не имеет смысла, и будет только каждый раз выдавать ошибку.

[Алексей]

Ипатьев, добрый день, спасибо за комментарий, if пофиксил я уже понял, что лишние. Но я до сих пор не понимаю, почему условие не работает.

[Ипатьев]

Вы код запускали по ссылке?
Условие работает.

Answer 1

[Siverius]

//я бы на вашем месте попробовал вот так для начала
        require('connect.php');
        if ($conn->connect_error) {
            die("Connection failed: " . $conn->connect_error);
        }
        if (isset($_POST['username']) && isset($_POST['password'])) {
            $username = $_POST['username'];
            $email = $_POST['email'];
            $password = $_POST['password'];

            $countUsersQuery = 'SELECT count(id) as count_users FROM users WHERE username=?';
            $query = $conn->prepare($countUsersQuery);
            $query->execute([$username]);
            $countUsersResult = $query->fetch();
            if ((int)$countUsersResult['count_users'] === 0) {
                $insertUserQuery = "
                    INSERT INTO users (`username`, `password`, `email`)
                    VALUES ('$username', '$password', '$email')";
            } else {
                echo "<script>alert('Логин уже занят')</script>";
            }

            if ($conn->query($insertUserQuery) === true) {
                header("Location: /");
            } else {
                echo "Error: " . $insertUserQuery . "<br>" . $conn->error;
            }

            $conn->close();
        }

Comments

[Алексей]

Siverius, добрый день попробовал ваш вариант условие не срабатывает, идёт переход на страницу и дублируются в БД, новый юзер с одинаковыми логинами.

[Siverius]

А вот так вам что напишет?

...
$countUsersResult = $query->fetch();
var_dump($countUsersResult);
...

[Siverius]

Алексей, кстати, а почему count(id)? Если юзер должен быть один, то почему бы просто не

SELECT id as exists_user_id FROM users WHERE username = ?

[Ипатьев]

Siverius, а какая разница?

[Ипатьев]

В целом непонятно, зачем вы вообще ответ писали.
Чтобы по-другому обозвать пару переменных? Серьёзно?

[Алексей]

Siverius, спасибо за комментарий я проверил все ваши варианты они не работают, условие не работает.

Answer 2

[Алексей Уколов]

Сейчас при дублировании логина, идёт просто переход не срабатывает условие.

Это потому, что вы переиспользуете название переменной и не обрабатываете корректно отрицательное условие.
Если сократить ваш код до описанного сценария, то он выглядит вот так:

$sql = 'SELECT count(id) as count FROM users WHERE username=?'; 
...
echo "<script>alert('Логин уже занят')</script>"; 
...
      
if ($conn->query($sql) === true) {
  header("Location: /");
}

Т.е. во второй проверке вы делаете снова самый первый select, а не insert.
Правда, в этом случае не должна бы проходить проверка на === true. Но, думаю, просто у вас в коде где-то ещё какая-то подобная ерунда написана.