发布到数据库

Question

所以我有这样的代码，它接受用户插入的消息/帖子，并将其发布到数据库，然后显示一个单独的页面。我已经让展示公园工作正常，它只是试图插入数据库，这就是问题所在
这段代码...

<?php  
mysql_connect("localhost", "root", "");  
mysql_select_db("test");     
$time = time();  
mysql_query "INSERT INTO threads (title, message, author, dated);" 
VALUES (NULL,'$_POST[title]','$_POST[message]','$_POST[author]','0','$time');      
echo "Thread Posted.<br><a href='Untitled9.php'>Return</a>";  
?> 

不会将信息发布到数据库中！

这是为什么？如何解决？

  id      int(11)                          No None AUTO_INCREMENT               
  title   varchar(255)  latin1_swedish_ci  No None                
  message text           latin1_swedish_ci  No None                
  author  varchar(255)   latin1_swedish_ci  No None                
  replies int(11)                           No None                
  posted  varchar(255)   latin1_swedish_ci  No None                
  votes_up int(11)                          No 0                
  votes_down int(11)                        No 0 

Answer 1

更新：

应该发布而不是注明日期。

这是你的问题：

mysql_query "INSERT INTO threads (title, message, author, posted);" 
VALUES (NULL,'$_POST[title]','$_POST[message]','$_POST[author]','0','$time');  

将其更改为：

mysql_query("INSERT INTO threads (title, message, author, posted) VALUES ('$_POST[title]','$_POST[message]','$_POST[author]','$time');"); 

我看到你也有空值，这让我相信你使用带有自动增量的ID，如果是这种情况，你也需要提供这个。示例：

编辑：此处

mysql_query("INSERT INTO threads (id,title, message, author, posted) VALUES (NULL,'$_POST[title]','$_POST[message]','$_POST[author]','$time');"); 

请注意，直接从发布数据插入值是不安全的，会让您容易受到各种攻击。

Answer 2

您尝试添加到新行的值多于指定的值。

mysql_query "INSERT INTO threads (title, message, author, dated);" 

您要设置 4 个值

VALUES (NULL,'$_POST[title]','$_POST[message]','$_POST[author]','0','$time');  

，并且要分配 6 个值。

这是不可能的

同时验证 $_POST 数据 = 阅读此 从不信任用户输入。

并阅读手册 PHP & MYSQL

Answer 3

分号正在结束您的 sql 语句。你的询问还没有结束。您仍然需要指定要插入的值。

Answer 4

mysql_query "INSERT INTO threads (title, message, author, dated);" 
VALUES ('$_POST[title]','$_POST[message]','$_POST[author]','$time');

您提前结束了字符串。应该是：

mysql_query("INSERT INTO threads (title, message, author, dated)
  VALUES ('$_POST[title]','$_POST[message]','$_POST[author]','$time')"); 

此外，您的代码很可能成为 SQL 注入的目标。您应该使用 MySQLi 类和 PreparedStatement 插入您的帖子。

Answer 5

问题数量：

1. 如果将 $_POST[] 放入字符串中，则需要将其放在大括号 {$_POST[]} 中，否则 PHP 将无法解析该变量
2. ，接下来需要引用 $_POST[] 中的变量名称这样 PHP 就不认为它们是常量，所以它们需要像 $_POST['title'] 或 $_POST["title"]
3. 正如其他人所说，您需要通过过滤发布的变量来防止 SQL 注入。最安全的方法是使用 PDO，我在下面提供了一个示例。你可以在这方面进行改进。
4. 打开错误报告，以便您可以在调试时看到错误

以下是经过测试的代码：

ini_set('error_reporting', E_ALL | E_STRICT);
ini_set('display_errors', 'On');
$user='root';
$pass='';
$dsn = 'mysql:dbname=test;host=localhost'; //for PDO later

mysql_connect("localhost",$user , $pass);  
mysql_select_db("test");     
$time = time();  
if (isset($_POST) && !empty($_POST))
{
// using braces {}
$sql=<<<SQL
INSERT INTO threads (title, message, author, posted) 
VALUES ('{$_POST['title']}','{$_POST['message']}','{$_POST['author']}','$time')

SQL;

echo "$_POST[title]"."Thread Posted.<br><a href='Untitled9.php'>Return</a>";

// now a PDO version of the same
try {
    $pdo = new PDO($dsn, $user, $pass);
} catch (PDOException $e) {
    echo 'Connection failed: ' . $e->getMessage();die;
}

$sth = $pdo->prepare("INSERT ino threads (title, message, author, posted)
                  VALUES (:title,:message,:author,:posted)");
$sth->execute(array(':title' => $_POST['title'],':message' => $_POST['message'], ':author' => $_POST['author'] ,':posted' => $time));
echo "Affected rows=".$sth->rowCount().",we are on line=".__LINE__."<br />";
echo $_POST['title']." Thread Posted.<br><a href='Untitled9.php'>Return</a>";

} // close if $_POST