PHP开发汽车经销商页面：数据库单条记录仅遍历一次的问题求助

嘿，我看你遇到的核心问题是循环逻辑嵌套错了，导致每辆车重复输出了数据库里该类别下的总条目数，咱们一步步把这个问题理顺：

你的现有代码问题分析 #

第一个代码里，你在外层while ($row = mysqli_fetch_assoc($result))（每次取一条数据）里面，套了一个for ($i = 1;$i <= $count; $i++)循环——这就意味着每拿到一条车辆数据，你就把它重复输出了$count次，自然会出现重复显示的问题。而且$count = mysqli_num_rows($result);放在while循环里也完全没必要，只需要在循环前计算一次就行。

第二个尝试的代码更糟：你把数据库查询mysqli_query放在了for循环里面，每次循环都重新查一遍数据库，而且do-while的条件写反了，直接导致无限循环，所以页面加载超时。

正确的实现思路 #

不需要嵌套循环，只需要用一个计数器来追踪当前是第几条数据，然后根据计数器的值来控制表格的行（每4辆车一行）：

1. 先执行查询，获取结果集，计算总条数，初始化计数器；
2. 循环遍历每条数据，每处理一条计数器加1；
3. 根据计数器判断：什么时候开始新的一行（第1、5、9...条），什么时候结束当前行（第4、8、12...条）；
4. 循环结束后，处理最后一行可能不够4辆车的情况，补全表格标签。

修正后的完整代码 #

// 先执行查询，只查一次！
$query = "SELECT year, brand, model, class, stocknr, price, status, pic FROM stock WHERE class = 'car' ORDER BY brand ASC";
$result = mysqli_query($con, $query);
$count = mysqli_num_rows($result);
$i = 0; // 初始化计数器

echo '<table>'; // 先输出外层表格的开始标签

while ($row = mysqli_fetch_assoc($result)) {
    $i++; // 每处理一条数据，计数器加1
    $title = $row['year'] . ' ' . $row['brand'] . ' ' . $row['model'];
    $uppic = $row['pic'];
    $picloc = "/upload/$uppic";
    $price = $row['price'];
    $stock = $row['stocknr'];
    $des = "d.php?stocknr=$stock"; // 补全你之前缺失的$des变量

    // 当是第1、5、9...条数据时，开启新的一行
    if ($i == 1 || ($i - 1) % 4 == 0) {
        echo '<tr>' . "\n";
    }

    // 输出单辆车的内容
    echo "<td>";
    echo "<table>";
    echo "<tr><td><br><a href='$des'><span>$title</span></a></td></tr>";
    echo "<tr><td><a href='$des'><img src='$picloc'></a></td></tr>";
    echo "<tr><td><br><a href='$des'><span>R$price</span></a></td></tr>"; // 修正你之前的$符号错误
    echo "</table>";
    echo "</td>";

    // 当是第4、8、12...条数据时，结束当前行
    if ($i % 4 == 0) {
        echo '</tr>' . "\n";
    }
}

// 处理最后一行：如果总条数不是4的倍数，补全结束标签
if ($count % 4 != 0) {
    echo '</tr>' . "\n";
}
echo '</table>';

额外的小提示 #

• 你之前的代码里有$<span>这种语法错误，我已经帮你修正成R$price了；
• 记得确保$con是有效的数据库连接对象；
• 虽然你现在用表格布局，之后学了CSS可以用Flexbox或者Grid来实现更灵活的布局，比表格布局更适合这种产品列表。

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