[PHP]Pętle i wyciąganie zamówień Opcje

[xaitras]

Witam wszystkich

Już dawno nie potrzebowałem pomocy ale znowu poległem.
Myślę, że to dla was prosta sprawa ale dla mnie pętle to czasami czarna magia

Opiszę najpierw problem:

Mam zbiór zamówień które wyciągam przez API (format danych json) - ale to ma mniejsze znaczenie.
Problem w tym, że jednorazowo mogę pobrać tylko 100 zamówień.
Próbuję na razie na prostym liczniku:

[PHP] pobierz, plaintext 
include_once('funkcje.php');
polacz_bl_przelewy();
$counter = 0;
foreach($dane['orders'] as $orders) {
$nr_zam = $orders['order_id'];
$counter++;
}
echo $counter;
[PHP] pobierz, plaintext 

Ale w takiej postaci liczy tylko do 100.
Za każdym razem kiedy łączę się z API musiałbym podać nowy numer zamówienia na którym skończył liczyć (taka opcja w api istnieje).
Teraz pytanie:

JAK TO ZROBIĆ Wiem, że potrzebna jest druga pętla tylko nie wiem jak ją zastosować.

Pomóżcie bo mózg mi się gotuje..

Ten post edytował xaitras 13.11.2017, 10:52:44

[trzczy]

Nic nie rozumiem z twego kodu. Może napisz
jak wygląda zamówienie pierwszej setki,
potem napisz jak wygląda zamówienie drugiej setki,
potem napisz jak wygląda zamówienie trzeciej setki.

Być może wtedy widoczna będzie jakaś reguła, która pozwoli wymyślić pętlę automatyczną.

Ten post edytował trzczy 13.11.2017, 11:09:25

[xaitras]

Ok, trochę to jest skomplikowane ale postaram się:

- kod który pokazałem jest w stanie policzyć tylko 100 pozycji bo na więcej nie pozwala API.

- oryginalnie schemat pobierania jest taki:

1. Ustalenie początkowej daty i podanie jej w polu date_confirmed_from
2. Przetworzenie wszystkich odebranych zamówień. Jeśli odebrano 100 zamówień, oznacza to, że do pobrania może być ich jeszcze więcej.
3. Pobranie kolejnej paczki zamówień, podając w polu date_confirmed_from wartość z date_confirmed ostatnio pobranego zamówienia. Operacja powtarzana aż do otrzymania paczki z ilością zamówień mniejszą niż 100 (oznacza to że nie ma więcej zamówień do pobrania).
4. Zapisanie date_confirmed ostatnio przetworzonego zamówienia. Kolejne pobranie zamówień można wykonać od tej daty, aby nie pobierać dwukrotnie tych samych

Jednak istnieje też możliwość podania numeru zamówienia zamiast daty ("order_id").

Trzeba podawać ostatni otrzymany numer czyli: 100,200,300 itd. Podaje się go w parametrach połączenia.


- potrzebuję pętli która będzie wyciągać zamówienia po 100 aż do momentu kiedy osiągnie 0


Kod do połączenia z api:

[PHP] pobierz, plaintext 
function polacz()
{
global $zam;
$params = array ('token' => 'xxxxx','method' => 'getOrders','parameters' => '{"order_id": '.$zam.',"status_id": 40196, "get_unconfirmed_orders": true}');
$query = http_build_query ($params);
$contextData = array ('method' => 'POST','header' => "Content-Type: application/x-www-form-urlencoded\r\n"."Content-Length: ".strlen($query)."\r\n"."User-Agent:MyAgent/1.0\r\n",'content'=> $query );
$context = stream_context_create (array ( 'http' => $contextData ));
$result =  file_get_contents ('https://xxx/xxx.php', false, $context);
global $dane;
$dane = json_decode($result, true);
return $dane;
}
[PHP] pobierz, plaintext 

[nospor]

Moze jednak podasz nam dokumentacje tego API?

[xaitras]

Proszę uprzejmie

https://api.baselinker.com/?method=getOrders

[trzczy]

Trzeba podawać ostatni otrzymany numer czyli: 100,200,300 itd. Podaje się go w parametrach połączenia.

No to jak ten numer, np. 100, dodać tu do parametrów?

[PHP] pobierz, plaintext 
$params = array ('token' => 'xxxxx','method' => 'getOrders','parameters' => '{"order_id": '.$zam.',"status_id": 40196, "get_unconfirmed_orders": true}');
[PHP] pobierz, plaintext 

Jak se odpowiesz na to pytanie, to napisz funkcję pobierającą w zależności od tego numeru:

[PHP] pobierz, plaintext 
function getData($number) {
//...
}
[PHP] pobierz, plaintext 

Potem napisz pętlę for:

[PHP] pobierz, plaintext 
for($i=0; $i<$total;$i=$i + 100) {
    //...
    getData($i);
    //...
}
[PHP] pobierz, plaintext 

Wartość $total wyciągasz wcześniej za pomącą API, bo powinno to być w API.

Ten post edytował trzczy 13.11.2017, 12:22:36

[nospor]

No masz podany algorytm jak to masz robic. Zapisujesz date ostatniego i lecisz dalej. W czym problem? Gdy liczba zwroconych rekordow mniejsza od 100 to wychodzisz z petli

[PHP] pobierz, plaintext 
$lastDate = null;
while (true) {
//pobierz paczke
 
if (count($daneZPaczki) < 100) {
break;
}
 
$lastDate = 'tu zapisz date ostatniego';
 
}
 
[PHP] pobierz, plaintext 

[xaitras]

No to jak ten numer, np. 100, dodać tu do parametrów?

[PHP] pobierz, plaintext 
$params = array ('token' => 'xxxxx','method' => 'getOrders','parameters' => '{"order_id": '.$zam.',"status_id": 40196, "get_unconfirmed_orders": true}');
[PHP] pobierz, plaintext 

Jak se odpowiesz na to pytanie, to napisz funkcję pobierającą w zależności od tego numeru:

[PHP] pobierz, plaintext 
function getData($number) {
//...
}
[PHP] pobierz, plaintext 

Potem napisz pętlę for:

[PHP] pobierz, plaintext 
for($i=0; $i<$total;$i=$i + 100) {
    //...
    getData($i);
    //...
}
[PHP] pobierz, plaintext 

Wartość $total wyciągasz wcześniej za pomącą API, bo powinno to być w API.

No problem w tym że w tablicy nie znalazłem informacji na temat liczby zapisanych zamówień. Z takim parametrem byłoby łatwiej.
Nie wiem jak wyciągnąć ilość zamówień w danym statusie..

[nospor]

trzczy nie przeczytal dokumentacji i wymysla rzeczy, ktore sie maja nijak do ciebie. Podalem ci poprawne rozwiazanie,zgodne z dokumentacja

[xaitras]

trzczy nie przeczytal dokumentacji i wymysla rzeczy, ktore sie maja nijak do ciebie. Podalem ci poprawne rozwiazanie,zgodne z dokumentacja

Wcześniej próbowałem z datą. Mój problem polega na tym, że pobieram też zamówienia które nie są potwierdzone a to koliduje z tą metodą.
Dlatego chciałem pobierać je po numerze 'order_id'

Siedzę nad tym już trochę czasu i wszystko mi się miesza.

Mógłbyś mi podpowiedzieć jak policzyć wszystkie zamówienia znajdujące się w danym statusie?
Potem już sobie poradzę z pobieraniem zamówień itd.

Żeby było jasne,nie chcę gotowego skryptu.. Niejasny jest dla mnie fragment kodu:

[PHP] pobierz, plaintext 
while (true) {
//pobierz paczke
[PHP] pobierz, plaintext 

wcześniej miałem:

[PHP] pobierz, plaintext 
foreach($dane['orders'] as $orders) {
$zam = $orders['order_id'];
}
[PHP] pobierz, plaintext 

Ten post edytował xaitras 13.11.2017, 13:43:37

[nospor]

Wcześniej próbowałem z datą. Mój problem polega na tym, że pobieram też zamówienia które nie są potwierdzone a to koliduje z tą metodą.
Dlatego chciałem pobierać je po numerze 'order_id'

Nic nie koliduje. Poprostu zamiast
date_confirmed_from
dajesz
date_from

[xaitras]

ok w funkcji do połączenia mam coś takiego:

[PHP] pobierz, plaintext 
function polacz_bl_przeniesione_spec()
{
global $lastDate;
$params7 = array ('token' => 'xxxx','method' => 'getOrders','parameters' => '{"status_id": 40196, "date_from": '.$lastDate.',"get_unconfirmed_orders": true}');
$query7 = http_build_query ($params7);
$contextData7 = array ('method' => 'POST','header' => "Content-Type: application/x-www-form-urlencoded\r\n"."Content-Length: ".strlen($query7)."\r\n"."User-Agent:MyAgent/1.0\r\n",'content'=> $query7 );
$context7 = stream_context_create (array ( 'http' => $contextData7 ));
$result7 =  file_get_contents ('https://api.baselinker.com/connector.php', false, $context7);
global $dane7;
$dane7 = json_decode($result7, true);
return $dane7;
}
[PHP] pobierz, plaintext 

A próbuję pobierać tak:

[PHP] pobierz, plaintext 
include_once('funkcje.php');
$zam = 0;
$lastDate = 0;
while (true) {
 
polacz_bl_przeniesione_spec();
foreach($dane7['orders'] as $orders) {
$zam = $orders['order_id'];
$data = $orders['date_from'];
echo $zam."<br>";
}
 
if (count($zam) < 100) {
break;
}
 
$lastDate = $data;
 
}
[PHP] pobierz, plaintext 

efekt:

Notice: Undefined index: date_from in C:\WebServ\httpd\PANEL\test.php on line 12
21406605

Notice: Undefined index: date_from in C:\WebServ\httpd\PANEL\test.php on line 12
21413239

Notice: Undefined index: date_from in C:\WebServ\httpd\PANEL\test.php on line 12
21422195

Notice: Undefined index: date_from in C:\WebServ\httpd\PANEL\test.php on line 12
21422499


itd.


zmieniłem

[PHP] pobierz, plaintext 
$data = $orders['date_from'];
[PHP] pobierz, plaintext 

na

[PHP] pobierz, plaintext 
$data = $orders['date_add'];
[PHP] pobierz, plaintext 

pobiera ale tylko 100 szt. jak pobrać następne i następne... ?


Ten post edytował xaitras 13.11.2017, 14:33:41

[nospor]

$zam = $orders['order_id'];
if (count($zam) < 100) {

Naprawde nie widzisz tu bledu?

poza tym date masz przekazac do polacz_bl_przeniesione_spec() jako parametr

[xaitras]

$zam = $orders['order_id'];
if (count($zam) < 100) {

Naprawde nie widzisz tu bledu?

poza tym date masz przekazac do polacz_bl_przeniesione_spec() jako parametr

Może faktycznie już nie dowidzę..
Ale wierz mi próbuję na wszystkie sposoby..

Datę przekazuję do parametrów połączenia:

[PHP] pobierz, plaintext 
$lastDate = $data;
[PHP] pobierz, plaintext 

a w połączeniu:

[PHP] pobierz, plaintext 
 'parameters' => '{"status_id": 40196, "date_from": '.$lastDate.',"get_unconfirmed_orders": true}'
[PHP] pobierz, plaintext 

Podpowiesz co robię źle? Pętle to nie jest moja mocna strona..
Z resztą sobie radzę..

Ten post edytował xaitras 13.11.2017, 14:46:33

[nospor]

Z resztą sobie radzę..

No wlasnie nie bardzo....:
nie wiesz co to parametr funkcji
nie wiesz na czym robi sie count(), nawet jak ci palcem wytkna
Obie te rzeczy nie zaliczaja sie do petli wiec nie tylko z petlami sobie nie radzisz

[xaitras]

No wlasnie nie bardzo....:
nie wiesz co to parametr funkcji
nie wiesz na czym robi sie count(), nawet jak ci palcem wytkna
Obie te rzeczy nie zaliczaja sie do petli wiec nie tylko z petlami sobie nie radzisz

Zgadzam się z tobą i właśnie po to jest forum i ten dział żeby się czegoś nauczyć.. mam rację?

ok zróbmy to bez funkcji:

[PHP] pobierz, plaintext 
$zam = '0';
$lastDate ='0';
 
while (true) {
 
$params7 = array ('token' => 'xxxxx','method' => 'getOrders','parameters' => '{"status_id": 40196, "date_from": '.$lastDate.',"get_unconfirmed_orders": true}');
$query7 = http_build_query ($params7);
$contextData7 = array ('method' => 'POST','header' => "Content-Type: application/x-www-form-urlencoded\r\n"."Content-Length: ".strlen($query7)."\r\n"."User-Agent:MyAgent/1.0\r\n",'content'=> $query7 );
$context7 = stream_context_create (array ( 'http' => $contextData7 ));
$result7 =  file_get_contents ('https://api.baselinker.com/connector.php', false, $context7);
$dane7 = json_decode($result7, true);
 
 
foreach($dane7['orders'] as $orders) {
$zam = $orders['order_id'];
$data = $orders['date_add'];
echo $zam."<br>";
}
 
if (count($orders) < 100) {
break;
}
 
$lastDate = '$data';
}
[PHP] pobierz, plaintext 

Pomoże ktoś


Ten post edytował xaitras 13.11.2017, 15:38:11

[nospor]

Zgadzam się z tobą i właśnie po to jest forum i ten dział żeby się czegoś nauczyć.. mam rację?

oczywiscie ze masz Chcialem jedynie cie poprawic, bo napisales ze tylko z petlami sobie nie radzisz. Chcialem cie nakierunkowac ze pare innych rzeczy tez nie ogarniasz.

Skoro juz to wyjasnilismy sobie, to teraz zajrzyj do manuala i poczytaja o parametrach funkcji. Date masz przekazac jako parametr funkcji

Co do count() to masz zliczyc zdaje sie liczbe zamowien a nie liczbe elementow w zamowieniu

wiec nie:
if (count($orders) < 100) {

a (wg ostatniego kodu co wrzuciles)
if (count($dane7['orders']) < 100) {

bo zdaje sie w $dane7['orders'] masz zamowienia, co nie?

edit:
no i to:

$lastDate = '$data';

No to juz naprawde podstawy podstaw niemajace z petlami zadnego zwiazku :/ Tak ma byc
$lastDate = $data;

[xaitras]

oczywiscie ze masz Chcialem jedynie cie poprawic, bo napisales ze tylko z petlami sobie nie radzisz. Chcialem cie nakierunkowac ze pare innych rzeczy tez nie ogarniasz.

Skoro juz to wyjasnilismy sobie, to teraz zajrzyj do manuala i poczytaja o parametrach funkcji. Date masz przekazac jako parametr funkcji

Co do count() to masz zliczyc zdaje sie liczbe zamowien a nie liczbe elementow w zamowieniu

wiec nie:
if (count($orders) < 100) {

a (wg ostatniego kodu co wrzuciles)
if (count($dane7['orders']) < 100) {

bo zdaje sie w $dane7['orders'] masz zamowienia, co nie?

edit:
no i to:

$lastDate = '$data';

No to juz naprawde podstawy podstaw niemajace z petlami zadnego zwiazku :/ Tak ma byc
$lastDate = $data;

[PHP] pobierz, plaintext 
include("funkcje.php");
 
$zam = '0';
$lastDate ='0';
 
while (true) {
 
polacz_bl_przeniesione_spec($lastDate);
foreach($dane7['orders'] as $orders) {
$zam = $orders['order_id'];
$data = $orders['date_add'];
echo $zam."<br>";
}
 
if (count($orders) < 100) {
break;
}
 
$lastDate = $data;
 
}
[PHP] pobierz, plaintext 

w połączeniu:

[PHP] pobierz, plaintext 
function polacz_bl_przeniesione_spec(&$lastDate)
{
global $lastDate;
$params7 = array ('token' => 'xxxxxx','method' => 'getOrders','parameters' => '{"status_id": 40196, "date_from": '.$lastDate.',"get_unconfirmed_orders": true}');
$query7 = http_build_query ($params7);
$contextData7 = array ('method' => 'POST','header' => "Content-Type: application/x-www-form-urlencoded\r\n"."Content-Length: ".strlen($query7)."\r\n"."User-Agent:MyAgent/1.0\r\n",'content'=> $query7 );
$context7 = stream_context_create (array ( 'http' => $contextData7 ));
$result7 =  file_get_contents ('https://api.baselinker.com/connector.php', false, $context7);
global $dane7;
$dane7 = json_decode($result7, true);
return $dane7;
}
[PHP] pobierz, plaintext 

Z tą zmienną $data próbowałem i tak i tak. Z $orders też.. W każdym przypadku nie wiem dlaczego z każdym wykonaniem pętli nie pobierają się kolejne zamówienia..

Ten post edytował xaitras 13.11.2017, 15:56:51

[nospor]

nadal masz
if (count($orders) < 100) {

z jednej strony prosisz o pomoc a jednoczesnie jak ci podadza poprawny kod to go ignorujesz

[xaitras]

nadal masz
if (count($orders) < 100) {

z jednej strony prosisz o pomoc a jednoczesnie jak ci podadza poprawny kod to go ignorujesz

no i wymodziłem

dziękuję za pomoc i cierpliwość
uczę się "w miarę potrzeby" i brakuje mi czasami podstawowej wiedzy

no i wymodziłem

dziękuję za pomoc i cierpliwość
uczę się "w miarę potrzeby" i brakuje mi czasami podstawowej wiedzy

Mam jeszcze drobny kłopot. Jeśli ilość zamówień nie przekracza sto to licznik liczy mi poprawnie a
jeśli przekroczy setkę to zlicza o 1 zamówienie więcej. Generalnie co setkę dolicza dodatkowe zamówienie.. nie wiadomo skąd

Kod:

[PHP] pobierz, plaintext 
include("funkcje.php");
$counter=0;
$zam = '0';
$lastDate ='0';
 
while (true) {
 
polacz_bl_przeniesione_spec($lastDate);
foreach($dane7['orders'] as $orders) {
$zam = $orders['order_id'];
$data = $orders['date_add'];
$counter++;
}
 
 
if (count($dane7['orders']) < 100) {
break;
}
 
$lastDate = $data;
 
}
echo $counter;
[PHP] pobierz, plaintext 

Ten post edytował xaitras 13.11.2017, 17:43:55