【问题标题】:How to dynamically generate MYSQL UPDATE statement based on defined variables from HTML FORM如何根据 HTML FORM 中定义的变量动态生成 MYSQL UPDATE 语句
【发布时间】:2014-10-30 06:46:24
【问题描述】:

我正在使用一个相当长的 HTML 表单来更新与产品相关的许多详细信息 - 为简洁起见,我不会完整地分享该表单。但是,出于说明目的,这里有一个 sn-p :

HTML 表单

<form name="form1" method="post" action="update_ac.php">
    <table width="100%" cellpadding="0" cellspacing="0">
    <tr>
      <td>
        <input name="season" type="text"  class="button_select" id="season" value="<?=$rows['season']; ?>" size="10" />
        <input name="restock" type="checkbox" id="restock" value="on" <?php if($rows['restock']=='on') { echo 'checked="checked"'; } ?>/>

      // other fields

      </td>
    </tr>
  </table>
</form>

我的问题是在将表单发布到update_ac.php 时 - 如何根据已完成的字段动态生成 MYSQL 更新语句?

这是我的表单操作页面的示例:

PHP 表单操作

<?php

       foreach ($_POST as $key => $value) {
        $$key = $value;
      }

$sql= mysql_query ("
UPDATE product SET
title='".$title."',
rating='".$rating."',
season='".$season."', 
brand_id='".$brand_id."',
category='".$category."', 

... etc ");

?>

我不想在 UPDATE 语句中声明每个可能需要更新的字段。鉴于存在从表单发布的已定义 PHP 变量,我希望 UPDATE 语句仅处理相关字段。

目前,我收到很多NOTICE: Undefined variable x,在发布表单时出现了空白字段。

我希望这是有道理的——有点啰嗦。

有什么建议吗?谢谢

更新

从@Styphon 的回答开始 - 我稍微修改它以在查询末尾包含WHERE 条件。

$query = "UPDATE product SET";
$comma = " ";
foreach($_POST as $key => $val) {
    if( ! empty($val)) {
        $query .= $comma . $key . " = '" . mysql_real_escape_string(trim($val)) . "'";
        $comma = ", ";
    }
}

$product_id = $_POST['product_id'];

$query = $query . "WHERE product_id = '".$product_id."' ";

【问题讨论】:

  • 看看isset 或许>
  • @Fluffeh - 这并不能解决我的问题,因为我必须针对 ISSET 测试每个可能的变量。我试图避免声明超过 50 个变量。

标签: php html mysql forms


【解决方案1】:

假设表中的所有字段名称都与表单输入的名称相同,这是直截了当的。你可以使用这个:

$query = "UPDATE product SET";
$comma = " ";
foreach($_POST as $key => $val) {
    if( ! empty($val)) {
        $query .= $comma . $key . " = '" . mysql_real_escape_string(trim($val)) . "'";
        $comma = ", ";
    }
}
$sql = mysql_query($query);

为了更安全,您应该创建一个可接受参数的白名单,即表中的列如下:

$query = "UPDATE product SET";
$comma = " ";
$whitelist = array(
    'title',
    'rating',
    'season',
    'brand_id',
    'cateogry',
    // ...etc
);
foreach($_POST as $key => $val) {
    if( ! empty($val) && in_array($key, $whitelist)) {
        $query .= $comma . $key . " = '" . mysql_real_escape_string(trim($val)) . "'";
        $comma = ", ";
    }
}
$sql = mysql_query($query);

这样您的查询只能包含您设置的参数,并且如果有人设法注入额外内容(例如通过更改表单输入的名称),它将不会被传递到您的数据库。


我还建议你停止使用 Mysql_*,它是 deprecated。您应该查看 MySQLiPDO 作为替代方案。

【讨论】:

  • @nalply 我在real_escape_string里加了,现在太习惯用prepared statements了。
  • @Styphon - 我刚刚意识到声明末尾缺少WHERE product_id = $product_id 行。我已经更新了上面的代码。谢谢
  • @nalply 我在查询周围使用单引号,所以这无关紧要,该查询受到保护。在您发表评论之前,您是否也看到了我更新后的带有白名单的答案?
  • 这不是易受 SQL 注入攻击吗?
  • @arviman 它有mysql_real_escape_string 可以防止大多数注入尝试,但它不如使用准备好的语句。
猜你喜欢
  • 2011-02-07
  • 1970-01-01
  • 2011-10-25
  • 1970-01-01
  • 2019-08-20
  • 2020-10-22
  • 2023-04-08
  • 1970-01-01
  • 2015-12-16
相关资源
最近更新 更多