17370845950

本文解决php中按分类筛选商品时，因逻辑错误导致“soon=2”的商品只显示一项的问题，核心在于修复循环内条件判断的位置与结构，并优化代码可维护性。

在开发电商类PHP应用时，常通过表单提交（$_POST）动态筛选商品分类。但如您所遇——当用户点击“Tech”或“Candles”分类后，本应展示该分类下所有 soon = "2"（即“已上架”）的商品，却仅渲染出一条记录——这通常并非数据库数据问题，而是PHP控制流逻辑嵌套错误所致。

原始代码的关键缺陷在于：
✅ while ($row = $result->fetch_assoc()) 正确获取每条记录；
❌ 但后续对 $row["soon"] == "2" 的判断被错误地置于循环体外（实际为复制粘贴导致的结构错位），或与 else if 分支混用，造成部分循环迭代被跳过、甚至提前终止。

更严重的是，原逻辑中：

if($row["soon"] == "2"){ ... }
else if($row['soon'] == "1" && isset($_POST['tech']) || isset($_POST['candle'])){ ... }

存在运算符优先级陷阱：&& 优先级高于 ||，因此该条件等价于
($row['soon'] == "1" && isset($_POST['tech'])) || isset($_POST['candle'])，
即只要用户提交了 candle（哪怕当前商品是 cloth 分类且 soon=1），也会进入“即将上线”分支，干扰主流程。

✅ 正确写法：将所有状态判断严格封装在 while 循环内

prepare($sql);
if ($category) $stmt->bind_param('s', $category);
$stmt->execute();
$result = $stmt->get_result();

if ($result->num_rows > 0): ?>
  
    fetch_assoc()): ?>
      
        
          

        
        
          @@##@@.jpeg" 
               alt="" 
               height="150" width="120">
          
 SEK
          ">
            Go to product 
          
        
      
    
  

  
Sorry!
  



close(); ?>

? 关键改进说明：

• 逻辑收束：所有商品渲染逻辑严格位于 while 内，确保每条 soon=2 记录都被处理；
• SQL预处理：使用 mysqli::prepare() 防止SQL注入，提升安全性；
• 状态解耦：soon = "1"（预售）与 soon = "2"（上架）应分页或独立路由处理，避免在主商品流中混杂条件判断；
• 安全输出：对用户数据使用 htmlspecialchars() 和 intval() 过滤，防止XSS与类型混淆；
• 结构清晰：采用 ?: 三元简化分类判定，消除冗长 if-else if 链。

⚠️ 注意事项：

• 切勿在循环中重复调用 $result->fetch_assoc() —— 这会跳过下一条记录；
• 启用 PHP 错误报告（error_reporting(E_ALL); ini_set('display_errors', 1);）可快速定位未定义索引或SQL执行失败；
• 建议将分类逻辑抽象为函数，例如 getProductsByCategory($cat, $status = '2')，提升复用性。

遵循以上重构原则，即可彻底解决“分类仅显示一项”的问题，并为后续功能扩展（如分页、搜索、多条件筛选）奠定健壮基础。