0

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

<?php
include 'core/init.php';
protect_page();
include 'includes/overall/header.php';
if (empty($_POST) === false) {
    $required_fields = array(
        'username',
        'newusername'
    );
    foreach ($_POST as $key => $value) {
        if (empty($value) && in_array($key, $required_fields) === true) {
            $errors[] = 'поля со звездочкой обязательны';
            break 1;
        }
        if (user_exists($_POST['username']) === ($_POST['newusername'])) {
            $errors[] = 'этот логин \'' . $_POST['username'] . '\' используется'; // условие которое должно выводить ошибку и не обновлять логин в БД если такой есть
        }
    }
}

?>
   <h1>пользователи!</h1>
    <?php
if (isset($_POST['username'], $_POST['newusername'])) {
    $username    = mysql_real_escape_string($_POST['username']);
    $newusername = mysql_real_escape_string($_POST['newusername']);
    mysql_query("UPDATE `users` SET `username` = '$newusername' WHERE `username` = '$username'");
}
$ath = mysql_query("SELECT * FROM `users` WHERE `type`=0");
if ($ath) {
    // Определяем таблицу и заголовок
    echo "<table border=1>";
    echo "<tr><td>user_id</td><td>логин</td><td>пароль</td><td>имя</td><td>email</td></tr>";
    // Так как запрос возвращает несколько строк, применяем цикл
    while ($author = mysql_fetch_array($ath)) {
        echo "<tr><td>" . $author['user_id'] . " </td><td>" . $author['username'] . " </td><td>" . $author['password'] . "
            &nbsp </td><td>" . $author['first_name'] . " </td><td>" . $author['email'] . "
            &nbsp </td></tr>";
    }
    echo "</table>";
} else {
    echo "<p><b>Error: " . mysql_error() . "</b><p>";
    exit();
}

2 ответа 2

1

Я бы сделал так:

файл verify.php (проверка логина и изменение)

<?php
$username = $_POST['username'];
$new_name = $_POST['new_name'];

if(strlen($username) == 0 || $new_name == 0){echo "Поля со звездочками обязательны";exit;}
if($username == $new_name){echo "Новый логин должен отличаться от старого!";exit;}
if(mysql_result(mysql_query("SELECT COUNT(`username`) FROM `users` WHERE `username` = '$new_name'"),0) == 1){echo "Пользователь с таким именем уже существует!";exit;}

mysql_query("UPDATE `users` SET `username` = '$new_name' WHERE `username` = '$username'");
echo 'Успешно измененно';
?>
6
  • так оно вообще ничего не меняет
    – LLIAKAJI
    27 июл 2012 в 16:20
  • названия таблиц и прочее под себя измените
    – mazanax
    27 июл 2012 в 17:08
  • нет так оно вообще ничего не пишет и не изменяет, даже после поправки таблиц и переменных в коде как надо! echo "Новый логин должен отличаться от старого!"; не от старого а новый не должен быть такой какой в таблице уже есть
    – LLIAKAJI
    27 июл 2012 в 17:20
  • ну от старого же он тоже должен отличаться, ведь бессмысленная нагрузка на сервер не есть хорошо
    – mazanax
    27 июл 2012 в 18:10
  • попробуйте вывести сам запрос и ввести вручную
    – mazanax
    27 июл 2012 в 18:11
0

Что за неописанная функция user_exists?

11
  • проверяет существует ли такой логин
    – LLIAKAJI
    27 июл 2012 в 16:02
  • Проблемка-то в ней :) Кидай её сюда.
    – Shevsky
    27 июл 2012 в 16:05
  • может дело в функции user_exists?
    – mazanax
    27 июл 2012 в 16:06
  • нет не в ней, на регистрации же она работает там правда такое условие if (user_exists($_POST['username'])===true) { $errors[]='логин \'' . $_POST['username'] . '\' занят'; } но если я здесь так сделаю, оно просто пишет что дожно писать но но все равно в БД изменяет логин!
    – LLIAKAJI
    27 июл 2012 в 16:07
  • === TRUE, а у вас === ($_POST['newusername'])
    – mazanax
    27 июл 2012 в 16:09

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.