foreach 中的 DomDocument removeChild 重新索引 dom

Question

我正在尝试删除带有data-spotid 属性的p 标签

        $dom = new DOMDocument();
        @$dom->loadHTML($description);
        $pTag = $dom->getElementsByTagName('p');

        foreach ($pTag as $value) {
            /** @var DOMElement $value */
            $id = $value->getAttribute('data-spotid');
            if ($id) {
                $value->parentNode->removeChild($value);
            }
        }

但是当我删除孩子时，它正在重新索引 dom。假设我有 8 个项目，我删除了第一个，它将重新索引它，第二个元素将成为第一个，它不会删除它将转到第二个，现在是第三个元素。

Answer 1

DomNode::removeChild 文档的几个 cmets 中提到了这一点，问题显然是 foreach 上的迭代器指针无法处理您在循环时从父数组中删除项目的事实孩子（或其他东西）的名单。

建议的解决方法是先循环遍历主节点，然后将要删除的子节点推送到其自己的数组中，然后循环遍历该“待删除”数组并从其父节点中删除这些子节点。示例：

$dom = new DOMDocument();
@$dom->loadHTML($description);
$pTag = $dom->getElementsByTagName('p');

$spotid_children = array();

foreach ($pTag as $value) {
    /** @var DOMElement $value */
    $id = $value->getAttribute('data-spotid');
    if ($id) {
        $spotid_children[] = $value; 
    }
}

foreach ($spotid_children as $spotid_child) {
    $spotid_child->parentNode->removeChild($spotid_child); 
}

Answer 2

我们可以这样使用：

        $dom = new DOMDocument();
        @$dom->loadHTML($description);
        $pTag = $dom->getElementsByTagName('p');
        $count = count($pTag)
        for($i = 0; $i < $count; $i++) {
            /** @var DOMElement $value */
            $value = $pTag[$i];
            $id = $value->getAttribute('data-spotid');
            if ($id) {
                $i--;$count--;
                $value->parentNode->removeChild($value);
            }
        }

Answer 3

就像我评论的那样，easy 解决方案是将迭代器cast 转换为数组。例如：

$elements = iterator_to_array($elements);

但是，如果我们谈论性能，更好的方法是仅选择所需的节点。很好的副作用，移除问题也消失了。

例如：

<?php
$doc = new DOMDocument('1.0', 'UTF-8');
$doc->loadXML(<<<__XML
<?xml version="1.0" encoding="UTF-8"?>
<root>
    <element>1</element>
    <element attr="a">2</element>
    <element>3</element>
    <element>4</element>
    <element attr="a">5</element>
    <element attr="a">6</element>
    <element>7</element>
    <element>8</element>
</root>
__XML
);

$xpath = new DOMXPath($doc);
$elements = $xpath->query('//element[@attr]');

foreach ($elements as $element) {
    $element->parentNode->removeChild($element);
}

echo $doc->saveXML();

演示：https://3v4l.org/CM9Fv

Answer 4

（假设 $dom 包含您需要过滤掉的（DOM）段落）。 让我们尝试一些好的旧 JavaScript：

$ptag = $dom.all.tags("p");
$ptag = [].slice.call($ptag);
$i = 0; 
while($ptag[$i]){
'data-spotid' in $ptag[$i].attributes ? $ptag[$i++].outerHTML = "" : 0
}

注意： 我正在使用 outerHTML 来销毁不需要的元素，以避免调用其父级并重新定位我们已经拥有的感兴趣的节点。最近的 Firefox 版本终于支持它（11+）。MDN ref

为了简洁起见，我还使用了简短的 all.tags() 语法； Firefox 可能还不支持它，所以你可能想回退到 'getElementsByTagName()' 调用那里。