如何将PHP逻辑表达式正确写入拼接HTML的字符串变量中

现有代码错误点 #

• 运算符优先级异常：PHP中字符串拼接符.的优先级高于三元运算符?:，未做括号包裹的三元表达式会先执行前置字符串拼接再做条件判断，直接导致类名动态判断逻辑完全失效
• HTML属性未加双引号：data-status、class两个属性的值没有用双引号包裹，当属性值存在空格等特殊字符时会引发HTML解析错误
• 标签结构错误：<span>开标签错误添加了自闭合标记/>，和后续的</span>闭合标签不匹配，会导致DOM结构异常
• 变量未初始化：若$statuses为空数组，未提前初始化的$html变量在拼接时会抛出PHP警告

修正后可运行代码 #

// 初始化变量避免空遍历警告
$html = '';
foreach($statuses as $status) {
    $html .= '<div class="dropdown-item">';
    // 三元表达式加括号调整优先级，属性补充双引号，删除多余的自闭合标记
    $html .= '<span data-status="' . $status['id'] . '" class="dropdown-item__sub' . (($last_ticket['status'] ?? '') == $status['status'] ? ' active' : '') . '">' . $status['status']. '</span>';
    $html .= '</div>';
}

echo $html;

可读性优化方案（可选） #

使用sprintf拼接可以避免优先级问题，代码可读性更高：

$html = '';
foreach($statuses as $status) {
    $html .= '<div class="dropdown-item">';
    $html .= sprintf(
        '<span data-status="%s" class="dropdown-item__sub%s">%s</span>',
        $status['id'],
        ($last_ticket['status'] ?? '') == $status['status'] ? ' active' : '',
        $status['status']
    );
    $html .= '</div>';
}
echo $html;

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