DomDocument removeChild in foreach reindexing den dom

Question

Ich versuche p Tags mit data-spotid Attribute

 $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); 
      } 
     } 

aber zu löschen, wenn ich Kind bin Entfernen sie den dom sind reindexing. Ich nehme an, ich habe 8 Items, die ich gelöscht habe, 1. es wird es neu indizieren und das zweite Element wird 1. und es wird nicht gelöscht, es wird auf 2. gehen, das jetzt 3. Element ist.

Answer 1

Dies wird in ein paar Kommentare auf der DomNode::removeChild Dokumentation erwähnt, mit der Frage offensichtlich sein, wie der Iterator Zeiger auf dem foreach nicht in der Lage zu sein mit der Tatsache zu tun dass Sie Elemente aus einem übergeordneten Array entfernen, während Sie die Liste der untergeordneten Elemente (oder etwas) durchlaufen.

Die empfohlene Lösung besteht darin, zunächst den Hauptknoten durchzulaufen und die untergeordneten Knoten, die Sie löschen möchten, in ein eigenes Array zu verschieben, dann das zu löschende Array zu durchlaufen und diese untergeordneten Elemente zu löschen. Beispiel:

$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

können wir wie folgt verwenden:

 $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

Wie ich bemerkte, die leicht Lösung wäre, nur werfen den Iterator auf ein Array. Beispiel:

Aber wenn wir über die Leistung sprechen, wäre ein besserer Weg, einfach nur die erforderlichen Knoten auszuwählen. Eine ordentliche Nebenwirkung, das Entfernungsproblem verschwindet ebenfalls.

Z. B .:

<?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(); 

Demo: https://3v4l.org/CM9Fv

Answer 4

(Unter der Annahme, dass der $ dom den (DOM enthält) Die Absätze müssen Sie herausfiltern). Lassen Sie uns einige gute alte JavaScript versuchen:

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

HINWEIS: I Outerhtml bin mit unerwünschten Elemente zu zerstören, um seine Eltern zu vermeiden Aufruf und verlagern Sie den Knoten von Interesse, das wir bereits haben. Aktuelle Firefox-Versionen unterstützen es endlich (11+). MDN ref

Ich verwende auch die kurze all.tags() Syntax für die Kürze; Firefox unterstützt es vielleicht noch nicht, also sollten Sie vielleicht wieder auf den Aufruf 'getElementsByTagName()' zurückgreifen.