如何从MySQL两个独立表分别提取指定行？解决重复结果问题

嘿，我来帮你排查下问题出在哪，以及怎么修复它。

问题根源分析 #

你现在遇到的重复显示主题行的问题，主要是这几个原因导致的：

1. 变量覆盖：在修改后的index3.php里，你用同一个$result变量存储两次查询的结果——第二次查询主题的结果直接覆盖了第一次查询帖子的结果，最后自然只会保留主题数据。
2. 参数未正确获取：$sql2里用到的$sub变量，你没有从$_GET里读取赋值，相当于查询条件无效，不过就算你加上了，变量覆盖的问题还是会存在。
3. 重复include的变量冲突：两次include index3.php会让脚本重复执行，全局的$obj数组会累加数据，而且全局变量的混乱会让逻辑变得不可控。

修正后的解决方案 #

我们可以把查询逻辑封装成独立函数，这样既能避免变量覆盖，又能灵活调用不同的查询，还能给每行结果单独加CSS样式。

第一步：重构index3.php为函数封装形式

<?php
header('Content-Type: text/html; charset=utf-8');
$servername = "localhost";
$username = "username";
$password = "password";
$dbname = "dbname";

// 封装数据库连接逻辑
function getDbConnection() {
    global $servername, $username, $password, $dbname;
    $conn = new mysqli($servername, $username, $password, $dbname);
    if ($conn->connect_error) {
        die("Connection failed: " . $conn->connect_error);
    }
    mysqli_set_charset($conn,"utf8");
    return $conn;
}

// 获取指定threadID的帖子内容
function getPostContent($threadId) {
    $conn = getDbConnection();
    // 用预处理语句防止SQL注入，比直接拼变量安全
    $sql = "SELECT message, threadID FROM wbb1_post WHERE threadID = ?";
    $stmt = $conn->prepare($sql);
    $stmt->bind_param("i", $threadId);
    $stmt->execute();
    $result = $stmt->get_result();
    
    $posts = [];
    if ($result->num_rows > 0) {
        while($row = $result->fetch_assoc()) {
            $posts[] = $row;
        }
    }
    $stmt->close();
    $conn->close();
    return $posts;
}

// 获取指定threadID的主题内容
function getThreadTopic($threadId) {
    $conn = getDbConnection();
    $sql = "SELECT topic, threadID FROM wbb1_thread WHERE threadID = ?";
    $stmt = $conn->prepare($sql);
    $stmt->bind_param("i", $threadId);
    $stmt->execute();
    $result = $stmt->get_result();
    
    $thread = [];
    if ($result->num_rows > 0) {
        $thread = $result->fetch_assoc();
    }
    $stmt->close();
    $conn->close();
    return $thread;
}
?>

第二步：修改test1.php调用函数并添加样式

<div>
    <?php
    include 'index3.php';
    $targetThreadId = 4;
    // 获取主题数据，单独加CSS样式
    $threadTopic = getThreadTopic($targetThreadId);
    if ($threadTopic) {
        echo "<h2 class='thread-topic'>" . htmlspecialchars($threadTopic['topic']) . "</h2>";
    }
    ?>
</div>
<br><br>
<div>
    <?php
    // 获取帖子数据，给每个帖子单独加CSS样式
    $posts = getPostContent($targetThreadId);
    if (!empty($posts)) {
        foreach ($posts as $post) {
            echo "<div class='post-item'>" . htmlspecialchars($post['message']) . "</div>";
        }
    }
    ?>
</div>

为什么这样改更合理？ #

• 用函数封装查询逻辑，避免了重复代码，也能精准获取你需要的某一类数据。
• 使用预处理语句替代直接拼接变量，能有效防止SQL注入，让代码更安全。
• 两次查询的结果用不同变量存储，彻底解决了变量覆盖的问题。
• 你可以直接给主题行加thread-topic类，给每个帖子加post-item类，然后在CSS里分别定义样式，完全满足你“每行单独设计样式”的需求。

内容的提问来源于stack exchange，提问作者StackBuck