本文详解 php 中因逻辑错误导致分类页面只显示一条商品数据的问题，核心在于 `while` 循环内条件判断位置不当及重复调用 `fetch_assoc()`，通过重构循环结构与条件嵌套即可修复。
在开发基于 PHP + MySQL 的商品分类展示功能时，一个常见却隐蔽的 Bug 是：用户点击某个分类（如 “Tech” 或 “Candles”）后，页面仅渲染出一条商品，即使数据库中该分类下存在多条 soon = '2' 的有效商品记录。问题根源并非 SQL 查询本身，而在于 PHP 数据遍历逻辑的设计缺陷。
? 问题定位：循环外提前终止与条件错位
原始代码中存在两个关键错误：
- $row = $result->fetch_assoc() 被意外重复调用（原文虽未显式写出两次，但上下文暗示其在 while 外存在冗余调用），导致游标提前移动，实际进入 while 循环时已丢失首条记录；
- if($row["soon"] == "2") 判断被错误地置于 while 循环之外或逻辑分支混乱，使得只有满足特定组合条件（如 soon==1 且提交了某分类）时才进入渲染流程，而 soon==2 的主展示逻辑未能对每条记录独立判断。
更严重的是，原代码中 else if($row['soon'] == "1" && ...) 分支与 if($row["soon"] == "2") 并列，但缺乏 else 统一兜底，且该 else if 条件中的逻辑运算符优先级未加括号（&& 优先级高于 ||），导致实际执行为：
// 错误等价于： if( ($row['soon'] == "1" && isset($_POST['tech'])) || isset($_POST['candle']) )
这意味着只要用户提交了 candle，所有 soon==1 的商品都会触发“即将上线”提示，而其他商品（包括 soon==2）可能被跳过——这直接破坏了循环的完整性。
立即学习“PHP免费学习笔记（深入）”；
✅ 正确实现：将全部渲染逻辑严格封装在 while 内
修复后的核心原则是：每一条 $row 都必须在 while 循环体内被独立、完整地判断与处理。以下是精简优化后的推荐写法：
<div class="card-deck col-lg-12 col-md-12 col-sm-12 justify-content-center text-center">
<?php
// 构建动态查询语句
$sql = "SELECT * FROM products";
if (isset($_POST['candle'])) {
$sql .= " WHERE category = 'candle'";
} elseif (isset($_POST['cloth'])) {
$sql .= " WHERE category = 'cloth'";
} elseif (isset($_POST['tech'])) {
$sql .= " WHERE category = 'tech'";
}
$result = $conn->query($sql);
if ($result && $result->num_rows > 0) {
while ($row = $result->fetch_assoc()) {
// 情况1：商品已上线（soon == '2'）→ 正常渲染
if ($row['soon'] == '2') {
?>
<div class="product_card col-xl-3 col-lg-4 col-md-5 col-sm-12 box-shadow">
<div class="card-header">
<h3 class="product_name my-0 font-weight-normal"><?php echo htmlspecialchars($row['productName']); ?></h3>
</div>
<div class="card-body">
<img class="pic"
src="productsImages/<?php echo intval($row['productID']); ?>.jpeg"
height="150" width="120" alt="<?php echo htmlspecialchars($row['productName']); ?>">
<p><?php echo floatval($row['productPrice']); ?> SEK</p>
<a class="toProduct" href="products_index.php?id=<?php echo intval($row['productID']); ?>">
<button class="btn_to_product btn btn-lg btn-outline-dark">Go to product <i class="fas fa-arrow-alt-circle-right"></i></button>
</a>
</div>
</div>
<?php
}
// 情况2：商品即将上线（soon == '1'）→ 显示占位图（可选：仅对特定分类启用）
elseif ($row['soon'] == '1') {
// 此处可根据业务需要添加分类限制，例如：仅 tech/candle 分类显示 "Coming Soon"
if (isset($_POST['tech']) || isset($_POST['candle'])) {
echo '<div class="col-12 mb-4"><h2>Coming Soon</h2><p>' . htmlspecialchars($row['productName']) . '</p></div>';
}
}
}
} else {
echo '<h2 class="col-12">Sorry!</h2><h3 class="col-12">No products found in this category.</h3>';
}
$conn->close();
?>
</div>⚠️ 关键改进点与最佳实践
- ✅ 严格单次 fetch_assoc()：确保 while 循环是唯一读取数据的地方，杜绝外部干扰；
- ✅ 条件解耦与防御性编码：使用 htmlspecialchars() 防 XSS，intval() / floatval() 强制类型安全输出；
- ✅ 移除冗余分支：不再用 else if 绑定分类状态到 soon==1，改为独立判断，提升可维护性；
-
✅ 错误报告开启（强烈建议）：在开发环境顶部加入：
error_reporting(E_ALL); ini_set('display_errors', 1);可即时捕获 fetch_assoc() 返回 null 等异常，避免静默失败。
遵循以上结构，你的分类页将稳定、准确地渲染所有匹配商品——无论 soon 值如何，每条记录都得到公平处理。记住：循环即契约，每一次迭代都应完成一次完整的业务逻辑闭环。
