SearchResultCrawler::getAdsId() - Code Metrics - Inspection of "add a public method to get number of ads per page." - rfussien/leboncoin-crawler - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#9)

unknown

created 2016-11-13 17:43 UTC

SearchResultCrawler::getAdsId() A

↳ Parent: SearchResultCrawler

Complexity

Conditions	1
Paths	1

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	11
ccs	0
cts	0
cp	0
rs	9.4285
cc	1
eloc	6
nc	1
nop	0
crap	2

<?php namespace Lbc\Crawler;

class SearchResultCrawler extends CrawlerAbstract
{
    /**
     * Return the total number of ads of the search
     *
     * @return int
     */
    public function getNbAds()
    {
        $nbAds = $this->crawler
            ->filter('a.tabsSwitch span.tabsSwitchNumbers')
            ->first();

        if ($nbAds->count()) {
            $nbAds = preg_replace('/\s+/', '', $nbAds->text());
            return (int) $nbAds;
        }

        return 0;
    }

    /**
     * Return the number of ads per page.
     *
     * Could be dynamically guessed in future, if Leboncoin change it frequently
     * Or if they add the ability for user to change it on result pages.
     *
     *
     * @return int
     */
    public function getNbAdsPerPage()
    {
        return 35;
    }

    /**
     * Return the number of page
     *
     * @return int
     */
    public function getNbPages()
    {
        return (int) ceil($this->getNbAds() / $this->getNbAdsPerPage());
    }

    /**
     * Get an array containing the ads of the current result page
     *
     * @return Array
     */
    public function getAds()
    {
        $ads = array();

        $this->crawler->filter('[itemtype="http://schema.org/Offer"] > a')
            ->each(function ($node) use (&$ads) {
                $ad = (new SearchResultAdCrawler($node))->getAll();
                $ads [$ad->id] = $ad;
            });

        return $ads;
    }

    /**
     * Return the Ad's ID
     *
     * @return array
     */
    public function getAdsId()
    {
        $adsID = array();

        $this->crawler->filter('[itemtype="http://schema.org/Offer"] > a')
            ->each(function ($node) use (&$adsID) {
                $adsID [] = (new SearchResultAdCrawler($node))->getId();
            });

        return $adsID;
    }
}


1		<?php namespace Lbc\Crawler;
2
3		class SearchResultCrawler extends CrawlerAbstract
4		{
5		/**
6		* Return the total number of ads of the search
7		*
8		* @return int
9		*/
10	6	public function getNbAds()
11		{
12	6	$nbAds = $this->crawler
13	6	->filter('a.tabsSwitch span.tabsSwitchNumbers')
14	6	->first();
15
16	6	if ($nbAds->count()) {
17	6	$nbAds = preg_replace('/\s+/', '', $nbAds->text());
18	6	return (int) $nbAds;
19		}
20
21		return 0;
22		}
23
24		/**
25		* Return the number of ads per page.
26		*
27		* Could be dynamically guessed in future, if Leboncoin change it frequently
28		* Or if they add the ability for user to change it on result pages.
29	6	*
30		*
31	6	* @return int
32		*/
33		public function getNbAdsPerPage()
34		{
35		return 35;
36		}
37
38		/**
39	4	* Return the number of page
40		*
41	4	* @return int
42		*/
43	4	public function getNbPages()
44		{
45	4	return (int) ceil($this->getNbAds() / $this->getNbAdsPerPage());
46	4	}
47	4
48		/**
49	4	* Get an array containing the ads of the current result page
50		*
51		* @return Array
52		*/
53		public function getAds()
54		{
55		$ads = array();
56
57	4	$this->crawler->filter('[itemtype="http://schema.org/Offer"] > a')
58		->each(function ($node) use (&$ads) {
59	4	$ad = (new SearchResultAdCrawler($node))->getAll();
60		$ads [$ad->id] = $ad;
61	4	});
62	4
63	4	return $ads;
64	4	}
65
66	4	/**
67		* Return the Ad's ID
68		*
69		* @return array
70		*/
71		public function getAdsId()
72		{
73		$adsID = array();
74
75		$this->crawler->filter('[itemtype="http://schema.org/Offer"] > a')
76		->each(function ($node) use (&$adsID) {
77		$adsID [] = (new SearchResultAdCrawler($node))->getId();
78		});
79
80		return $adsID;
81		}
82		}
83

rfussien / leboncoin-crawler

Pull Request — master (#9)

SearchResultCrawler::getAdsId() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like