尝试确保 html 标签没有在 php 中保持打开状态

Question

我无法弄清楚为什么这段代码不起作用：

<?php
  $text = "<a><li><ul><ol>Hello";
  $tags = array('a', 'li', 'ul', 'ol');
  $tagcount = count($tags);
  $i = 0;

  while ($i < $tagcount) {
      $opentag = "<".$tags[$i];
      $closetag = "</".$tags[$i].">";

      if (stripos($text, $opentag)) {
          $lastopen = strripos($text, $opentag);
          $lastclose = strripos($text, $closetag);

          if ($lastopen > $lastclose) {
              $text = substr($text, 0, $lastopen);
              echo $tags[$i] . " tag was open. ";
          } else {
              echo $tags[$i] . " tag was closed. ";
      } else {
          echo $tags[$i] . " tag was not open. ";
      $i++;
  }
?>

它应该做的至少表明 $tags 数组中的所有标签都已打开。它的目的是使用 substr() 来确保没有标签打开，但它不起作用。运行这个给出：

标签未打开。 li 标签已打开。 ul 标签未打开。 ol 标签未打开。

即使它们都是开放的。任何帮助将不胜感激。

Answer 1

看来你的逻辑有缺陷：如果没有找到针，strripos返回false，因此在你的内部if语句中你正在测试一个数字是否是大于false。

对于外部 if 语句，您需要测试 false：

if (stripos($text, $opentag) !== false) {
  // found at position 0 or more...

您的内部 if 应该类似于：

    if (($lastclose !== false) && ($lastopen > $lastclose)) {

Answer 2

是“未打开”，因为 stripos 将返回第一次出现的位置，并且第一次出现位于索引 0（计算结果为 false）。

被发现是打开的，因为它的索引不为零。但随后您截断搜索字符串，以便在零索引处找到下一次迭代

...

将 if 更改为 stripos($text, $opentag) === false 并查看是否允许您找到处于打开状态的 a 标记。您必须弄清楚如何处理 substr(...) 因为我认为您的业务逻辑很可能会决定这一点。

Answer 3

下面是一个使用正则表达式的示例：

  $text = "<a><li><ul><ol>Hello";
  $tags = array('a', 'li', 'ul', 'ol');
  $tagcount = count($tags);
  $i = 0;
  $matches = array();
  foreach ($tags as $tag)
  {
    $closed = preg_match_all("/<\/".$tag.">/i", $text, $matches);
    $open = preg_match_all("/<".$tag.">/i", $text, $matches);

    if ($open == 0)
    {
        echo $tag." was not opened, ";
    }
    else if ($open > $closed)
    {
        echo $tag." was left open, ";
    }
    else
    {
        echo $tag." was closed properly, ";
    }
}

Answer 4

解析 HTML 并不简单，有一些很好的库可以为您完成这项工作。 Tidy 库自 PHP 5 起可用，可用于解析和整齐的 HTML 片段，或完整的页面输出。有一篇关于 devzone 的很好的 文章 展示了如何使用它，包括如何将其与 输出缓冲。

对于您发布的代码，您不应该在这样的 if 语句中使用 strpos 。引用PHP手册：

警告：此函数可能返回布尔值 FALSE，但也可能返回非布尔值其计算结果为 FALSE，例如 0 或 "" 。 。 。使用 === 运算符来测试该函数的返回值。

因此，要测试是否在字符串中找不到子字符串，请执行以下操作：

if(strpos($haystack, $needle) === FALSE)

并测试是否找到了子字符串：：

if(strpos($haystack, $needle) !== FALSE)

但我真的、真的建议使用预先存在的库进行 HTML 操作或验证，尤其是在安全敏感的情况下（例如，反 XSS）。