我的循环中的数据不匹配，不知道为什么

Question

我有一个代码可以在用户之间来回显示消息。出于某种原因，数据不匹配，我不确定我的代码中哪里有错误。我试图改变我的 $x < $sender 的 if 语句，但这没有用。

但显示的数据与数据库信息不一致。

“测试 2”消息是从 woof3 发送的，因此当以 woof 身份登录时，它应该是绿色而不是蓝色。并且“再次测试”是从woof发送的，所以以woof登录时应该是蓝色的

在下面编辑工作代码，没有 pdo，因为我还没有完成这个项目，我将它们保存到最后以使测试更容易。

<?php

include "../Site/db.php";

$url = $_SERVER['REQUEST_URI'];
$urlArray = explode('=', $url);
$Username = $urlArray[sizeof($urlArray) - 1];

$Myself = $_SESSION['username'];

$sql = "SELECT * FROM messages ORDER BY `Id` ASC";
$result = $conn->query($sql);

// Header of message box
echo '<div id="messages" class="messagePersonBox shadow">
        <span class="usersname" name="receiver"><a href="Account?='.$Username.'">'.$Username.'</a></span>
<hr style="margin-top:24px">
<div id="allMessages"class="allMessages">
';

// $sql2 = "SELECT * FROM messages WHERE Sender = '$Myself' ORDER BY `Id` ASC";
// $result2 = $conn->query($sql2);

$sql3 = "SELECT *  FROM messages 
        WHERE (Sender = '$Myself' AND Receiver = '$Username')
           OR (Receiver = '$Myself' AND Sender = '$Username') ORDER BY `MessageTime` ASC";


$result3 = $conn->query($sql3);
print_r($result3);
$x = 0;

$sql4 = "SELECT * FROM messages WHERE Sender = '$Myself'";
$result4 = $conn->query($sql4);

$sql5 = "SELECT * FROM messages WHERE Receiver = '$Myself'";
$result5 = $conn->query($sql5);

$totalSender = mysqli_num_rows($result4);
$totalReceiver = mysqli_num_rows($result5);

// might have to add both totals for while loop
// to get total messages
$totalmessages = mysqli_num_rows($result);

// For receiving
while ($x < $totalmessages) {

    // output data of each row

     if ($result3->num_rows > 0) {
         if ($row = $result3->fetch_assoc()) {
             $Sender = $row['Sender'];
             $Message = $row['Message'];
             $Receiver = $row['Receiver'];
             $TimeSent = $row['MessageTime'];
         }
     }
       $class = $Sender == $Myself ? 'messageDisplay' : 'messageDisplay2';


     // sending a message

         echo '<div id="messages2" class="'.$class.'">'.$Message.'<span style="float: right;margin-top:2px"><font size=1>'.time_elapsed_string($TimeSent).'</span></font></div>';



$x++;
}

// Typing part of message
echo '</div>
      <input type="text" id="messagetosend" name="message" class="messagetype" placeholder="Type a message..."><span id="send" class="messageSend">↩</span>

</div>';
?>

如果需要，我可以添加更多信息。

Answer 1

我建议你退后一步，稍微调整一下这种方法。

例如，您不需要conversationID（至少不是当前格式），因为您可以从Sender 和Receiver 中扣除它，这样更准确。您也不需要MessageDate，因为MessageTime 是一个包含日期和时间的日期时间值。所以，conversationID 和 MessageDate 可以被删除。

然后，我们可以清理查询的数量。您希望在您是发件人而其他用户是收件人的情况下获取所有邮件，或您是收件人而其他用户是该邮件的发件人。然后我们按日期（而不是 ID！）排序，并根据该行的 Sender 值检查消息是绿色还是蓝色 - 如果发件人是当前，则设置一个带有蓝色值的 $class 变量用户，否则为绿色。

请注意，当输出 HTML 块时，您可以退出 PHP，这样更易​​于阅读（并处理引号）。您还拥有具有相同 ID 的 HTML 元素，这是不允许的。

在处理查询中的变量时，您还应该使用准备好的语句。

<?php

include "../Site/db.php";

$url = $_SERVER['REQUEST_URI'];
$urlArray = explode('=', $url);
$Username = $urlArray[sizeof($urlArray) - 1];

$Myself = $_SESSION['username'];

$sql = "SELECT Message, MessageTime, Sender, Receiver
        FROM messages 
        WHERE (Sender = ? AND Receiver = ?)
           OR (Receiver = ? AND Sender = ?)
        ORDER BY `MessageTime` ASC");
if (!$stmt = $conn->prepare($sql)) {
    error_log($conn->error);
    die("An error occurred fetching messages");
}
$stmt->bind_param("ssss", $Myself, $Username, $Myself, $Username);
$stmt->bind_result($message, $timeSent, $sender, $received);
if (!$stmt->execute()) {
    error_log($stmt->error);
    die("An error occurred fetching messages");
}
?>
<!-- Header of message box -->
<div id="messages" class="messagePersonBox shadow">
    <span class="usersname" name="receiver"><a href="Account?=<?php echo $Username; ?>"><?php echo $Username; ?></a></span>
    <hr style="margin-top:24px">
    <div id="allMessages" class="allMessages">
        <?php 
        while ($stmt->fetch()) {
            $class = $sender == $Myself ? 'messageDisplay' : 'messageDisplay1'; // Check which class should be used; if I'm the sender, show messageDisplay, if not show MessageDisplay1 
            ?>
            <div class="<?php echo $class; ?>
                <?php echo $message; ?>
                <span style="float: right; margin-top: 2px">
                    <?php echo time_elapsed_string($timeSent); ?>
                </span>
            </div>
            <?php 
        }
        ?>
    <!-- Typing part of message -->
    </div>
    <input type="text" id="messagetosend" name="message" class="messagetype" placeholder="Type a message..."><span id="send" class="messageSend">↩</span>
</div>
<?php 
$stmt->close();