PHP下拉菜单分类聚合问题：重复分类需合并同类别产品

核心问题 #

原代码在遍历客户产品的循环里，每处理一个产品就会重新遍历并输出整个分类分组数组，导致每新增一个产品就重复渲染所有<optgroup>，最终出现分类重复的情况。此外还存在SQL查询效率低、有注入风险的问题。

修复后的代码 #

<?php
$customer_id = "27";

// 用联查一次性获取关联的产品与分类，避免多次查询数据库
$sql = "SELECT cp.product, ap.category 
        FROM customer_product cp
        INNER JOIN addproducts ap ON cp.product = ap.name
        WHERE cp.customer = ?";
$stmt = $conn_register->prepare($sql);
$stmt->bind_param("s", $customer_id);
$stmt->execute();
$result = $stmt->get_result();

// 先构建分类-产品的分组数组，完成所有数据整理
$groups = [];
while ($row = $result->fetch_assoc()) {
    $category = $row['category'];
    $product = $row['product'];
    // 避免同一产品重复添加到分类下
    if (!isset($groups[$category][$product])) {
        $groups[$category][$product] = $product;
    }
}
?>

<select>
<?php
// 统一遍历分组数组，只输出一次所有分类与产品
foreach ($groups as $category => $products) { ?>
    <optgroup label="<?php echo htmlspecialchars($category); ?>">
        <?php foreach ($products as $product) { ?>
            <option value="<?php echo htmlspecialchars($product); ?>">
                <?php echo htmlspecialchars($product); ?>
            </option>
        <?php } ?>
    </optgroup>
<?php } ?>
</select>

优化说明 #

• 分离数据处理与渲染：先把所有产品按分类整理到$groups数组，再单独遍历数组渲染下拉菜单，彻底避免重复输出分类。
• 合并数据库查询：通过JOIN语句一次性关联两张表获取数据，替代原代码的多次循环查询，提升效率。
• 安全优化：使用预处理语句防止SQL注入，用htmlspecialchars()转义输出内容，避免XSS风险。
• 逻辑严谨性：添加判断避免同一产品重复出现在分类下。

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