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..

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ą.

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 

Moze jednak podasz nam dokumentacje tego API?

Proszę uprzejmie

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

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 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 

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..

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 

Nic nie koliduje. Poprostu zamiast
date_confirmed_from
dajesz
date_from

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... ?

$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ę..

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ś

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..

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




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 

Pewnei zamiast zwracac ci zamowienie od daty wyzej to zwraca ci od daty wyzej lub rownej, przez co powtarza po kazdym kroku jedno zamowienie

dodałem $data++ i jest git dzięki jeszcze raz