为什么不能在 PHP 中只发送一条评论？

Question

我正在尝试编写一个 PHP 项目。在这个站点中，人们可以通过登录来分享照片，照片显示在程序主页的 index.php 中。现在我有三个数据库用户的图像和消息。我可以通过保存有关照片的信息来提供用户分享他们的照片，然后从图像表中的数据库中获取它们。现在我想提供一个用户对主页中的照片进行评论。有 textarea 和图片下方的发布按钮。但是当我对图片进行评论时，它不仅注册了相关图片ID，还注册了其他图片ID。为什么会发生这种情况？

这里是部分php代码，我也在必要的部分打开和关闭了数据库连接。

 while($x < mysql_num_rows($result))
        {   
      $picid=mysql_result($result, $x, 'id');
      $name=mysql_result($result, $x, 'imageName');
      $date=mysql_result($result, $x, 'imageDate');

        echo "<div><img src = 'upload/$name' height='300 width='300' /></div>";
        echo "Name: $name   Date: $date";
        echo "<br/>";
       ?>
       <p>Post a Comment</p>

      <form action="<?= $_SERVER['PHP_SELF']?>" method="POST">
      <textarea name="message" cols="70" rows="6" placeholder="Enter a comment">

     </textarea><br/><br/><br/><br/><br/><br/>
      <input type="submit" name="submit" value="Post" style="float:left"><br/><br/>    
      <br/>
      </form>
     <!--<textarea name="message" cols="70" rows="6" ></textarea><br/><br/><br/><br/>   
     <br/><br/>-->
       <?php

        extract($_REQUEST);
        if (isset($submit)) {
          $y=0;
          $email=$_SESSION['email'];
           $asdf = mysql_query("SELECT * from users WHERE email='$email'");
           //var_dump(mysql_error());
            while( $y<mysql_num_rows($asdf) )
           { 
            $iduser=mysql_result($asdf,$y,'id');
             $y++;
          }

               echo "$iduser";
               $sql ="INSERT INTO messages (id_user, message,picid) VALUES   
               ('$iduser','$message','$picid');";

               $rslt = mysql_query($sql);

               if ($rslt== false) {
                echo '<p>Error: cannot execute query</p>';
               }

              } ?>
              <?php

      $x++;

    }

Answer 1

试试这个：

while($x < mysql_num_rows($result)) {   
      $picid=mysql_result($result, $x, 'id');
      $name=mysql_result($result, $x, 'imageName');
      $date=mysql_result($result, $x, 'imageDate');

        echo "<div><img src = 'upload/$name' height='300 width='300' /></div>";
        echo "Name: $name   Date: $date";
        echo "<br/>";
       ?>
       <p>Post a Comment</p>

      <form action="<?= $_SERVER['PHP_SELF']?>" method="POST">
      <textarea name="message" cols="70" rows="6" placeholder="Enter a comment">
     </textarea><br/><br/><br/><br/><br/><br/>
      <input tyep='hidden' name='pic_id' value='<?php= $picid; ?>' />
      <input type="submit" name="submit" value="Post" style="float:left"><br/><br/>    
      <br/>
      </form>
   <?php
  $x++;

    }
 ?>

这是生成表单和图片，当您提交表单时，您将在post 中获得pic_id 和message，并且您已经在$_SESSION 中获得email。

现在，尝试插入。

Answer 2

首先，您在while 循环中执行INSERT 语句。这意味着当他们提交表单时，您的输出循环（顶部的while）将检索记录，并且在构建 HTML 输出时，它将为每个输出记录执行插入。当while 遍历输出记录时，$picid 变量被设置为下一张图片 ID。但是INSERT 在该循环内运行，为每个$picid 添加相同的注释。

让我们暂且不说有更简洁的方法来编写此代码。要么您是新手程序员，要么对项目结构没有太多控制权——这两者都不是坏事，但让我们坚持目前的情况。

即使在您拥有的结构内，此代码也需要进行重大更改。

while($row < mysql_fetch_assoc($result))
{
  // $row is an array with all the columns, 
  //   No need to keep calling a separate function to get the columns.
  //
  //$picid=mysql_result($result, $x, 'id');
  //$name=mysql_result($result, $x, 'imageName');
  //$date=mysql_result($result, $x, 'imageDate');

    echo "<div><img src = 'upload/{$row['name']}' height='300 width='300' /></div>";
    echo "Name: {$row['name']}   Date: {$row['date']}";
    echo "<br/>";

  // Notice the new <input> tag in the FORM below. 
  // You still need to identify which picid to add the message to.
 ?>
   <p>Post a Comment</p>

  <form action="<?= $_SERVER['PHP_SELF']?>" method="POST">
    <input type="hidden" name="picid" value="<?= $row['id']; ?>"/>
    <textarea name="message" cols="70" rows="6" placeholder="Enter a comment">
    </textarea>
    <br/><br/><br/><br/><br/><br/>
    <input type="submit" name="submit" value="Post" style="float:left">
    <br/><br/><br/>
  </form>

   <?php
} // END RESULTS OUTPUT (while)

// BEGIN SUBMIT DATA PROCESSING
if (isset($submit))
{
  // This is not recommended, but it's not the point here.
  extract($_REQUEST);
  //$y=0;
  $email=$_SESSION['email'];
  $result = mysql_query("SELECT * from users WHERE email='{$email}'");

  // Unless an email can have multiple records in [users] table, 
  // the loop is unnecessary.
  //while ($row = mysql_fetch_assoc($result))
  //{ 
    // $row is an array with all the columns, 
    //   No need to keep calling a separate function to get the columns.
    // $iduser=mysql_result($asdf,$y,'id');
    //     $y++;
  //}
  // Instead... This automatically gets the first row from $result.
  $row = mysql_fetch_assoc($result);
  $iduser = $row['id'];

  //echo "$iduser";
  $sql ="INSERT INTO messages (id_user, message,picid) VALUES   
    ('{$iduser}','{$message}','{$picid}');";

  $result = mysql_query($sql);

  if ($result === false)
  {
    echo '<p>Error: cannot execute query</p>';
  }
} // END IF isset($submit)

?>

这是未经测试的，但希望它能让你朝着更好的方向前进。

Answer 3

您正在使用 while 循环 while($x < mysql_num_rows($result)) 插入直到条件结束。

在$picid =mysql_result($result, $x, 'id'); 中给出所有$picid 并且由于isset($submit) 为真，程序将在所有行中插入消息。