Почему DomDocument getElementsByTagName возвращает половину NodeList? - PullRequest
Новая
       16

Почему DomDocument getElementsByTagName возвращает половину NodeList?

6 голосов
/ 01 апреля 2011

Я генерирую некоторые нестандартные теги HTML с DomDocument, и результат таков: 

/* Input HTML
  <div id="toobar_top">
    <widget id="flag_holder"></widget>
    <widget id="horizontal_menu"></widget>
  </div>
  <div id="header">
    <widget name="header"></widget>
  </div>
*/

Что я хочу сделать, это «перевести» каждый виджет во что-то полезное ... они простызаполнители с параметрами.

Функция извлекает из класса: 

private function widgeter($doc) { //Give it an DomDocument HTML containing <widget> elements and will translate them into usable stuff
    $this->_widgetList = $doc->getElementsByTagName($this->_widgetTransformTo);
    foreach ($this->_widgetList as $widget) {
        $data = array();
        if ($widget->hasAttributes()) {
        foreach ($widget->attributes as $attribute) {
            $data[][$attribute->name] = $attribute->value;
            // @TODO: Implements Widget Transformation

        }
        }
        // Next 2 lines are just for debug
        $string = serialize($data);
        $newWidget = $doc->createElement('p', $string);
        $widget->parentNode->replaceChild($newWidget, $widget);
    }
    return $doc;
    }

, затем, когда я сохраняю HTTML () $ doc, я вижу: 

/* Output HTML
  <div id="toobar_top">
    <p>[{"id":"flag_holder"}]</p>
    <widget id="horizontal_menu"></widget>
  </div>
  <div id="header">
    <p>[{"id":"header"}]</p>
  </div>
*/

почему "Horizontal_menu "не было переведено?

Неважно, где находятся виджеты (я пробовал только один div со всеми виджетами в и с div на виджет).

Я не могувыяснить это ...

Ответы [ 2 ]

8 голосов
/ 01 апреля 2011

Это происходит потому, что вы заменяете элементы в DOMNodeList, зацикливаясь на них. DOMNodeList - это не массив , поэтому foreach работает не с копией , а с самим объектом.

По сути, я думаю, что происходит:

  • Вы заменяете первый экземпляр <widget> (Элемент 0).
  • Указатель перемещается к следующему элементу (Элемент 1).
  • Элемент 0 былзаменен и больше не существует.
  • Произошло смещение предмета: Предмет 1 становится Предметом 0, Предмет 2 становится Предметом 1.
  • Указатель по-прежнему указывает на Предмет 1 (который изначально был Предметом 2, фактическипропуская узел).

Вам нужно сохранить элементы в массиве и затем изменить их, вместо зацикливания в DOMNodeList: 

$this->_widgetList = array();
foreach ($domNodeList as $node) {
   $this->_widgetList[] = $node;    
}

foreach ($this->_widgetList as $widget) {
   // do stuff
}
0 голосов
/ 09 июля 2013

Чтобы избежать двух итераций, вы можете разобрать список элементов, обратный 

$widgets = $doc->getElementsByTagName( 'widget' ); // get all elements

for( $i = $widget->length; $i > 0; $i-- ){
    $widget = $doc->getElementsByTagName( 'widget' )->item( $i - 1 );

    // do stuff whith the widget
}
...