-1

我正在尝试编写一个 PHP 项目。在这个站点中，人们可以通过登录来分享照片，照片显示在程序主页的 index.php 中。现在我有三个数据库用户的图像和消息。我可以提供用户通过保存有关照片的信息来分享他们的照片，然后从图像表中从数据库中获取它们。现在我想提供一个用户对主页中的照片进行评论。下面有 textarea 和 post 按钮图像。但是当我对图像进行评论时，它不仅会注册相关的图片 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++;

    }
4

4 回答 4

0

尝试这个：

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++;

    }
 ?>

这是生成表单和图片，当您提交表单时，您将获得和pic_id并且您已经拥有.messagepostemail$_SESSION

现在，尝试插入。

于 2013-05-23T17:55:22.713 回答
0

一方面，您正在while循环内执行 INSERT 语句。这意味着当他们提交表单时，您的输出循环（顶部）将检索记录，并且在构建 HTML 输出时，它将为每个输出记录while执行 INSERT 。随着输出记录的经过，该变量被设置为下一个图片 ID 。但是在该循环内运行，向每个.$picidwhileINSERT$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)

?>

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

于 2013-05-23T17:52:29.607 回答
0

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

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

于 2013-05-23T17:49:05.243 回答
0

更好的是，您应该尝试将 AJAx 用于评论系统，您遇到此问题可能是因为此代码-

$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++;
          }
于 2013-05-23T17:34:55.360 回答