PHP MySQL Вставить данные

Question

PHP MySQL Вставить данные

0

Я пытаюсь вставить данные в таблицу в MySQL. Я нашел/модифицировал некоторый код из w3Schools и все еще не мог заставить его работать. Вот что я до сих пор:

<?php
$rusername=$_POST['username']; 
$rname=$_POST['name'];
$remail=$_POST['emailadr'];
$rpassword=$_POST['pass'];
$rconfirmpassword=$_POST['cpass'];

if ($rpassword==$rconfirmpassword) {

    $con = mysql_connect("host","user","password");
    if (!$con)
     {
         die('Could not connect: ' . mysql_error());
     }
        mysql_select_db("mydbname ", $con);
    }

    mysql_query("INSERT INTO members (id, username, password)
    VALUES ('4', $rusername, $rpassword)");


?>

Я что-то ошибся? По моему мнению, "члены" - это имя таблицы. Если кто-то знает, что не так, я ценю помощь.

Спасибо

nosedive25 22 май 2010, в 02:17

Источник

0

Почему вы добавляете 4 в качестве идентификатора? Просто поставьте id как автоинкремент, и вам больше не придется с этим сталкиваться. Кроме того, id должен быть целым числом, поэтому нет необходимости использовать там кавычки
nico 22 май 2010, в 00:04
2

Это уязвимость SQL-инъекций. Пожалуйста, используйте параметризованные запросы (см. Документацию PHP для PDO: php.net/pdo ) или, по крайней мере, экранирующий ввод с помощью функции, подобной mysql_real_escape_string .
eyelidlessness 22 май 2010, в 00:12
0

Я просто вставил 4, чтобы проверить это. Теперь он генерирует случайное число. И да, я слышал об SQL-инъекции, и я использую mysql_real_escape_string для входа в систему. Я еще не добавил это к созданию учетной записи все же.
nosedive25 22 май 2010, в 00:24
0

это напоминает мне о: xkcd.com/327
Amirshk 22 май 2010, в 00:35

Показать ещё 2 комментария

Теги:

php

mysql

post

2 ответа

0

И имя вашей базы данных?

mysql_select_db("mydbname ", $con);

Даже если он называется mydbname, там есть странное пространство char..

Вы также можете сделать

mysql_query("use mydbname", $con);

ИЗМЕНИТЬ

Попробуйте включить в запрос флаг подключения к базе данных, а также заменить "4" (что, если nico сказал, что это ключ автоинкремента [id], он должен быть целым числом без кавычек) с помощью null:

mysql_query("INSERT INTO members VALUES
            (null, '$rusername', '$rpassword');", $con);

novato 21 май 2010, в 21:31

0

Я изменяю это так, чтобы было легче читать.
nosedive25 22 май 2010, в 00:06

Ещё вопросы

Почему вы добавляете 4 в качестве идентификатора? Просто поставьте id как автоинкремент, и вам больше не придется с этим сталкиваться. Кроме того, id должен быть целым числом, поэтому нет необходимости использовать там кавычки
Это уязвимость SQL-инъекций. Пожалуйста, используйте параметризованные запросы (см. Документацию PHP для PDO: php.net/pdo ) или, по крайней мере, экранирующий ввод с помощью функции, подобной mysql_real_escape_string .
Я просто вставил 4, чтобы проверить это. Теперь он генерирует случайное число. И да, я слышал об SQL-инъекции, и я использую mysql_real_escape_string для входа в систему. Я еще не добавил это к созданию учетной записи все же.
Я изменяю это так, чтобы было легче читать.

Amirshk · Accepted Answer · 2010-05-21T21-01-00.000Z

Запрос, полученный в результате вашего кода:

INSERT INTO members (id, username, password) VALUES ('4', rusername, rpassword)

Обратите внимание, что строка SQL должна быть окружена '.

Итак, обновите свой код:

mysql_query("INSERT INTO members (id, username, password)
             VALUES ('4', '$rusername', '$rpassword')");

Я обновил его, но я все еще не вижу новый список в БД.
пожалуйста, разместите вашу схему БД