使用 PHP 将下拉列表中的数据插入数据库

Question

首先我需要一个可以轻松更新的下拉列表，因此我创建了一个名为 我在表格中列出要选择的制造商的制造商。

我终于用这段代码完成了：

<?php
 // Connect to the test datbase on localhost
 // That's where we created the countries table above
 mysql_connect('localhost','##user##','##pass##');  mysql_select_db('wordpress');

 // Query the countries table and load all of the records
 // into an array.
 $sql = 'select * FROM manufacturers';
 $res = mysql_query($sql) or die(mysql_error());
 while ($rec = mysql_fetch_assoc($res))
 $manufacturers[] = $rec;
 ?>
<form action="select.php" method="post">
<?php
 echo '<select name="dropdown">';
 foreach ($manufacturers as $c)
{
  if ($c['id'] == $_GET['id'])
   echo "<option value=\"{$c['meta_id']}\" selected=\"selected\">{$c['meta_value']}              </option>\n";
 else
  echo "<option value=\"{$c['meta_id']}\">{$c['meta_value']}</option>\n";
 }
echo '</select>';
?>
 <input type="submit" value="Submit" name="submit"/>
 </form>

效果很好，我现在有一个下拉列表，从我的 数据库制造商。

现在我需要将它发送到现有的数据库调用 post_meta 以便我可以从那里永久显示用户选择。

我尝试了几个不同的选项，但我正在尝试使用以下代码将其发送到我的 post_meta 数据库。

<?php
$con = mysql_connect("localhost","##user##","##pass##");
if (!$con)
{
die('Could not connect: ' . mysql_error());
}

mysql_select_db("wordpress", $con);

$sql="INSERT INTO wp_postmeta (meta_id, post_id, meta_key, meta_value)
VALUES
('$_POST['meta_id']}','$_POST[post_id]','$_POST[meta_key]','$_POST[meta_value]')";

if (!mysql_query($sql,$con))
{
die('Error: ' . mysql_error());
}
echo "1 record added";
?>

这实际上插入到数据库中，但不记录任何值。

请帮我弄清楚我做错了什么。

Answer 1

你的价值观没有出现吗？看起来您的键值周围缺少引号。例如，不应该是：

$_POST['post_id']  

要进行完整性检查，只需回显您的 $_POST 变量，而不是立即进行插入。这将帮助您确定您是否有一些语法错误。另外，我会阅读 Brad 的评论并牢记在心。

Answer 2

试试这个查询：

$sql="
INSERT INTO wp_postmeta
(meta_id, post_id, meta_key, meta_value)
VALUES
(
    '{$_POST['meta_id']}',
    '{$_POST['post_id']}',
    '{$_POST['meta_key']}',
    '{$_POST['meta_value']}'
)
";

而且，正如人们在 cmets 中所说，这段代码非常脆弱，请考虑找到更好的选项将变量传递给查询。

Answer 3

执行此操作的正确方法是 A：转义所有 $_POST 超全局变量。
B. 编写如下所示的查询。

这是 wp_postmeta 的表定义：
http://codex.wordpress.org/Database_Description#Table:_wp_postmeta

因为 meta_id 是一个 auto_increment 主键，你不提供它，MySQL 提供。

//$meta_id = mysql_real_escape_string($_POST['meta_id']);  <<-- not needed.
$post_id = mysql_real_escape_string($_POST['post_id']);
$meta_key = mysql_real_escape_string($_POST['meta_key']);
$meta_value = mysql_real_escape_string($_POST['meta_value']);
$sql=" INSERT INTO wp_postmeta
       (post_id, meta_key, meta_value)
       VALUES
       ('$post_id','$meta_key','$meta_value') ";  //<<-- don't forget the quotes!
if ($result = mysql_query($sql)) {
  //You can get the new meta_id using:
   $new_meta_id = mysql_insert_id($result);
} else { 
   die ("could not insert ".mysql_error());
}