ArrayRequestLogger::getRequestsSince() - Code Metrics - Inspection of "Mark package as abandoned" - treehouselabs/io-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( dbcf6b...d53ab6 )

by Peter

created 2018-12-12 10:42 UTC

ArrayRequestLogger::getRequestsSince() A

↳ Parent: ArrayRequestLogger

Complexity

Conditions	5
Paths	8

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	5

Importance

Changes

Metric	Value
dl	0
loc	15
ccs	8
cts	8
cp	1
rs	9.4555
c	0
b	0
f	0
cc	5
nc	8
nop	1
crap	5

<?php

namespace TreeHouse\IoBundle\Scrape\Crawler\Log;

/**
 * In-memory request logger. Use for very basic or testing purposes.
 *
 * CAUTION: it's not a good idea to use this when crawling large amounts of pages,
 * since memory usage keeps increasing, as well as the time consumed in this class.
 * Also this class does not work when working with multiple processes.
 *
 * You should configure one of the other loggers.
 */
class ArrayRequestLogger implements RequestLoggerInterface
{
    /**
     * @var array<integer, string[]>
     */
    protected $requests = [];

    /**
     * @inheritdoc
     */
    public function logRequest($url, \DateTime $date = null)
    {
        if (null === $date) {
            $date = new \DateTime();
        }

        $hashKey = $date->getTimestamp();
        $this->requests[$hashKey][] = $url;
    }

    /**
     * @inheritdoc
     */
    public function getRequestsSince(\DateTime $date = null)
    {
        $start = $date ? $date->getTimestamp() : 0;

        $requests = [];
        foreach ($this->requests as $time => $reqs) {
            if ($time >= $start) {
                foreach ($reqs as $req) {
                    array_unshift($requests, [$time, $req]);
                }
            }
        }

        return $requests;
    }
}


1		<?php
2
3		namespace TreeHouse\IoBundle\Scrape\Crawler\Log;
4
5		/**
6		* In-memory request logger. Use for very basic or testing purposes.
7		*
8		* CAUTION: it's not a good idea to use this when crawling large amounts of pages,
9		* since memory usage keeps increasing, as well as the time consumed in this class.
10		* Also this class does not work when working with multiple processes.
11		*
12		* You should configure one of the other loggers.
13		*/
14		class ArrayRequestLogger implements RequestLoggerInterface
15		{
16		/**
17		* @var array<integer, string[]>
18		*/
19		protected $requests = [];
20
21		/**
22		* @inheritdoc
23		*/
24	6	public function logRequest($url, \DateTime $date = null)
25		{
26	6	if (null === $date) {
27		$date = new \DateTime();
28		}
29
30	6	$hashKey = $date->getTimestamp();
31	6	$this->requests[$hashKey][] = $url;
32	6	}
33
34		/**
35		* @inheritdoc
36		*/
37	6	public function getRequestsSince(\DateTime $date = null)
38		{
39	6	$start = $date ? $date->getTimestamp() : 0;
40
41	6	$requests = [];
42	6	foreach ($this->requests as $time => $reqs) {
43	6	if ($time >= $start) {
44	4	foreach ($reqs as $req) {
45	6	array_unshift($requests, [$time, $req]);
46		}
47		}
48		}
49
50	6	return $requests;
51		}
52		}
53

treehouselabs / io-bundle

Push — master ( dbcf6b...d53ab6 )

ArrayRequestLogger::getRequestsSince() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like