PHP循环中HTML标签自动提前闭合问题求助

问题根源 #

你的代码逻辑错误在于：每次检测到$v1 != $v2时，同时输出了<ul>和</ul>，导致刚打开列表就立刻闭合，所有<li>都被留在了列表外部，最终出现空的<ul></ul>和游离的<li>。

正确实现代码 #

// 初始化变量
$prev_variation = null;
$k = 0;
$ul_is_open = false;

for ($i = 0; $i < $count; $i++) {
    $v1 = $data3['body']['asinVariationValues'][$i]['variationName'];
    $img = $data3['body']['asinVariationValues'][$i]['variationImageURL'];
    $val = $data3['body']['asinVariationValues'][$i]['variationValue'];
    
    // 当分组变化时，先闭合上一个列表（如果存在），再开启新分组
    if ($v1 != $prev_variation) {
        // 关闭之前打开的ul
        if ($ul_is_open) {
            echo '</ul>';
            $ul_is_open = false;
        }
        
        // 输出新分组的标题和ul
        $k++;
        echo '<strong>'. htmlspecialchars($v1) .'</strong><ul id="j-sku-list-'.$k.'">';
        $ul_is_open = true;
    }

    // 输出列表项，注意转义HTML特殊字符避免XSS
    $escaped_val = htmlspecialchars($val);
    if (!empty($img)) { 
        echo '<li><span title="'.$escaped_val.'"><img src="'.htmlspecialchars($img).'"></span></li>';
    } else {
        echo '<li><span title="'.$escaped_val.'">'.$escaped_val.'</span></li>';
    }

    // 更新上一个分组名称
    $prev_variation = $v1;
}

// 循环结束后，闭合最后一个未关闭的ul
if ($ul_is_open) {
    echo '</ul>';
}

关键调整点 #

1. 分组切换逻辑：只有当分组名称变化时，先关闭上一个已打开的<ul>，再创建新的分组标题和<ul>，避免重复开闭。
2. 状态标记：用$ul_is_open变量跟踪当前是否有未闭合的<ul>，确保循环结束后不会遗漏闭合标签。
3. 安全优化：添加htmlspecialchars()转义输出内容，避免HTML注入或XSS风险（原代码直接输出用户数据存在安全隐患）。
4. 变量顺序修正：原代码中$v2 = $v1的赋值顺序错误，改为用$prev_variation存储上一次的分组名称，避免覆盖当前值导致判断失效。

内容的提问来源于stack exchange，提问作者Fake Tracking Number Generator