使用WHILE循环更新回显数据.只更新一条记录


Update echoed data using WHILE loop. Only updates one record

我似乎无法更新任何记录,除了第一个。我不知道如何修改任何显示的记录。

<?php
    if(isset($_POST["action"]) == "update")
    {
        $id = $_POST['m_id'][0];
        $type = $_POST['type'][0];
        // if I echo $id & $type, it only gives me the first record.**
        mysql_query("
          UPDATE membership_type
          SET mt_type ='$type'
          WHERE mt_id = '$id'"
        );
    } 
    ?>

所有这些都在同一个php页面中。

<form name=form action='' method='post'>
  <?php 
  $result=mysql_query("SELECT * FROM membership_type;");
  while($rows=mysql_fetch_array($result))
  {  ?>
    <input size=35 class=textField type=text name='type[]' value='<?php echo  $rows['mt_type']; ?>'>
    <input type=hidden name='m_id[]' value="<?php echo $rows['mt_id']; ?>">
    <input type=submit value="Update">
  <?php 
  } 
  ?>

我如何编辑任何显示的记录,只需点击更新按钮??

第一:你应该永远不要使用mysql_*函数,因为它们已被弃用。
第二步:试试下面的代码:

<?php  
// Get a connection to the database
$mysqli = new mysqli('host', 'user', 'password', 'database');
// Check if there's POST request in this file
if($_POST){ 
    foreach($_POST['m_id'] as $id => $type){
      $query = "UPDATE membership_type
                SET mt_type = '".$type."'
                WHERE mt_id = '".$id."'";
      // Try to exec the query
      $mysqli->query($query) or die($mysqli->error);
    }
}else{ 
  // Get all membership_type records and then iterate
  $result = $mysqli->query("SELECT * FROM membership_type") or die($mysqli->error); ?>
  <form name='form' action='<?php echo $_SERVER['PHP_SELF'] ?>' method='post'>
    <?php while($row = $result->fetch_object()){ ?>
      <input size='35' 
             class='textField' 
             type='text' 
             name='m_id[<?php echo $row->mt_id ?>]' 
             value='<?php echo $row->mt_type; ?>'>
      <input type='submit' value="Update">
    <?php } ?>
  </form>
<?php } ?>
第三:为了增加更多的安全性(这段代码是脆弱的),尝试mysqli_prepare

每次表单提交时只更新第一个记录,因为您设置了$id = $_POST['m_id'][0],其中包含第一个type[]文本框的值。要同时更新所有其他记录,循环$_POST['m_id'] .

替换。

<?php
    if(isset($_POST["action"]) == "update")
    {
        $id = $_POST['m_id'];
        $type = $_POST['type'];
        $i = 0;
        foreach($id as $mid) {
             mysql_query("UPDATE membership_type
                         SET mt_type='".mysql_real_escape_string($type[$i])."'
                         WHERE mt_id = '".intval($mid)."'") OR mysql_error();
        $i++;
        }
    } 
    ?>

试试这个:

if(isset($_POST["action"]) == "update")
{
        $id = $_POST['m_id'];
        $type = $_POST['type'];
        $loopcount = count($id);
        for($i=0; $i<$loopcount; $i++)
        {
          mysql_query("
           UPDATE membership_type
           SET mt_type ='$type[$i]'
           WHERE mt_id = '$id[$i]'"
          );
       }
} 

你的HTML是畸形的,你作为一个数组传递,但只使用第一个元素。考虑:

<form name="form" action="" method="post">
<?php 
$result = mysql_query("SELECT * FROM membership_type;");
while($row = mysql_fetch_array($result))
    echo sprintf('<input size="35" class="textField" type="text" name="m_ids[%s]" value="%s" />', $row['mt_id'], $row['mt_type']);
?>
    <input type="submit" value="Update">
</form>

服务器端脚本:

<?php
if(isset($_POST["action"]) && $_POST["action"] == "Update"){
    foreach($_POST['m_ids'] as $mt_id => $mt_type)
        mysql_query(sprintf("UPDATE membership_type SET mt_type ='%s' WHERE mt_id = %s LIMIT 1", addslashes($mt_type), (int) $mt_id));
} 

你还可以在这里做其他事情,例如:预处理语句,但这应该可以工作。