Router::get_router_name() - Code Metrics - Inspection of "New sitemaps work! Removes old site sitemap" - jacobemerick/web - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( dfa8f4...cde30c )

by Jacob

created 2016-01-01 00:04 UTC

Router::get_router_name() C

↳ Parent: Router

Complexity

Conditions	11
Paths	11

Size

Total Lines	37
Code Lines	31

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	37
rs	5.2653
cc	11
eloc	31
nc	11
nop	0

How to fix Complexity

<?


Loader::load('utility', array(
	'Request',
	'URLDecode'));

abstract class Router
{

	public function __construct() {}

	public static function instance()
	{
		$router_name = self::get_router_name();
		$router = Loader::loadNew('router', $router_name);
		$router->route();
	}

	private static function get_router_name()
	{
		if(Request::isAJAX())
			return 'AJAXRouter';
		if(URLDecode::getURI() == '/sitemap.xml')
			return 'SitemapRouter';
		if(URLDecode::getURI() == '/rss/')
			return 'RSSRouter';
		
		switch(URLDecode::getSite())
		{
			case 'ajax' :
				return 'AjaxRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'blog' :
				return 'BlogRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'home' :
				return 'HomeRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'lifestream' :
				return 'LifestreamRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'portfolio' :
				return 'PortfolioRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'site' :
				return 'SiteRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
			case 'waterfalls' :
				return 'WaterfallRouter';
			break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
		}
		
		Debugger::logMessage("The router for " . URLDecode::getSite() . " was not loaded.");
		Loader::loadNew('controller', '/Error404Controller')->activate();
	}

	protected function route()
	{
		$uri = URLDecode::getURI();
		
		$this->check_for_redirect($uri);
		
		$controller = $this->get_controller($uri);
		Loader::loadNew('controller', $controller)
			->activate();
	}

	abstract protected function get_redirect_array();
	abstract protected function get_direct_array();

	final protected function check_for_redirect($redirect_uri)
	{
		foreach($this->get_redirect_array() as $check)
		{
			$redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
		}
		
		$redirect_uri = $this->check_for_special_redirect($redirect_uri);
		
		if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
			$redirect_uri .= '/';
		
        if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
            $redirect_uri = 'http://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
        }
        
		if($redirect_uri == URLDecode::getURI())
			return;
		
		$controller_check = $redirect_uri;
		if(substr($redirect_uri, 0, 4) == 'http')
			$controller_check = preg_replace('@^http://([a-z\.]+)@', '', $redirect_uri);
		
		$controller = $this->get_controller($controller_check);
		if($controller == '/Error404Controller')
		{
			Loader::loadNew('controller', '/Error404Controller')
				->activate();
			exit;

		}

		if(substr($redirect_uri, 0, 4) != 'http')
		{
			$redirect_uri = substr($redirect_uri, 1);
			$redirect_uri = URLDecode::getBase() . $redirect_uri;
		}
		
		Loader::loadNew('controller', '/Error301Controller', (array) $redirect_uri)
			->activate();
	}

	protected function check_for_special_redirect($uri)
	{
		return $uri;
	}

	final private function get_controller($uri)
	{
		foreach($this->get_direct_array() as $check)
		{
			if($uri == $check->match)
				return "{$this->get_primary_folder()}/{$check->controller}";
			
			if(preg_match("@^{$check->match}$@", $uri))
				return "{$this->get_primary_folder()}/{$check->controller}";
		}
		
		return '/Error404Controller';
	}

	final private function get_primary_folder()
	{
		if(Request::isAjax())
			return 'ajax';
		if(URLDecode::getURI() == '/sitemap.xml')
			return 'sitemap';
		if(URLDecode::getURI() == '/rss/')
			return 'rss';
		
		return URLDecode::getSite();
	}

	private function requires_trailing_slash()
	{
		return (
			URLDecode::getURI() != '/sitemap.xml' &&
			URLDecode::getExtension() != 'json' &&
            strstr(URLDecode::getURI(), '#') === false);
	}

}


1			<?
			0 ignored issues – show Security Best Practice introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report It is not recommend to use PHP's short opening tag `<?`, better use `<?php`, or `<?=` in case of outputting. Short opening tags are disabled in PHP’s default configuration. In such a case, all content of this file is output verbatim to the browser without being parsed, or executed. As a precaution to avoid these problems better use the long opening tag `<?php`. Loading history...
2
3			Loader::load('utility', array(
4			'Request',
5			'URLDecode'));
6
7			abstract class Router
8			{
9
10			public function __construct() {}
11
12			public static function instance()
13			{
14			$router_name = self::get_router_name();
15			$router = Loader::loadNew('router', $router_name);
16			$router->route();
17			}
18
19			private static function get_router_name()
20			{
21			if(Request::isAJAX())
22			return 'AJAXRouter';
23			if(URLDecode::getURI() == '/sitemap.xml')
24			return 'SitemapRouter';
25			if(URLDecode::getURI() == '/rss/')
26			return 'RSSRouter';
27
28			switch(URLDecode::getSite())
29			{
30			case 'ajax' :
31			return 'AjaxRouter';
32			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
33			case 'blog' :
34			return 'BlogRouter';
35			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
36			case 'home' :
37			return 'HomeRouter';
38			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
39			case 'lifestream' :
40			return 'LifestreamRouter';
41			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
42			case 'portfolio' :
43			return 'PortfolioRouter';
44			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
45			case 'site' :
46			return 'SiteRouter';
47			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
48			case 'waterfalls' :
49			return 'WaterfallRouter';
50			break;
			0 ignored issues – show Unused Code introduced 2015-11-28 18:19 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
51			}
52
53			Debugger::logMessage("The router for " . URLDecode::getSite() . " was not loaded.");
54			Loader::loadNew('controller', '/Error404Controller')->activate();
55			}
56
57			protected function route()
58			{
59			$uri = URLDecode::getURI();
60
61			$this->check_for_redirect($uri);
62
63			$controller = $this->get_controller($uri);
64			Loader::loadNew('controller', $controller)
65			->activate();
66			}
67
68			abstract protected function get_redirect_array();
69			abstract protected function get_direct_array();
70
71			final protected function check_for_redirect($redirect_uri)
72			{
73			foreach($this->get_redirect_array() as $check)
74			{
75			$redirect_uri = preg_replace($check->pattern, $check->replace, $redirect_uri);
76			}
77
78			$redirect_uri = $this->check_for_special_redirect($redirect_uri);
79
80			if($this->requires_trailing_slash() && substr($redirect_uri, -1) != '/')
81			$redirect_uri .= '/';
82
83			if (URLDecode::getHost() == 'waterfalls.jacobemerick.com') {
84			$redirect_uri = 'http://' . (!Loader::isLive() ? 'dev' : 'www') . '.waterfallsofthekeweenaw.com' . $redirect_uri;
85			}
86
87			if($redirect_uri == URLDecode::getURI())
88			return;
89
90			$controller_check = $redirect_uri;
91			if(substr($redirect_uri, 0, 4) == 'http')
92			$controller_check = preg_replace('@^http://([a-z\.]+)@', '', $redirect_uri);
93
94			$controller = $this->get_controller($controller_check);
95			if($controller == '/Error404Controller')
96			{
97			Loader::loadNew('controller', '/Error404Controller')
98			->activate();
99			exit;
			0 ignored issues – show Coding Style Compatibility introduced 2015-11-28 18:20 UTC by Report Bug Copy Issue Report The method `check_for_redirect()` contains an exit expression. An exit expression should only be used in rare cases. For example, if you write a short command line script. In most cases however, using an `exit` expression makes the code untestable and often causes incompatibilities with other libraries. Thus, unless you are absolutely sure it is required here, we recommend to refactor your code to avoid its usage. Loading history...
100			}
101
102			if(substr($redirect_uri, 0, 4) != 'http')
103			{
104			$redirect_uri = substr($redirect_uri, 1);
105			$redirect_uri = URLDecode::getBase() . $redirect_uri;
106			}
107
108			Loader::loadNew('controller', '/Error301Controller', (array) $redirect_uri)
109			->activate();
110			}
111
112			protected function check_for_special_redirect($uri)
113			{
114			return $uri;
115			}
116
117			final private function get_controller($uri)
118			{
119			foreach($this->get_direct_array() as $check)
120			{
121			if($uri == $check->match)
122			return "{$this->get_primary_folder()}/{$check->controller}";
123
124			if(preg_match("@^{$check->match}$@", $uri))
125			return "{$this->get_primary_folder()}/{$check->controller}";
126			}
127
128			return '/Error404Controller';
129			}
130
131			final private function get_primary_folder()
132			{
133			if(Request::isAjax())
134			return 'ajax';
135			if(URLDecode::getURI() == '/sitemap.xml')
136			return 'sitemap';
137			if(URLDecode::getURI() == '/rss/')
138			return 'rss';
139
140			return URLDecode::getSite();
141			}
142
143			private function requires_trailing_slash()
144			{
145			return (
146			URLDecode::getURI() != '/sitemap.xml' &&
147			URLDecode::getExtension() != 'json' &&
148			strstr(URLDecode::getURI(), '#') === false);
149			}
150
151			}
152

jacobemerick / web

Push — master ( dfa8f4...cde30c )

Router::get_router_name() C

Complexity

Size

Duplication

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like