商品折扣显示异常排查：非折扣商品仅最后一项显示原价

咱们来拆解下你代码里导致异常的几个关键问题：

1. 完全没必要的二次数据库查询 #

你主查询已经把products表的所有字段（包括discount）都查出来了，但在循环里又重复执行了一次SELECT discount FROM products WHERE product_name='$name'。这不仅白白浪费数据库资源，还会引入数据错误——如果有同名但不同折扣的商品，你拿到的折扣可能根本不是当前商品的，而是同名称下另一个商品的。

2. GROUP BY的错误使用 #

主查询里的GROUP BY product_name是个大坑！SQL标准里，SELECT的非聚合字段必须出现在GROUP BY里，MySQL如果关闭了ONLY_FULL_GROUP_BY模式，会随机返回每个product_name对应的一行数据，这就导致你看到的商品信息可能不是你预期的，甚至会出现部分商品的折扣/价格被错误覆盖的情况。如果你是想去掉重复的商品名，应该用DISTINCT或者调整业务逻辑，但GROUP BY在这里完全不适用。

3. 判断逻辑完全偏离需求 #

你用mysqli_num_rows($dis)==1来决定显示折扣还是原价，这逻辑和你“有折扣值显示折扣，无值显示原价”的需求完全不搭边。这个条件判断的是查询到的商品行数是否为1，而不是折扣字段是否有有效值。比如如果某个商品名对应多个商品，哪怕其中一个有折扣，这个条件也不成立，会错误显示原价；反之如果商品名只对应一个商品但折扣是0，它还是会显示折扣0而不是原价。

修正后的代码 #

把这些问题都改掉，代码就正常了：

// 去掉错误的GROUP BY，直接查询库存大于0的商品
$get = mysqli_query($conn," SELECT * FROM products WHERE quantity >0 LIMIT ".$this_page_first_result.','.$result_per_page);
$number_of_results=mysqli_num_rows($get);

while ($row=mysqli_fetch_array($get)) {
    $id = $row['product_id'];
    $name = $row['product_name'];
    $type = $row['type'];
    $img = $row['img'];
    $weight=$row['weight'];
    $price=$row['product_price'];
    $discount = $row['discount']; // 直接从主查询获取折扣值
?>
<article style="background-color: white; border-left-width: 2px; border-bottom-width: 2px;margin :4px; box-shadow: 2px 2px 3px 2px #65b7c2; ">
<img src="adminpanel/product_img/<?php echo $img?>" alt="" style="height:13rem;width:13rem;">
<h3 ><?php echo $name;?></h3>
<?php
// 正确判断：折扣不为空且大于0时显示折扣，否则显示原价
if (!empty($discount) && $discount > 0) {
?>
<h4>$<?php echo $discount ?></h4>
<?php } else { ?>
<h4>$<?php echo $price ?></h4>
<?php } ?>

额外建议 #

• 以后尽量避免在循环里执行数据库查询，会严重影响性能，能一次查询拿到的数据就一次搞定。
• 建议开启MySQL的ONLY_FULL_GROUP_BY模式，这样能避免这种不合规的GROUP BY写法，提前发现问题。
• 注意SQL注入风险！你当前的代码直接把变量拼进SQL语句里，很容易被注入攻击，建议使用预处理语句（prepared statements）来替代。

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