Как правильно прописать функцию при выводе в ассоциативном массиве xpath запроса?

Question

[ayapergenov]

Нужно массивом передать вывод списка xpath запроса. Прописал xpath правильно и даже могу вывести отдельный результат (Пример закомментировал в коде).
Как исправить ошибку Notice: Undefined property: DOMNodeList::$nodeValue ?

<?php
$specoffer = "http://www.msk.lenta.com/show/product/92003/";
libxml_use_internal_errors(true);
$doc = new DOMDocument();
libxml_clear_errors();

$doc->loadHTMLFile($specoffer);
$doc->saveHTML();

$doc->preserveWhiteSpace = false;
$xpath = new DOMXPath($doc);

$prices_name = "Цена: ";
$discounts_name = "Цена со скидкой: ";
$goodsnames_name = "Название: ";
$imglinks_name =  "Ссылка на изображение: ";
$goodsdesriptions_name = "Описание: ";

$prices_value = $xpath->query("/html/body/div[1]/div/div/div/section/div/div/main/div[2]/div[2]/div[1]/div/span[2]");
$discounts_value = $xpath->query("/html/body/div[1]/div/div/div/section/div/div/main/div[2]/div[2]/div[1]/div/span[1]");
$goodsnames_value = $xpath->query("/html/body/div[1]/div/div/div/section/div/div/main/div[2]/div[2]/h3");
$imglinks_value =  $xpath->query("/html/body/div[1]/div/div/div/section/div/div/main/div[2]/div[1]/a/img");
$goodsdesriptions_value = $xpath->query("/html/body/div[1]/div/div/div/section/div/div/main/div[2]/div[2]/p[1]");

$arr = array(
            $prices_name => $prices_value,
            $discounts_name => $discounts_value,
            $goodsnames_name => $goodsnames_value,
            $imglinks_name => $imglinks_value,
            $goodsdesriptions_name => $goodsdesriptions_value
            );

foreach($arr as $key => $value)
  {
     echo $key . $value->nodeValue . "<br />";
  }
// Рабочий вариант
// foreach ($prices_value as $price) {
//      echo "Цена: {$price->nodeValue} <br>";
// }

Answer 1

[Ilya]

Чтобы получить единичное значение после xpath-запроса, вам надо использовать метод item().
То есть, условно:

$prices_value = $xpath->query("//div/span[2]");
if($prices_value !== FALSE && $prices_value->length > 0)
   echo $prices_value->item(0)->nodeValue; // тут ваше значение

Без использования item() вы можете обходиться только если итерируете результаты query(), например:

$prices_value = $xpath->query('//div/span');
if($prices_value !== FALSE && $prices_value->length > 0)
{
   foreach($items as $item)
      echo $item->nodeValue;
}

И это уже немного за рамками вашего вопроса, но я бы рекомендовал сократить xpath-выражения в query(). Я понимаю, что вы их скопировали из Firebug или другой вед-дев тулзы, но в вашем случае минимальное изменение верстки донора приведет к отказу работы ваших xpath-запросов.

Comments

[ayapergenov]

Спасибо за обширные ответ. Получилось пофиксить. Можно ли в данном массиве вывести url в 4 строке массива? А то у меня везде идет запрос на nodeValue?

[ayapergenov]

По поводу xpath-выражений я пробовал использовать выражения из FireXpath, но браузер ругается.

[Ilya]

ayapergenov: Рад помочь. По поводу url - легко, есть 2 пути. Первый - использовать метод getAttribute() для выбранного элемента, то есть

$query = $xpath->query('//img');
$img = $query->item(0);
echo  $img->getAttribute('src');

Второй вариант: внутри xpath-запроса обращаться к атрибуту, то есть
$query = $xpath->query('//img/@src');

Первый вариант считаю более грамотным.

[ayapergenov]

Ilya: Благодарю за помощь. Получилось!

[Ilya]

ayapergenov: Прекрасно.

[Ilya]

ayapergenov: по поводу xpath-выражений - сокращайте до более простых и универсальных, используя обращения по классу, id или другим атрибутам тегов, даже используя данные из соседних тегов. То есть, к примеру, чтобы получить список тегов на любой странице вопроса на Тостере, вам надо лишь сделать такого рода запрос
$query = $xpath->query('//ul[@class="tags-list"/li);
И вот вы уже получаете список всех тегов.

В случае копирования из дев-тулзы Хрома мы получим вот такой ужас
$query = $xpath->query('//*[@id="question_show"]/div[1]/nav/ul/li[1]'); // к первому тегу

А в случае с Firebug и того хуже
$query = $xpath->query('/html/body/div[2]/div/div[1]/section/div/main/div[3]/div[1]/nav/ul/li[1]'); // к первому тегу

[ayapergenov]

Ilya: Сегодня бился целый день пытаясь получить dom дерево другой страницы. Вместо списка товаров получаю какие-то шаблоны в фигурных кавычках.
<?php
$specoffer = "www.msk.lenta.com/special-nye-predlozheniya";
libxml_use_internal_errors(true);
$doc = new DOMDocument();
$doc->loadHTMLFile("$specoffer");
libxml_clear_errors();
print $doc->saveHTML();

Блок который мне нужно парсить - Специальные предложения выглядит так:

{{{short_text1}}} {{#short_text2}}
{{{short_text2}}} {{/short_text2}}
{{new_price}} {{#old_price}}{{old_price}}{{/old_price}}
{{/items}}

//Как получить доступ к списку товаров по данным шаблонам?

[Ilya]

ayapergenov: Там используется шаблонизатор Mustache, который на клиенте уже преобразовывает данные в DOM-дерево. Выхода у вас 2:
1. Брать массив myTovarList.items из страницы (смотрите исходники) и просто вытащить оттуда данные.
2. Рендерить страницу как браузер, то есть использовать CasperJS например. Это сложнее немного.

[ayapergenov]

Ilya: Спасибо, будем тащить.

[Ilya]

ayapergenov: Рад помочь

[ayapergenov]

Ilya: Илья, будьте добры. Вразумите бестолкового.
Пытаюсь в итоге получить массив массивов через var_dump со страницы со списком продуктов. Получается,
Главный массив - это список продуктов
Массивы второго уровня содержат свойства каждого товара.
Например я получаю через xpath нужные ноды по имени класса
Затем описываю двухуровневый ассоциативный массив. в котором через $goodsnames->item($i)->nodeValue получаю нужные значения. И в итоге через foreach пытаюсь сделать обход всех товаров. Но тут еще надо чтобы и item как-то перебирался. Не могу все это связать никак. Будьте добры, подскажите.

[Ilya]

ayapergenov: Честно сказать, я не совсем понял, в чем именно проблема. Получаете в массив $items список товаров со страницы списка, ключом для каждого элемента массива берете какое-то уникальное значение, например артикул товара. Потом при заходе в каждую карточку товара, где обычно тоже есть значение артикула, в $items по ключу-артикулу записываете нужные данные.
$items['code2045'] = array('name' => 'Супер-товар', 'weight' => '2.5 кг');

Нет артикула, берите id из url товара или любое другое уникальное значение.

[ayapergenov]

Имеется:
//запрос к классу каталога
$queryList = $xpath->query('//div[@class="cleara"]');
//запрос ко всем продуктам по классу.
$queryItem = $xpath->query('//div[@class="catalog__item"]');
//Запросы к одинаковым свойствам каждого отдельно взятого продукта.
Я создал 2уровневый ассоциативный массив в котором на 1 уровне хочу список продуктов
А на втором уровне свойства каждого из них: цена, название, описание.
Получилось вывести var_dump
array (size=2)
'catalog' =>
object(DOMNodeList)[73]
public 'length' => int 69
'keyItem' =>
array (size=5) со списком свойств первого продукта.
Делаю foreach
foreach ($queryList as $key => $value) {
$keyItem = $queryItem;
foreach ($keyItem as $key => $value) {
}
}
Но ожидаемого списка продукт-свойство не выходит

[Ilya]

ayapergenov: для начала переделайте конструкцию с foreach в такой вид:

foreach ($queryList as $key => $value) {
     $keyItem = $queryItem;
     foreach ($keyItem as $key_i => $value_i) {
    }
}

потому что $key и $value перезаписываются. В остальном, извините, помочь не могу, потому как надо видеть весь код, иначе это просто на гуще гадание. Да и со временем напряженно.

[ayapergenov]

Ilya: В том то и дело, что пока я не использовал уникального значения. У меня все лежит в одном классе. Может ли foreach просто представить объекты каждого отдельного продукта в классе?
$queryList - содержит родительский div из 70 дочерних div с классом .catalog_item
$queryItem - также содержит те же 70 div но уже класс .catalog_item является собственным для каждого из div
Как правильно организовать массив и его foreach в данном примере?

[ayapergenov]

Ilya: Вот весь код

//Queries by classNames
$queryList = $xpath->query('//div[@class="cleara"]');
$queryItem = $xpath->query('//div[@class="catalog__item"]');
$queryNames = $xpath->query('//a[@class="goods__title"]');
$queryPrices = $xpath->query('//span[@class="price_block__cur-price"]');
$queryDiscounts = $xpath->query('//span[@class="price_block__old-price"]');
$queryImglinks =  $xpath->query('//img[@class="catalog__item__image"]/@src');
$queryDesriptions = $xpath->query('//div[@class="catalog__item__text"]/p');

//Massive
$queryList = array(
            $keyItem => $queryItem = array(
                                $keyName => $queryNames->item(0)->nodeValue,
                                $keyPrice => $queryPrices->item(0)->nodeValue,
                                $keyDiscount => $queryDiscounts->item(0)->nodeValue,
                                $keyImgurl => $queryImglinks->item(0)->nodeValue,
                                $keyDescription => $queryDesriptions->item(0)->nodeValue
                                )
                );

//Loops for massives
foreach ($queryList as $keyItem => $queryItem) {
    foreach ($queryItem as $key => $value) {

    }
}

var_dump($queryList);

[Ilya]

ayapergenov: честно? это полный бред. переписать с нуля.

[ayapergenov]

Ilya: Переписал так:

$specoffer = "source/specoffers.html";
libxml_use_internal_errors(true);
$doc = new DOMDocument();
$doc->loadHTMLFile("$specoffer");
libxml_clear_errors();
$doc->saveHTML();

$doc->preserveWhiteSpace = false;
$xpath = new DOMXPath($doc);

//Queries by classNames
$queryItem = $xpath->query('//div[@class="catalog__item"]');

foreach ($queryItem as $goodsItem) {
    $result = array (
        $goodsItem = array (
                "queryName" => $xpath->query('//a[@class="goods__title"]', $goodsItem),
                "queryPrice" => $xpath->query('//span[@class="price_block__cur-price"]', $goodsItem),
                "queryDiscount" => $xpath->query('//span[@class="price_block__old-price"]', $goodsItem),
                "queryImglink" => $xpath->query('//img[@class="catalog__item__image"]/@src', $goodsItem),
                "queryDesription" => $xpath->query('//div[@class="catalog__item__text"]/p', $goodsItem)
                    )
        );
}
var_dump($result);

[ayapergenov]

Ilya: Получаю массив с коллекцией элементов всех продуктов в одном массиве.
Как правильно организовать массив, чтобы получить массивы каждого продукта отдельно? Чтобы получить такой результат:
array(
'goodsItem' =>
array('name' => 'milk', 'price' => 10),
array('name' => 'meat', 'price' => 20),
array('name' => 'bread', 'price' => 1),
)

[Ilya]

ayapergenov:
$doc->loadHTMLFile("$specoffer"); заменить на $doc->loadHTMLFile($specoffer);
$doc->saveHTML(); - не нужно

$result = array();
$i = 0;
foreach ($queryItem as $goodsItem) {
$result['goodsItem'][$i]['name'] = $xpath->query('//a[@class="goods__title"]', $goodsItem);
$result['goodsItem'][$i]['price'] = $xpath->query('//span[@class="price_block__cur-price"]', $goodsItem);
/// и т.д.

$i++;
}
var_dump($result);

[ayapergenov]

Ilya: Благодарю Илья. Подобную реализацию я уже делал.
А как можно реализовать без использования $i?
Вот так я получаю массив из массивов всех элементов, но когда прописываю ->item(0)->nodeValue;
То получаю итерации с первым продуктом во всех массивах.

$queryItem = $xpath->query('//div[@class="catalog__item"]');
        foreach ($queryItem as $goodsItem) {
                $result[] = $goodsItem = array (
                        "queryName" => ($xpath->query('//a[@class="goods__title"]', $goodsItem)),
                        "queryPrice" => $xpath->query('//span[@class="price_block__cur-price"]', $goodsItem),
                        "queryDiscount" => $xpath->query('//span[@class="price_block__old-price"]', $goodsItem),
                        "queryImglink" => $xpath->query('//img[@class="catalog__item__image"]/@src', $goodsItem),
                        "queryDesription" => $xpath->query('//div[@class="catalog__item__text"]/p', $goodsItem)
                                    );
                }
        var_dump($result);

Answer 2

[Михаил Соколов]

Заменить
$value->nodeValue
на
$value->item(0)->nodeValue

Comments

[Михаил Соколов]

php.net/manual/ru/domnodelist.item.php