AbstractList::getResponses() - Code Metrics - Inspection of "Повторно опрашиваем ресурс, если он выдает ошибку." - sima-land/api-php-client - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — gavrilov (3735ac)

by Evgeny

created 2016-09-06 04:24 UTC

AbstractList::getResponses() D

↳ Parent: Project

Complexity

Conditions	10
Paths	40

Size

Total Lines	29
Code Lines	22

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
c	1
b	0
f	0
dl	0
loc	29
rs	4.8196
cc	10
eloc	22
nc	40
nop	0

How to fix Complexity

<?php

namespace SimaLand\API;

use GuzzleHttp\Psr7\Response;
use SimaLand\API\Rest\Client;
use SimaLand\API\Rest\Request;

/**
 * Абстрактный класс для загрузки данных сущности.
 *
 * Класс реализует интерфейс Iterator.
 */
abstract class AbstractList extends Object implements \Iterator
{
    /**
     * Кол-во потоков.
     *
     * @var int
     */
    public $countThreads = 5;

    /**
     * GET параметр отвечающий за поток.
     *
     * @var string
     */
    public $keyThreads = 'page';

    /**
     * Ключ альтернативной пагинации.
     *
     * @var string
     */
    public $keyAlternativePagination = 'id-greater-than';

    /**
     * GET параметры запроса.
     *
     * @var array
     */
    public $getParams = [];

    /**
     * Кол-во повторов обращение к ресурсу при ошибках.
     *
     * @var int
     */
    public $repeatTimeout = 30;

    /**
     * Время в секундак до следующего обращения к ресурсу.
     *
     * @var int
     */
    public $repeatCount = 30;


    /**
     * SimaLand кдиент для запросов.
     *
     * @var \SimaLand\API\Rest\Client
     */
    private $client;

    /**
     * Список запросов.
     *
     * @var Request[]
     */
    private $requests = [];

    /**
     * Список данных полученные по API.
     *
     * @var array
     */
    private $values = [];

    /**
     * Ключ текущей записи.
     *
     * @var int
     */
    private $key;

    /**
     * Текущая запись.
     *
     * @var mixed
     */
    private $current;

    /**
     * @param Client $client
     * @param array $options
     */
    public function __construct(Client $client, array $options = [])
    {
        $this->client = $client;
        parent::__construct($options);
    }

    /**
     * Получить наименование сущности.
     *
     * @return string
     */
    abstract public function getEntity();

    /**
     * Добавить get параметры.
     *
     * @param array $params
     * @return AbstractList
     */
    public function addGetParams(array $params)
    {
        $this->getParams = array_merge($this->getParams, $params);
        return $this;
    }

    /**
     * Назначить следующию страницу запросу.
     *
     * @param Request $request
     * @param Record|null $record
     */
    public function assignPage(Request &$request, Record $record = null)
    {
        $currentPage = 1;
        if (!is_array($request->getParams)) {
            $request->getParams = (array)$request->getParams;
        }
        if (isset($request->getParams[$this->keyThreads])) {
            $currentPage = (int)$request->getParams[$this->keyThreads];
        }
        $request->getParams[$this->keyThreads] = $currentPage + $this->countThreads;
    }

    /**
     * Назначить номер потока для запроса.
     *
     * @param Request $request
     * @param int $number
     */
    public function assignThreadsNumber(Request &$request, $number = 0)
    {
        if (!is_array($request->getParams)) {
            $request->getParams = (array)$request->getParams;
        }
        if (!isset($request->getParams[$this->keyThreads])) {
            $request->getParams[$this->keyThreads] = 1;
        }
        $request->getParams[$this->keyThreads] += $number;
    }

    /**
     * Наименование ключа содержащего набора данных сущности.
     *
     * @return string
     */
    public function getCollectionKey()
    {
        return 'items';
    }

    /**
     * Наименование ключа содержащего мета данные.
     *
     * @return string
     */
    public function getMetaKey()
    {
        return '_meta';
    }

    /**
     * Палучить набор данных сущности.
     *
     * @return Response[]
     * @throws \Exception
     */
    public function get()
    {
        return $this->client->batchQuery($this->getRequests());
    }

    /**
     * Установить запросы к API.
     *
     * @param Request[] $requests
     * @throws \Exception
     */
    public function setRequests(array $requests)
    {
        $this->requests = [];
        foreach ($requests as $request) {
            if (!$request instanceof Request) {
                throw new \Exception('Request must be implement "\SimaLand\API\Rest\Request"');
            }
            $this->requests[] = $request;
        }
    }

    /**
     * Получить запросы к API.
     *
     * @return array|Rest\Request[]
     */
    public function getRequests()
    {
        if (empty($this->requests)) {
            $requests = [];
            if (!is_null($this->keyThreads) && $this->countThreads > 1) {
                for ($i = 0; $i < $this->countThreads; $i++) {
                    $requests[$i] = new Request([
                        'entity' => $this->getEntity(),
                        'getParams' => $this->getParams,
                    ]);
                    $this->assignThreadsNumber($requests[$i], $i);
                }
            } else {
                $requests[] = new Request([
                    'entity' => $this->getEntity(),
                    'getParams' => $this->getParams,
                ]);
            }
            $this->requests = $requests;
        }
        return $this->requests;
    }

    /**
     * @inheritdoc
     */
    public function current()
    {
        return $this->current;
    }

    /**
     * @inheritdoc
     */
    public function next()
    {
        if (empty($this->values)) {
            $this->getData();
        }
        $this->current = array_shift($this->values);
    }

    /**
     * @inheritdoc
     */
    public function key()
    {
        return $this->key++;
    }

    /**
     * @inheritdoc
     */
    public function valid()
    {
        return !empty($this->current);
    }

    /**
     * @inheritdoc
     */
    public function rewind()
    {
        $this->values = [];
        $this->current = null;
        $this->key = 0;
        $this->next();
    }

    /**
     * Получение и обработка ответов от API
     *
     * @return \GuzzleHttp\Psr7\Response[]
     * @throws \Exception
     */
    private function getResponses()
    {
        $i = 0;
        do {
            if ($i > 0) {
                sleep($this->repeatTimeout);
            }
            try {
                $responses = $this->get();
                foreach ($responses as $response) {
                    $statusCode = $response->getStatusCode();
                    if (($statusCode < 200 || $statusCode >= 300) && $statusCode != 404) {
                        $body = json_decode($response->getBody(), true);
                        $message = isset($body['message']) ? $body['message'] : $response->getReasonPhrase();
                        throw new \Exception($message, $statusCode);
                    }
                }
                $e = null;
                break;
            } catch (\Exception $e) {
                $e = new \Exception($e->getMessage(), $e->getCode(), $e);
            }
            $i++;
        } while ($i <= $this->repeatCount);
        if ($e) {
            throw new \Exception($e->getMessage(), $e->getCode(), $e);
        }
        return $responses;
    }

    /**
     * Получить тело ответа от API.
     *
     * @param Response $response
     * @return bool
     * @throws \Exception
     */
    private function getBody(Response $response)
    {
        $body = json_decode($response->getBody(), true);
        if (
            !$body ||
            ($body && !isset($body[$this->getCollectionKey()]))
        ) {
            return false;
        }
        return $body;
    }

    /**
     * Получить набор данных от API.
     *
     * @throws \Exception
     */
    private function getData()
    {
        $responses = $this->getResponses();
        $collectionKey = $this->getCollectionKey();
        $metaKey = $this->getMetaKey();
        $requests = $this->getRequests();
        $record = null;
        foreach ($responses as $key => $response) {
            $body = $this->getBody($response);
            if (!$body) {
                unset($requests[$key]);
                continue;
            }
            foreach ($body[$collectionKey] as $item) {
                $record = new Record([
                    'data' => $item,
                    'meta' => isset($body[$metaKey]) ? $body[$metaKey] : null,
                ]);
                $this->values[] = $record;
            }
            if (!is_null($record)) {
                $this->assignPage($requests[$key], $record);
            }
        }
        $this->setRequests($requests);
    }
}


GitHub Access Token became invalid

Branch — gavrilov (3735ac)

AbstractList::getResponses() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

1			<?php
2
3			namespace SimaLand\API;
4
5			use GuzzleHttp\Psr7\Response;
6			use SimaLand\API\Rest\Client;
7			use SimaLand\API\Rest\Request;
8
9			/**
10			* Абстрактный класс для загрузки данных сущности.
11			*
12			* Класс реализует интерфейс Iterator.
13			*/
14			abstract class AbstractList extends Object implements \Iterator
15			{
16			/**
17			* Кол-во потоков.
18			*
19			* @var int
20			*/
21			public $countThreads = 5;
22
23			/**
24			* GET параметр отвечающий за поток.
25			*
26			* @var string
27			*/
28			public $keyThreads = 'page';
29
30			/**
31			* Ключ альтернативной пагинации.
32			*
33			* @var string
34			*/
35			public $keyAlternativePagination = 'id-greater-than';
36
37			/**
38			* GET параметры запроса.
39			*
40			* @var array
41			*/
42			public $getParams = [];
43
44			/**
45			* Кол-во повторов обращение к ресурсу при ошибках.
46			*
47			* @var int
48			*/
49			public $repeatTimeout = 30;
50
51			/**
52			* Время в секундак до следующего обращения к ресурсу.
53			*
54			* @var int
55			*/
56			public $repeatCount = 30;
57
58
59			/**
60			* SimaLand кдиент для запросов.
61			*
62			* @var \SimaLand\API\Rest\Client
63			*/
64			private $client;
65
66			/**
67			* Список запросов.
68			*
69			* @var Request[]
70			*/
71			private $requests = [];
72
73			/**
74			* Список данных полученные по API.
75			*
76			* @var array
77			*/
78			private $values = [];
79
80			/**
81			* Ключ текущей записи.
82			*
83			* @var int
84			*/
85			private $key;
86
87			/**
88			* Текущая запись.
89			*
90			* @var mixed
91			*/
92			private $current;
93
94			/**
95			* @param Client $client
96			* @param array $options
97			*/
98			public function __construct(Client $client, array $options = [])
99			{
100			$this->client = $client;
101			parent::__construct($options);
102			}
103
104			/**
105			* Получить наименование сущности.
106			*
107			* @return string
108			*/
109			abstract public function getEntity();
110
111			/**
112			* Добавить get параметры.
113			*
114			* @param array $params
115			* @return AbstractList
116			*/
117			public function addGetParams(array $params)
118			{
119			$this->getParams = array_merge($this->getParams, $params);
120			return $this;
121			}
122
123			/**
124			* Назначить следующию страницу запросу.
125			*
126			* @param Request $request
127			* @param Record\|null $record
128			*/
129			public function assignPage(Request &$request, Record $record = null)
130			{
131			$currentPage = 1;
132			if (!is_array($request->getParams)) {
133			$request->getParams = (array)$request->getParams;
134			}
135			if (isset($request->getParams[$this->keyThreads])) {
136			$currentPage = (int)$request->getParams[$this->keyThreads];
137			}
138			$request->getParams[$this->keyThreads] = $currentPage + $this->countThreads;
139			}
140
141			/**
142			* Назначить номер потока для запроса.
143			*
144			* @param Request $request
145			* @param int $number
146			*/
147			public function assignThreadsNumber(Request &$request, $number = 0)
148			{
149			if (!is_array($request->getParams)) {
150			$request->getParams = (array)$request->getParams;
151			}
152			if (!isset($request->getParams[$this->keyThreads])) {
153			$request->getParams[$this->keyThreads] = 1;
154			}
155			$request->getParams[$this->keyThreads] += $number;
156			}
157
158			/**
159			* Наименование ключа содержащего набора данных сущности.
160			*
161			* @return string
162			*/
163			public function getCollectionKey()
164			{
165			return 'items';
166			}
167
168			/**
169			* Наименование ключа содержащего мета данные.
170			*
171			* @return string
172			*/
173			public function getMetaKey()
174			{
175			return '_meta';
176			}
177
178			/**
179			* Палучить набор данных сущности.
180			*
181			* @return Response[]
182			* @throws \Exception
183			*/
184			public function get()
185			{
186			return $this->client->batchQuery($this->getRequests());
187			}
188
189			/**
190			* Установить запросы к API.
191			*
192			* @param Request[] $requests
193			* @throws \Exception
194			*/
195			public function setRequests(array $requests)
196			{
197			$this->requests = [];
198			foreach ($requests as $request) {
199			if (!$request instanceof Request) {
200			throw new \Exception('Request must be implement "\SimaLand\API\Rest\Request"');
201			}
202			$this->requests[] = $request;
203			}
204			}
205
206			/**
207			* Получить запросы к API.
208			*
209			* @return array\|Rest\Request[]
210			*/
211			public function getRequests()
212			{
213			if (empty($this->requests)) {
214			$requests = [];
215			if (!is_null($this->keyThreads) && $this->countThreads > 1) {
216			for ($i = 0; $i < $this->countThreads; $i++) {
217			$requests[$i] = new Request([
218			'entity' => $this->getEntity(),
219			'getParams' => $this->getParams,
220			]);
221			$this->assignThreadsNumber($requests[$i], $i);
222			}
223			} else {
224			$requests[] = new Request([
225			'entity' => $this->getEntity(),
226			'getParams' => $this->getParams,
227			]);
228			}
229			$this->requests = $requests;
230			}
231			return $this->requests;
232			}
233
234			/**
235			* @inheritdoc
236			*/
237			public function current()
238			{
239			return $this->current;
240			}
241
242			/**
243			* @inheritdoc
244			*/
245			public function next()
246			{
247			if (empty($this->values)) {
248			$this->getData();
249			}
250			$this->current = array_shift($this->values);
251			}
252
253			/**
254			* @inheritdoc
255			*/
256			public function key()
257			{
258			return $this->key++;
259			}
260
261			/**
262			* @inheritdoc
263			*/
264			public function valid()
265			{
266			return !empty($this->current);
267			}
268
269			/**
270			* @inheritdoc
271			*/
272			public function rewind()
273			{
274			$this->values = [];
275			$this->current = null;
276			$this->key = 0;
277			$this->next();
278			}
279
280			/**
281			* Получение и обработка ответов от API
282			*
283			* @return \GuzzleHttp\Psr7\Response[]
284			* @throws \Exception
285			*/
286			private function getResponses()
287			{
288			$i = 0;
289			do {
290			if ($i > 0) {
291			sleep($this->repeatTimeout);
292			}
293			try {
294			$responses = $this->get();
295			foreach ($responses as $response) {
296			$statusCode = $response->getStatusCode();
297			if (($statusCode < 200 \|\| $statusCode >= 300) && $statusCode != 404) {
298			$body = json_decode($response->getBody(), true);
299			$message = isset($body['message']) ? $body['message'] : $response->getReasonPhrase();
300			throw new \Exception($message, $statusCode);
301			}
302			}
303			$e = null;
304			break;
305			} catch (\Exception $e) {
306			$e = new \Exception($e->getMessage(), $e->getCode(), $e);
307			}
308			$i++;
309			} while ($i <= $this->repeatCount);
310			if ($e) {
311			throw new \Exception($e->getMessage(), $e->getCode(), $e);
312			}
313			return $responses;
314			}
315
316			/**
317			* Получить тело ответа от API.
318			*
319			* @param Response $response
320			* @return bool
321			* @throws \Exception
322			*/
323			private function getBody(Response $response)
324			{
325			$body = json_decode($response->getBody(), true);
326			if (
327			!$body \|\|
328			($body && !isset($body[$this->getCollectionKey()]))
329			) {
330			return false;
331			}
332			return $body;
333			}
334
335			/**
336			* Получить набор данных от API.
337			*
338			* @throws \Exception
339			*/
340			private function getData()
341			{
342			$responses = $this->getResponses();
343			$collectionKey = $this->getCollectionKey();
344			$metaKey = $this->getMetaKey();
345			$requests = $this->getRequests();
346			$record = null;
347			foreach ($responses as $key => $response) {
348			$body = $this->getBody($response);
349			if (!$body) {
350			unset($requests[$key]);
351			continue;
352			}
353			foreach ($body[$collectionKey] as $item) {
354			$record = new Record([
355			'data' => $item,
356			'meta' => isset($body[$metaKey]) ? $body[$metaKey] : null,
357			]);
358			$this->values[] = $record;
359			}
360			if (!is_null($record)) {
361			$this->assignPage($requests[$key], $record);
362			}
363			}
364			$this->setRequests($requests);
365			}
366			}
367

sima-land / api-php-client

GitHub Access Token became invalid

Branch — gavrilov (3735ac)

AbstractList::getResponses() D

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like