Url - Code Metrics - Inspection of "refactor" - soyFelixBarros/Scraper - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 484738...127df7 )

by Felix

created 2018-02-23 18:52 UTC

Url A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

85%

Importance

Changes	6
Bugs	0	Features	1

Metric	Value
wmc	12
c	6
b	0
f	1
lcom	1
cbo	0
dl	0
loc	94
ccs	17
cts	20
cp	0.85
rs	10

6 Methods

Rating	Name	Size	Complexity
A	part()	4	2
A	has()	4	1
A	addHost()	4	1
A	addScheme()	4	1
A	decodeUnreserved()	4	1
B	normalize()	20	6

<?php

namespace Felix\Scraper;

class Url
{
    /** @var array */
    private static $parts;

    /**
     * Obtener una parte de la URL.
     *
     * @param $part string Parte de la URL (Ej: host).
     *
     * @return string|null
     */
    public static function part($part)
    {
        return array_key_exists($part, self::$parts) ? self::$parts[$part] : null;
    }

    /**
     * Url tiene una parte.
     *
     * @return bool true|false
     */
    public static function has($part)
    {
        return self::part($part) !== null;
    }

    /**
     * Agregar host a url.
     *
     * @param $url string Url
     * @param $host string Dominio
     *
     * @return string
     */
    public static function addHost($url, $host)
    {
        return rtrim($host, '/') . $url;
    }

    /**
     * Agregar http a url.
     *
     * @param $url string Url
     * @param $scheme string Esquema
     *
     * @return string $url
     */
    public static function addScheme($url, $scheme = 'http://')
    {
        return $scheme . ltrim($url, '/');
    }

    /**
     * Decode unreserved characters
     *
     * @param $url string Url.
     *
     * @return string
     */
    public static function decodeUnreserved($url)
    {
        return str_replace('–', '%E2%80%93', $url);
    }

    /**
     * Normalizar URL.
     *
     * @param $url string Url a normalizar.
     * @param $schemeAndHost string Esquema y dominio base (Ej. http://example.com)
     *
     * @return string
     */
    public static function normalize($url, $host = null)
    {
        self::$parts = parse_url($url);
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}

        if (self::$parts === false) {
            throw new \Exception($url.' es una URL mal formada y no se puede procesar');
        }

        if (! self::part('host') && $host !== null) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $url = self::addHost($url, $host);
        }

        if (! self::part('scheme') && self::part('host')) {
''   == false // true
''   == null  // true
'ab' == false // false
'ab' == null  // false

// It is often better to use strict comparison
'' === false // false
'' === null  // false
            $url = self::addScheme($url);
        }

        $url = self::decodeUnreserved($url);
        
        return $url;
    }
}


1		<?php
2
3		namespace Felix\Scraper;
4
5		class Url
6		{
7		/** @var array */
8		private static $parts;
9
10		/**
11		* Obtener una parte de la URL.
12		*
13		* @param $part string Parte de la URL (Ej: host).
14		*
15		* @return string\|null
16		*/
17	1	public static function part($part)
18		{
19	1	return array_key_exists($part, self::$parts) ? self::$parts[$part] : null;
20		}
21
22		/**
23		* Url tiene una parte.
24		*
25		* @return bool true\|false
26		*/
27		public static function has($part)
28		{
29		return self::part($part) !== null;
30		}
31
32		/**
33		* Agregar host a url.
34		*
35		* @param $url string Url
36		* @param $host string Dominio
37		*
38		* @return string
39		*/
40	1	public static function addHost($url, $host)
41		{
42	1	return rtrim($host, '/') . $url;
43		}
44
45		/**
46		* Agregar http a url.
47		*
48		* @param $url string Url
49		* @param $scheme string Esquema
50		*
51		* @return string $url
52		*/
53	1	public static function addScheme($url, $scheme = 'http://')
54		{
55	1	return $scheme . ltrim($url, '/');
56		}
57
58		/**
59		* Decode unreserved characters
60		*
61		* @param $url string Url.
62		*
63		* @return string
64		*/
65	1	public static function decodeUnreserved($url)
66		{
67	1	return str_replace('–', '%E2%80%93', $url);
68		}
69
70		/**
71		* Normalizar URL.
72		*
73		* @param $url string Url a normalizar.
74		* @param $schemeAndHost string Esquema y dominio base (Ej. http://example.com)
75		*
76		* @return string
77		*/
78	1	public static function normalize($url, $host = null)
79		{
80	1	self::$parts = parse_url($url);
		0 ignored issues – show Documentation Bug introduced 2018-02-23 18:42 UTC by Report Bug Copy Issue Report It seems like `parse_url($url)` can also be of type `false`. However, the property `$parts` is declared as type `array`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
81
82	1	if (self::$parts === false) {
83		throw new \Exception($url.' es una URL mal formada y no se puede procesar');
84		}
85
86	1	if (! self::part('host') && $host !== null) {
		0 ignored issues – show Bug Best Practice introduced 2018-02-23 18:42 UTC by Report Bug Copy Issue Report The expression `self::part('host')` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
87	1	$url = self::addHost($url, $host);
88		}
89
90	1	if (! self::part('scheme') && self::part('host')) {
		0 ignored issues – show Bug Best Practice introduced 2018-02-23 18:54 UTC by Report Bug Copy Issue Report The expression `self::part('scheme')` of type `string\|null` is loosely compared to `false`; this is ambiguous if the string can be empty. You might want to explicitly use `=== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history... Bug Best Practice introduced 2018-02-23 18:54 UTC by Report Bug Copy Issue Report The expression `self::part('host')` of type `string\|null` is loosely compared to `true`; this is ambiguous if the string can be empty. You might want to explicitly use `!== null` instead. In PHP, under loose comparison (like `==`, or `!=`, or `switch` conditions), values of different types might be equal. For `string` values, the empty string `''` is a special case, in particular the following results might be unexpected: '' == false // true '' == null // true 'ab' == false // false 'ab' == null // false // It is often better to use strict comparison '' === false // false '' === null // false Loading history...
91	1	$url = self::addScheme($url);
92		}
93
94	1	$url = self::decodeUnreserved($url);
95
96	1	return $url;
97		}
98		}
99

soyFelixBarros / Scraper

Push — master ( 484738...127df7 )

Url A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

6 Methods

Duplication Side-by-Side

Filter issues like