JeffLiteral Ответов: 3

Что делает следующие коды неспособными отправлять данные на сервер mysql?


Вот коды, которые я использую с
<form action="code.php" method="POST"></form>


Это мои коды внутри code.php.
<?php
session_start();

$connection = mysqli_connect("localhost","root","","adminpanel");

if(isset($_POST['register_btn']))
{
    $fullname = $_POST['fname'];
    $office = $_POST['office'];
    $school = $_POST['school'];
    $email = $_POST['email'];
    $username = $_POST['uname'];
    $password = $_POST['pword'];
    $cpassword = $_POST['cpword'];
    //$image = $_POST['photo'];
 
    if($password === $cpassword) {


    $query = "INSERT INTO register (fname, office, school, email, uname, pword) VALUES ('$fullname', '$office', '$school', '$email', '$username')";
    $query_run = mysqli_query($connection, $query);

            if($query_run) {
                $_SESSION ['success'] = "New Site Admin added";
                header ('Location: register.php');
            }
            else {
                $_SESSION ['status'] = "New Admin entry NOT succesful";
                header ('Location: register.php');
            }
    }
    else {
        $_SESSION ['status'] = "Password and Confirm Password Do Not Match";
        header ('Location: register.php');
        }

}
?>


Что я уже пробовал:

Я ввел все необходимые данные правильно и тщательно но они всегда возвращались
New Admin entry NOT succesful


Пожалуйста, помогите мне с этим. Спасибо всем.

3 Ответов

Рейтинг:
20

Patrice T

вам нужно научиться отлаживать свой код.
сообщение

<pre lang="text">
Новая запись администратора не увенчалась успехом
исходит из вашего кода
$_SESSION ['status'] = "New Admin entry NOT succesful";

а это значит, что в нем происходит что-то неожиданное.
$query_run = mysqli_query($connection, $query);
if($query_run) {

только вы можете обнаружить причину.

Ваш код ведет себя не так, как вы ожидаете, или вы не понимаете, почему !

Существует почти универсальное решение: запускайте свой код на отладчике шаг за шагом, проверяйте переменные.
Отладчик здесь, чтобы показать вам, что делает ваш код, и ваша задача-сравнить с тем, что он должен делать.
В отладчике нет никакой магии, он не знает, что должен делать ваш код, он не находит ошибок, он просто помогает вам, показывая, что происходит. Когда код не делает того, что ожидается, вы близки к ошибке.
Чтобы увидеть, что делает ваш код: просто установите точку останова и посмотрите, как работает ваш код, отладчик позволит вам выполнять строки 1 на 1 и проверять переменные по мере их выполнения.

Отладчик - Википедия, свободная энциклопедия[^]

Освоение отладки в Visual Studio 2010 - руководство для начинающих[^]
Базовая отладка с помощью Visual Studio 2010 - YouTube[^]

phpdbg | php отладчик[^]
Методы отладки для PHP-программистов[^]

Отладчик здесь только для того, чтобы показать вам, что делает ваш код, и ваша задача-сравнить его с тем, что он должен делать.


Рейтинг:
2

OriginalGriff

Мы не знаем - и пока вы не выясните, что это за ошибка, никто не узнает.
Так что используйте PHP mysqli_error() функция / mysqli::$error - w3resource[^] чтобы узнать, на что жалуется MySql, и идти оттуда.

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

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?


JeffLiteral

Это значит...?

$query = "INSERT INTO 'register'


с одной кавычкой ".

Извиняюсь...

Рейтинг:
1

Wendelius

Кажется, что вы упускаете одно значение. Вы вставляете в 6 столбцов, но предоставляете только 5 значений

Взгляните на это отображение

 $query = "INSERT INTO register (fname, office, school, email, uname, pword) VALUES ('$fullname', '$office', '$school', '$email', '$username')";

- fname  <= $fullname
- office <= $office
- school <= $school
- email  <= $email
- uname  <= $username
- pword  <= ?????

Но прежде чем просто предоставить for pword, измените код, чтобы использовать переменные привязки, как описано в первом решении

Also note! Если поле pword предназначено для пароля, как следует из названия, никогда не храните пароли в виде текста в базе данных. Чтобы сделать это правильно, взгляните на Хранение паролей: как это сделать.[^]


JeffLiteral

Я сделал кое-что, как вы и предлагали.


    $fullname = $_POST['fname'];
    $office = $_POST['office'];
    $school = $_POST['school'];
    $email = $_POST['email'];
    $username = $_POST['uname'];
    $password = md5($_POST['pword']);
    $cpassword = $_POST['cpword'];

$query = "INSERT INTO 'register' (fname, office, school, email, uname, pword) VALUES ('$fullname', '$office', '$school', '$email', '$username', '$password')";


но прямо сейчас он возвращает результат:

$_SESSION ['status'] = "Password and Confirm Password Do Not Match";
        header ('Location: register.php');

Wendelius

При проверке совпадения паролей пока не хэшируйте их, а используйте входные данные пользователя как есть.

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

Wendelius

Также обратите внимание, что это не очень хорошая идея использовать MD5 для хэширования. Для получения более подробной информации см. PHP: хэширование паролей - руководство пользователя[^]

JeffLiteral

Я действительно достал хеширование md5. Но прямо сейчас он переходит в другой статус, который является

$_SESSION ['status'] = "New Admin entry NOT successful";


Как это может быть очень трудно для меня? :( Но я очень благодарен Вам за ваше терпение и готовность помочь.

Что еще я сделал не так, чтобы вызвать эти ошибки?

Wendelius

Чтобы узнать фактическую ошибку, возвращаемую базой данных, используйте PHP mysqli_error() функция / mysqli::$error - w3resource[^]

Что такое текст ошибки, который он возвращает?