CRouteBasic::match() - Code Metrics - mosbth/anax - Measure and Improve Code Quality continuously with Scrutinizer

CRouteBasic::match() B
last analyzed 2018-10-19 17:34 UTC

↳ Parent: CRouteBasic

Complexity

Conditions	6
Paths	17

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	6

Importance

Changes

Metric	Value
cc	6
nc	17
nop	1
dl	0
loc	32
ccs	20
cts	20
cp	1
crap	6
rs	8.7857
c	0
b	0
f	0

<?php

namespace Anax\Route;

/**
 * A container for routes.
 *
 */
class CRouteBasic
{

    /**
    * Properties
    *
    */
    private $name;   // A name for this route
    private $rule;   // The rule for this route
    private $action; // The controller action to handle this route



    /**
     * Set values for route.
     *
     * @param string   $rule   for this route
     * @param callable $action callable to implement a controller for the route
     *
     * @return $this
     */
    public function set($rule, $action)
    {
        $this->rule = $rule;
        $this->action = $action;

        return $this;
    }



    /**
     * Check if the route matches a query
     *
     * @param string $query to match against
     *
     * @return boolean true if query matches the route
     */
    public function match($query)
    {
        $ruleParts  = explode('/', $this->rule);
        $queryParts = explode('/', $query);

        $ruleCount = count($ruleParts);
        //$querCount = count($queryParts);
        
        $match = false;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        for ($i = 0; $i < $ruleCount; $i++) {
            $rp = $ruleParts[$i];
            $qp = isset($queryParts[$i]) ? $queryParts[$i] : null;
            
            $first = isset($rp[0]) ? $rp[0] : '';
            switch ($first) {
                case '*':
                    $match = true;
                    break;
                
                default:
                    $match = ($rp == $qp);
                    break;
            }
            
            if (!$match) {
                return false;
            }
        }
        
        //echo "$this->rule == $query\n";
        return true;
    }



    /**
     * Handle the action for the route.
     *
     * @return void
     */
    public function handle()
    {
        return call_user_func($this->action);
    }



    /**
     * Set the name of the route.
     *
     * @param string $name set a name for the route
     *
     * @return $this
     */
    public function setName($name)
    {
        $this->name = $name;
        return $this;
    }



    /**
     * Get the rule for the route.
     *
     * @return string
     */
    public function getRule()
    {
        return $this->rule;
    }
}


1		<?php
2
3		namespace Anax\Route;
4
5		/**
6		* A container for routes.
7		*
8		*/
9		class CRouteBasic
10		{
11
12		/**
13		* Properties
14		*
15		*/
16		private $name; // A name for this route
17		private $rule; // The rule for this route
18		private $action; // The controller action to handle this route
19
20
21
22		/**
23		* Set values for route.
24		*
25		* @param string $rule for this route
26		* @param callable $action callable to implement a controller for the route
27		*
28		* @return $this
29		*/
30	1	public function set($rule, $action)
31		{
32	1	$this->rule = $rule;
33	1	$this->action = $action;
34
35	1	return $this;
36		}
37
38
39
40		/**
41		* Check if the route matches a query
42		*
43		* @param string $query to match against
44		*
45		* @return boolean true if query matches the route
46		*/
47	1	public function match($query)
48		{
49	1	$ruleParts = explode('/', $this->rule);
50	1	$queryParts = explode('/', $query);
51
52	1	$ruleCount = count($ruleParts);
53		//$querCount = count($queryParts);
54
55	1	$match = false;
		0 ignored issues – show Unused Code introduced 2015-12-03 20:17 UTC by Report Bug Copy Issue Report `$match` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
56	1	for ($i = 0; $i < $ruleCount; $i++) {
57	1	$rp = $ruleParts[$i];
58	1	$qp = isset($queryParts[$i]) ? $queryParts[$i] : null;
59
60	1	$first = isset($rp[0]) ? $rp[0] : '';
61		switch ($first) {
62	1	case '*':
63	1	$match = true;
64	1	break;
65
66	1	default:
67	1	$match = ($rp == $qp);
68	1	break;
69	1	}
70
71	1	if (!$match) {
72	1	return false;
73		}
74	1	}
75
76		//echo "$this->rule == $query\n";
77	1	return true;
78		}
79
80
81
82		/**
83		* Handle the action for the route.
84		*
85		* @return void
86		*/
87		public function handle()
88		{
89		return call_user_func($this->action);
90		}
91
92
93
94		/**
95		* Set the name of the route.
96		*
97		* @param string $name set a name for the route
98		*
99		* @return $this
100		*/
101		public function setName($name)
102		{
103		$this->name = $name;
104		return $this;
105		}
106
107
108
109		/**
110		* Get the rule for the route.
111		*
112		* @return string
113		*/
114		public function getRule()
115		{
116		return $this->rule;
117		}
118		}
119

mosbth / anax

CRouteBasic::match() B last analyzed 2018-10-19 17:34 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

CRouteBasic::match() B
last analyzed 2018-10-19 17:34 UTC