Forum PHP.pl > [SOLVED] pobieranie aukcji przez api i paginacja

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [SOLVED] pobieranie aukcji przez api i paginacja

Aquarius

22.07.2015, 12:28:14

witam,
udało mi się przerobić po wielu bojach starą metodę pobierania informacji o aukcjach na nową (w końcu zrozumiałem czemu ktoś w internecie napisał że to Allegro API to koszmar programisty), nie otrzymuję już błędów przy pobieraniu informacji, mam jednak problem z paginacją (stronicowaniem), załączam cały kod, być może przyda się komuś do przejścia na nową metodę:

[PHP] pobierz, plaintext 
$offset = ($_GET['id']) ;
$limit = 12;
$soap = new SoapClient('https://webapi.allegro.pl/service.php?wsdl', array('trace' => true));
 
              $items = $soap->doGetItemsList(
                  array(
                      'webapiKey' => 'XXXXXXXX',
                      'countryId' => 1,
                      'filterOptions' => array(
                          array(
                              'filterId' => 'userId',
                              'filterValueId' => array('XXXXXXX')
                          )
                      )
                  )
              );
 
$items = get_object_vars($items);
$ile_stron = $items['itemsCount'] / $limit;
$pag = floor($ile_stron);
if ($offset >= "1") {
$back = ($_GET['id'] - 1);
}
else {
$back = $pag;
}
if ($offset != $ile_stron) {
$next = ($_GET['id'] + 1);
}
else {
$next = "0";
}
echo "<span style=\"font-size: 16px\"><strong>Strona:</strong></span><br /><div style=\"line-height: 16px;padding-top: 5px\"><a href=\"aukcje?id=".$back."\"><img src=\"images/arrow-left.png\" style=\"padding-right:3px\" /></a> <span style=\"font-size: 16px\"><a href=\"aukcje?id=0\"> 0 </a>";
for($i = 1; $i < $ile_stron; $i++)
{
   echo '<a href="aukcje?id='.$i.'"> '.$i.' </a>';
}
echo " <a href=\"aukcje?id=".$next."\"><img src=\"images/arrow-right.png\" style=\"padding-left:3px\" /></a></span></div><br /><br />";
 
for($i=0;$i<$limit ;$i++)
{
	echo '<div style="margin-right: 10px;margin-bottom: 10px;display: inline-block;">';
    echo '<a target="_blank" href="http://allegro.pl/show_item.php?item='.$items[$i]['itemsList']['itemId'].'"><img src="'.$items[$i]['itemsList']['photosInfo']['photoUrl'].'"/></a>';
    echo '<div style="width: 130px;margin-top: 5px;margin-bottom: 8px;line-height: 16px"><span style="line-height:14px;font-size: 11px;font-weight: bold">'.$items[$i]['itemsList']['itemTitle'].'</span><br />';
	if ($items[$i]['itemsList']['priceInfo']['priceType'] == "buyNow") {
	echo '<span style="line-height:20px;color: red;font-size: 16px;font-weight: bold">'.$items[$i]['itemsList']['priceInfo']['priceValue'].' zł</span></div>';
	}
	else {
	echo '<span style="line-height:20px;color: red;font-size: 16px;font-weight: bold">'.$items[$i]['itemsList']['priceInfo']['priceValue'].' zł</span></div>';
	}
	echo '</div>';
}
[PHP] pobierz, plaintext

problem w tym że serwer zwraca błąd:

Kod

PHP Notice:  Undefined offset: 0
PHP Notice:  Undefined offset: 1
PHP Notice:  Undefined offset: 2
PHP Notice:  Undefined offset: 3
PHP Notice:  Undefined offset: 4
PHP Notice:  Undefined offset: 5
PHP Notice:  Undefined offset: 6
PHP Notice:  Undefined offset: 7
PHP Notice:  Undefined offset: 8
PHP Notice:  Undefined offset: 9
PHP Notice:  Undefined offset: 10
PHP Notice:  Undefined offset: 11

przypuszczam że mam błąd w kodzie, jednak przejrzałem go już tyle razy i nie mam pojęcia w czym problem, proszę o pomoc

IProSoft

22.07.2015, 14:28:45

Co zwróci

[PHP] pobierz, plaintext 
var_dump($items);
[PHP] pobierz, plaintext

Aquarius

23.07.2015, 07:21:03

mniej więcej tak to wygląda:

Kod

array(5) { ["itemsCount"]=> int(501) ["itemsFeaturedCount"]=> int(0) ["itemsList"]=> object(stdClass)#3 (1) { ["item"]=> array(100) { [0]=> object(stdClass)#4 (14) { ["itemId"]=> int(5118545340) ["itemTitle"]=> string(36) "RENAULT TRAFIC 2011 2,5 DCI KOMPLET" ["leftCount"]=> int(1) ["bidsCount"]=> int(0) ["biddersCount"]=> int(0) ["quantityType"]=> string(6) "pieces" ["timeToEnd"]=> string(27) "do wyczerpania przedmiotów" ["categoryId"]=> int(50851) ["conditionInfo"]=> string(4) "used" ["promotionInfo"]=> int(0) ["additionalInfo"]=> int(0) ["sellerInfo"]=> object(stdClass)#5 (5) { ["userId"]=> int(4921350) ["userLogin"]=> string(9) "xxxxxx" ["userRating"]=> int(225) ["userIcons"]=> int(10) ["countryId"]=> int(1) } ["priceInfo"]=> object(stdClass)#6 (1) { ["item"]=> array(2) { [0]=> object(stdClass)#7 (2) { ["priceType"]=> string(6) "buyNow" ["priceValue"]=> float(14000) } [1]=> object(stdClass)#8 (2) { ["priceType"]=> string(12) "withDelivery" ["priceValue"]=> float(14160) } } } ["photosInfo"]=> object(stdClass)#9 (0) { } } [1]=> object(stdClass)#10 (14)

...

pominąłem konwersję na zmienne:

[PHP] pobierz, plaintext 
//$items = get_object_vars($items)
[PHP] pobierz, plaintext

i chcę teraz wywołać dane w ten sposób:

[PHP] pobierz, plaintext 
for($i=0; $i < $limit; $i++)
{
	echo '<div style="margin-right: 10px;margin-bottom: 10px;display: inline-block;">';
        echo '<a target="_blank" href="http://allegro.pl/show_item.php?item='.$items[$i]->itemsList->itemId.'"><img src="'.$items[$i]->itemsList->photosInfo->photoUrl.'"/></a>';
}
[PHP] pobierz, plaintext

jednak tak jest jeszcze gorzej:

Kod

PHP Fatal error: Cannot use object of type stdClass as array

thek

23.07.2015, 08:18:54

Przyjrzyj się strukturze zwrotnej z serwisu i dopasuj do tego pętlę

var_dump Ci jasno pokazuje ją, a PHP także daje wskazówkę, że coś nie tak jest z indeksem. Może po prostu iterujesz nie po tym co trzeba?

Jeśli nadal nie kumasz to rozpiszę Ci to nieco według tego co pokazuje var_dump jaki dałeś:

Kod

$items = array(5) {
   ["itemsCount"]=> int(501) <- element tablicy typu int
   ["itemsFeaturedCount"]=> int(0)  <- element tablicy typu int
   ["itemsList"]=> object(stdClass)#3 (1) { element tablicy typu object
      ["item"]=> array(100) { <- atrybut obiektu typu array
         [0]=> object(stdClass)#4 (14) { element tablicy typu object
            ["itemId"]=> int(5118545340) <- atrybut obiektu typu int
(...)

Widzisz dlaczego $items[0] zawoła błędem? Oraz dlaczego $items[0]['itemsList']['itemId'] także się wysypie? Wcale nie patrzysz na dane zwracane, ich strukturę i typy oraz z bardzo radosną twórczością podchodzisz do nich

Popatrz na to jak ja rozpisałem co jest czym i wyciągnij wnioski kiedy używać [], kiedy -> a kiedy użyć pętli...

EDIT: I uważaj na zawartość "itemsList" oraz "item", bo "itemsCount" sugeruje istnienie 501 przedmiotów, ale już "item" wskazuje 100, a reszty var_dump brak, więc ciężko wyrokować o dalszej strukturze całości i jak obie wartości współgrają ze sobą. Myślę, że ta mała uwaga naprowadzi Cię na to, gdzie byka w logice robisz.

Aquarius

23.07.2015, 08:30:04

ok, właśnie dochodziłem do tego że trzeba każdy element tej pętli "wylistować" aby otrzymać element docelowy, pomogło rozpisanie sobie struktury całej pętli elementów obiektu do nowej linii, każdą pętlę elementu do nowej itd.,
czyli:

[PHP] pobierz, plaintext 
for($i=0; $i < $limit; $i++)
{
    echo '<div style="margin-right: 10px;margin-bottom: 10px;display: inline-block;">';
	echo '<a target="_blank" href="http://allegro.pl/show_item.php?item='.$items->itemsList->item[0]->itemId.'"><img src="'.$items->itemsList->item[0]->photosInfo->item[1]->photoUrl.'"/></a>';
}
[PHP] pobierz, plaintext

i to działa już dobrze, informacje pobrane i poprawnie wyświetlone

słabo u mnie z pisaniem obiektowym i to tylu elementów, czas to podciągnąć bo to podobno przyszłość PHP, pytanie teraz jak to zapętlić aby otrzymać te 12 aukcji? jak tylko próbuję zapętlić wywala mi błąd że nie można zapętlić obiektu:

Kod

Cannot use object of type stdClass as array

co do item które podaje wartość 100 elementów to dlatego że Allegro ma limit pobierania 100 aukcji jednocześnie, aby pobrać większą zawartość trzeba to zdefiniować przy inicjacji połączenia api

thek

23.07.2015, 08:42:11

To może napisz jak tę pętlę chcesz przeprowadzić? Bo sprawa jest prosta jeśli tylko chwilkę pomyślisz. To wariacja na temat jednej pętli, numeru strony i limitu na stronę. Inna sprawa że użycie floor do wyliczania ilości stron wprowadzi Cię w mały kłopot, skoro mając 14 elementów floor(14/12) = 1?

EDIT: Jeśli limit przedmiotów to 100, to chyba warto też zwrócić uwagę czy dostajemy 100 czy tyle ile wskazuje "itemsCount", bo inaczej z paginacją też będą cyrki

bo 501 podzielisz na 12, da Ci to około 40 stron, ale tylko 9 będzie cokolwiek zawierać

Aquarius

23.07.2015, 08:52:18

zapętlać się zapętla do tych 12 elementów, ale dostaję 12 tych samych, jak miałem zmienne zamiast obiektów w poprzedniej metodzie pobierania aukcji to najnormalniej zwiększałem numer wyświetlanej aukcji poprzez

Kod

[$i]

no i pętla for

[PHP] pobierz, plaintext 
for($i=0; $i < $limit; $i++)
[PHP] pobierz, plaintext

robiła swoje

tutaj właśnie kiedy chcę zapętlić dostaję informację

Kod

Cannot use object of type stdClass as array

co do limitów API - powinienem oczywiście to inaczej zrobić, mianowicie definiować na każdej stronie że chcę pobrać 12 aukcji, na każdej kolejnej również 12 począwszy od (strona poprzednia * 12), ale problem sprawiało mi samo wyświetlanie jak na razie

thek

23.07.2015, 08:55:32

Chodzi mi o to po czym iterujesz... gdzie jest to $i, które potem łapiesz. Jak wygląda Twoje wnętrze pętli for. Bo to tam siedzi diabeł

Aquarius

23.07.2015, 09:00:14

starą dobrą metodą prób i błędów sprawdzałem i na początku:

[PHP] pobierz, plaintext 
for($i=0; $i < $limit; $i++)
{
    echo '<a target="_blank" href="http://allegro.pl/show_item.php?item='.$items[$i]->itemsList->item[0]->itemId.'"><img src="'.$items[$i]->itemsList->item[0]->photosInfo->item[1]->photoUrl.'"/></a>';
}
[PHP] pobierz, plaintext

i na końcu:

[PHP] pobierz, plaintext 
for($i=0; $i < $limit; $i++)
{
   echo '<a target="_blank" href="http://allegro.pl/show_item.php?item='.$items->itemsList->item[0]->itemId[$i].'"><img src="'.$items[$i]->itemsList->item[0]->photosInfo->item[1]->photoUrl[$i].'"/></a>';
}
[PHP] pobierz, plaintext

zazwyczaj problemu ze zrozumieniem że tak jest źle albo tak jest dobrze nie mam, ale pierwszy raz trafiło mi się zderzyć z taką ilością elementów i pętli w obrębie jednego obiektu, oczywiście zdaję sobie sprawę że tak to nie może być bo oznaczałoby to element w strukturze samego obiektu a nie iterację

aniolekx

23.07.2015, 09:36:53

[PHP] pobierz, plaintext 
foreach ($items->itemsList->item as $item) {
{
    echo '<a target="_blank" href="http://allegro.pl/show_item.php?item='.$item->itemId.'"><img src="'.$item->photosInfo->item[1]->photoUrl.'"/></a>';
}
[PHP] pobierz, plaintext

chyba tak bedzie czytelniej?

thek

23.07.2015, 09:45:00

W takim wypadku chyba nadal nie widzisz gdzie jest Twoja tablica z obiektami. Chyba zrobię to co poprzednio i Ci to rozpiszę

Kod

$items = array(5) {
   ["itemsCount"]=> int(501) <- element tablicy zawierający liczbę przedmiotów
   ["itemsFeaturedCount"]=> int(0)  <- element tablicy zawierający liczbę przedmiotów wyszczególnionych
   ["itemsList"]=> object(stdClass)#3 (1) { element tablicy typu object zawierający listę wszystkich informacji związanych z przedmiotami
      ["item"]=> array(100) { <- atrybut obiektu  będący tablicą przedmiotów
         [0]=> object(stdClass)#4 (14) { element tablicy typu object będący konkretnym przedmiotem
            ["itemId"]=> int(5118545340) <- atrybut przedmiotu
(...)

Aquarius

23.07.2015, 10:23:17

zrobiłem tak jak się należało od początku, czyli pobranie 12 aukcji (a nie 100), dla każdej strony inną porcję zaczynając tak jak pisałem wcześniej od

[PHP] pobierz, plaintext 
$resultOffset = ($offset - 1) * 12;
[PHP] pobierz, plaintext

i tu już było łatwiej dla mnie bo wystarczyło zapętlić w pętli foreach tak jak zasugerował aniolekx

w oczekiwaniu na odpowiedzi do tematu poczytałem trochę o zapętlaniu obiektów, ale nie wpadłem na żaden trop jaki podsuwałeś thek, być może będę miał jeszcze w przyszłości okazję (czyt. potrzebę)
w każdym razie sposób z pobieraniem od razu właściwej porcji aukcji jest poprawniejszy,

podziękował pięknie za podpowiedzi

PS. co do funkcji floor - uważałem że jest ona tutaj właściwa, ale zaokrąglenie w dół np. 41,75 strony da mi 41 stron czyli niewłaściwą ilość (41 * 12 = 492), w moim przypadku przy 12 wynikach na stronę, zamieniłem ją na ceil

thek

23.07.2015, 12:25:03

I o to mi chodziło co rzucił @aniolekx

Chciałem byś zauważył, że u Ciebie iteracja idzie po $items->itemsList->item, bo to jest wlaściwa tablica obiektów klasy stdClass

Ty wcześniej iterowałeś albo za płytko (po $items), więc rzucało Ci undefined index, albo za głęboko ($items->itemsList->item[0]->itemId), co kończyło się próbą iteracji po liczbie typu int

A nie sugerowałem Ci foreach, bo foreach potrafi iterować po publicznych atrybutach obiektu. To by Cię mogło zupełnie w błąd wprowadzić. Dlatego taki nacisk położyłem na zrozumienie przez Ciebie struktury odpowiedzi zwracanej przez serwer. Tak byś sam załapał co jest nie tak.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.