Проверка логина на уникальность

@Irina_Polushkin · Регистрация: 07.12.2013

Студворк — интернет-сервис помощи студентам

Есть два файла :

user.php

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
<?php
 
require_once 'DB.php';
 
function create_user($user) {
    $connection = db_connect();
    $s = $connection->prepare(<<<SQL
INSERT INTO table.users
(
    login,
)
VALUES (?)
SQL
    );
    
    $s->execute(array(
        $user['login']));           
}

И файл обработки формы:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
require_once 'user.php';
 
$messages = array();
 
if ('POST' === $_SERVER['REQUEST_METHOD']) {
    if (empty($_POST['login'])) {
        $messages[] = 'Вы должны ввести логин';
    }
  
    $connection = db_connect();
    $statement = $connection->prepare('SELECT COUNT(*) FROM table.users WHERE login=?');
    $statement->execute($login);
    $loginCount = $statement->fetch();
    if ($loginCount > 0) {
        $messages[] = 'Такой пользователь уже есть';
    }

Выдает ошибку на строчку $statement->execute($login); с текстом Undefined variable: login . Как это исправить??

@philin · 18.12.2013, 16:31

Для начала прочитать ошибку и посмотреть в код. "Undefined variable: $login" - "необъявленная переменная $login". $login нигде не присваивается значение, переменная попросту не существует — из нее нельзя ничего прочитать.

@Irina_Polushkin · 18.12.2013, 17:37 **[ТС]**

Сообщение от philin

Для начала прочитать ошибку и посмотреть в код. "Undefined variable: $login" - "необъявленная переменная $login". $login нигде не присваивается значение, переменная попросту не существует — из нее нельзя ничего прочитать.

ну а если сделать так $login = $user['login']

Добавлено через 59 минут
Ну хоть кто-нибудь скажите как решить эту ошибку

@TerMih · 18.12.2013, 19:47

Ирина, не могли бы вы выложить норм код и сказать что вы хотите сделать? такое впечатление что у вас тут не одна ошибка, а их много.
Выложите целиком свой скрипт и форму, чтоб можно было запустить и посмотреть, db.php только не надо

@Irina_Polushkin · 18.12.2013, 19:58 **[ТС]**

Сообщение от TerMih

Ирина, не могли бы вы выложить норм код и сказать что вы хотите сделать? такое впечатление что у вас тут не одна ошибка, а их много.
Выложите целиком свой скрипт и форму, чтоб можно было запустить и посмотреть, db.php только не надо

Файл user.php

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
function create_user($user) {
    $connection = db_connect();
    $s = $connection->prepare(<<<SQL
INSERT INTO table.users
(
    login,
)
VALUES (?)
SQL
    );
    
    $s->execute(array(
        $user['login']));
        
}

Файл обработки

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
require_once 'user.php';
 
$messages = array();
 
if ('POST' === $_SERVER['REQUEST_METHOD']) {
    if (empty($_POST['login'])) {
        $messages[] = 'Вы должны ввести логин';
    }
    
    $connection = db_connect();
    $statement = $connection->prepare('SELECT COUNT(*) FROM table.users WHERE login=?');
    $statement->execute([$login]);
    $loginCount = $statement->fetch()[0];
    if ($loginCount > 0) {
        $messages[] = 'Такой пользователь уже есть';
    }
    if (!$messages) {
        create_user($_POST);        
        die;
    }
}

Ошибку выдает на строке : $statement->execute([$login]); Undefined variable: login , я не знаю как ее правильно объявить

@philin · 18.12.2013, 20:06

Зачем лезть в сложные структуры, если основы непонятны...
Чтобы взять данные из $login, нужно их туда сначала положить. Где это действие?
11 строка составляет SQL-запрос, 12 подставляет в него логин. А логин лежит в $_POST['login'], он пришел из формы. Хотите читать именно из $login - запишите в него данные сначала.

PHP
1
$login = $_POST['login'];

@TerMih · 18.12.2013, 20:12

Я так понимаю вы работаете с cms drupal? я с нею к сожалению не знаком, но

PHP
1
$statement = $connection->prepare('SELECT COUNT(*) FROM table.users WHERE login=?');

Похоже на параметризованные запросы, тут вы подготавливаете запрос.

PHP
1
$statement->execute([$login]);

Тут вы выполняете запрос, но скорей всего вам надо было сделать так

[PHP]$statement = $connection->prepare('SELECT COUNT(*) FROM table.users WHERE login=?');
(функция примерно такого содержания)bind_param ($login)
$statement->execute();

Наверно так, суть bind_param в том что вы подставляете значение в запрос за место '?', а execute уже выполняет запрос, и проверьте чтоб $login имела значение, к примеру var_dump($login);

philin, а я совсем не прав?

@TerMih · 18.12.2013, 21:43

Сделал, напишу сюда, во первых у шаблона формы я не увидел action, в ваш скрипт обработки, во вторых

PHP
1
2
3
4
5
6
7
8
 $statement = $connection->prepare('SELECT COUNT(*) as count FROM users WHERE login=?');
    $login = $_POST['login'];
    $statement->execute(array($login));
    $loginCount = $statement->fetch(PDO::FETCH_ASSOC);
    print_r($loginCount);
    if ($loginCount['count'] > 0) {
        echo  'Идите лесом! Такой пользователь у нас уже есть!';
    }

В третьих у вас не правильные регулярные выражения) как минимум это к телефону, остальное не смотрел)

@LeMax24 · 18.12.2013, 22:37

А у меня как- то так)) Думаю тоже не плохо, да и работает отлично!

PHP
1
2
3
4
5
6
7
$result = mysql_query("SELECT * FROM users ORDER BY id");
    $kol = mysql_num_rows($result);
            while ( $postrow[] = mysql_fetch_array($result))
            for($i = 0; $i < $koll; $i++)
                if($_POST['login'] == $postrow[$i]['login'])
                    echo "Пользователь с таким логином уже существует";
    }

@TerMih · 18.12.2013, 22:51

Ваш вариант не очень хорош, в php версии > 5.3 mysql нет, поэтому надо использовать mysqli или pdo api, да и доведите до ума параметризованные запросы чтобы обезопасить себя от sql инъекций.

PHP
1
2
3
4
5
6
7
$result = mysql_query("SELECT * FROM users ORDER BY id");
    $kol = mysql_num_rows($result);
            while ( $postrow[] = mysql_fetch_array($result))
            for($i = 0; $i < $koll; $i++)
                if($_POST['login'] == $postrow[$i]['login'])
                    echo "Пользователь с таким логином уже существует";
    }

И вообще у вас лишние действия, код может и работает, но ваш for тут ни к чему. И у вас не работает код)

PHP
1
2
$kol = mysql_num_rows($result);
for($i = 0; $i < $koll; $i++)

$koll или $kol?
P.s а спасибо не будет?)

@LeMax24 · 18.12.2013, 22:59

Сообщение от TerMih

Ваш вариант не очень хорош, в php версии > 5.3 mysql нет, поэтому надо использовать mysqli или pdo api, да и доведите до ума параметризованные запросы чтобы обезопасить себя от sql инъекций.

PHP
1
2
3
4
5
6
7
$result = mysql_query("SELECT * FROM users ORDER BY id");
    $kol = mysql_num_rows($result);
            while ( $postrow[] = mysql_fetch_array($result))
            for($i = 0; $i < $koll; $i++)
                if($_POST['login'] == $postrow[$i]['login'])
                    echo "Пользователь с таким логином уже существует";
    }

И вообще у вас лишние действия, код может и работает, но ваш for тут ни к чему. И у вас не работает код)

PHP
1
2
$kol = mysql_num_rows($result);
for($i = 0; $i < $koll; $i++)

$koll или $kol?
P.s а спасибо не будет?)

Ну и немного подумав сделал вывод что так будет ещё проще)

PHP
1
2
3
4
$row = mysql_num_rows($query);
if($row > 0){
echo"Такой пользователь уже существует!";
}

@philin · 18.12.2013, 23:05

Еще сократим?)

PHP
1
2
if (mysql_num_rows($query) > 0)
    echo "Такой пользователь уже существует!";

@TerMih · 18.12.2013, 23:05

У вас какой то странный вывод, вы выполнили какой-то запрос и посмотрели сколько строк он вам вернул, но вы уже не используете pdo как использовали раньше, у вас же уже почти все было написано и все работало, делайте как считаете нужным и еще регулярка для телефона не правильна, она в любом случае на норм телефон будет записывать ошибку.

@LeMax24 · 18.12.2013, 23:27

Сообщение от TerMih

У вас какой то странный вывод, вы выполнили какой-то запрос и посмотрели сколько строк он вам вернул, но вы уже не используете pdo как использовали раньше, у вас же уже почти все было написано и все работало, делайте как считаете нужным и еще регулярка для телефона не правильна, она в любом случае на норм телефон будет записывать ошибку.

PHP
1
$query = mysql_query("SELECT * FROM `users`  WHERE `login`='".$_POST['login']."'");

Вот и весь запрос)
А прошлый там переделывать надо еще много. он выводить та будет сообщение об ошибке столько сколько пользователей) тоесть в зыкли ещё одну проверочку и break)

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@philin Иллюзионист 154 / 153 / 27 Регистрация: 02.10.2013 Сообщений: 330
	18.12.2013, 16:31
	Для начала прочитать ошибку и посмотреть в код. "Undefined variable: $login" - "необъявленная переменная $login". $login нигде не присваивается значение, переменная попросту не существует — из нее нельзя ничего прочитать. 0

@TerMih 10 / 10 / 2 Регистрация: 18.05.2013 Сообщений: 21
	18.12.2013, 19:47
	Ирина, не могли бы вы выложить норм код и сказать что вы хотите сделать? такое впечатление что у вас тут не одна ошибка, а их много. Выложите целиком свой скрипт и форму, чтоб можно было запустить и посмотреть, db.php только не надо 1

@TerMih 10 / 10 / 2 Регистрация: 18.05.2013 Сообщений: 21
	18.12.2013, 23:05
	У вас какой то странный вывод, вы выполнили какой-то запрос и посмотрели сколько строк он вам вернул, но вы уже не используете pdo как использовали раньше, у вас же уже почти все было написано и все работало, делайте как считаете нужным и еще регулярка для телефона не правильна, она в любом случае на норм телефон будет записывать ошибку. 1