ConceptSearchParameters - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/NatLib..." - NatLibFi/Skosmos - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( ea4aee...b9ee15 )

by Henri

created 2016-01-11 07:11 UTC

ConceptSearchParameters B

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	157
Duplicated Lines	8.92 %

Coupling/Cohesion

Components	2
Dependencies	0

Metric	Value
wmc	54
lcom	2
cbo	0
dl	14
loc	157
rs	7.0642

20 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
A	getLang()	0	7	3
A	getVocabs()	0	10	3
B	getVocabIds()	8	16	8
A	setVocabularies()	0	4	1
A	getArrayClass()	0	8	2
B	getSearchTerm()	0	8	5
A	getContentLang()	0	4	1
A	getSearchLang()	0	7	3
B	getTypeLimit()	6	20	11
A	getGroupLimit()	0	4	2
A	getParentLimit()	0	4	2
A	getOffset()	0	4	4
A	getSearchLimit()	0	4	1
A	getUnique()	0	3	1
A	setUnique()	0	3	1
A	getAdditionalFields()	0	3	2
A	getHidden()	0	3	1
A	setHidden()	0	3	1
A	setAdditionalFields()	0	3	1

How to fix Duplicated Code Complexity

Duplicated Code

Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.

Common duplication problems, and corresponding solutions are:

If you have the same expression in different places: Extract expression to a method
If you have the same method in different sub-classes: Extract method, and pull up field to the parent class
If you have the same code in unrelated classes: Consider extracting the code to a new class, and injecting that class

Complex Class

Tip: Before tackling complexity, make sure that you eliminate any duplication first. This often can reduce the size of classes significantly.

Complex classes like ConceptSearchParameters often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.

Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.

While breaking up the class, it is a good idea to analyze how other classes use ConceptSearchParameters, and based on these observations, apply Extract Interface, too.

<?php

/**
 * ConceptSearchParameters is used for search parameters.
 */
class ConceptSearchParameters
namespace YourVendor;

class YourClass { }
{

    private $config;
    private $request;
    private $vocabs;
    private $rest;
    private $hidden;
    private $unique;

    public function __construct($request, $config, $rest = false) 
    {
        $this->request = $request;
        $this->config = $config;
        $this->rest = $rest;
        $this->hidden = true;
        $this->unique = $request->getQueryParamBoolean('unique', false);
    }

    public function getLang() 
    {
        if ($this->rest && $this->request->getQueryParam('labellang')) {
            return $this->request->getQueryParam('labellang');
        }
        return $this->request->getLang();
    } 

    public function getVocabs() 
    {
        if ($this->vocabs) {
            return $this->vocabs;
        }
        if ($this->request->getVocab()) {
            return array($this->request->getVocab());
        }
        return array();
    } 

    public function getVocabIds()
    {
        if ($this->rest) {

            $vocabs = $this->request->getQueryParam('vocab');
            return ($vocabs !== null && $vocabs !== '') ? explode(' ', $vocabs) : null;
        }
        if ($this->request->getQueryParam('vocabs')) {

            $vocabs = $this->request->getQueryParam('vocabs'); 
            return ($vocabs !== null && $vocabs !== '') ? explode(' ', $vocabs) : null;
        }
        $vocabs = $this->getVocabs();
        if ($vocabs[0]) {
            return array($vocabs[0]->getId());
        }
        return null;
    }

    public function setVocabularies($vocabs) 
    {
        $this->vocabs = $vocabs;
    }
    
    public function getArrayClass() 
    {
        if (sizeof($this->getVocabs()) == 1) { // search within vocabulary
            $vocabs = $this->getVocabs();
            return $vocabs[0]->getConfig()->getArrayClassURI();
        }
        return null;
    }
    
    public function getSearchTerm() 
    {
        $term = $this->request->getQueryParam('q') ? $this->request->getQueryParam('q') : $this->request->getQueryParam('query');
        if (!$term && $this->rest)
            $term = $this->request->getQueryParam('label');
        $term = trim($term); // surrounding whitespace is not considered significant
        return strpos($term, "*") === false ? $term . "*" : $term; // default to prefix search
    }
    
    public function getContentLang() 
    {
        return $this->request->getContentLang();
    }
    
    public function getSearchLang() 
    {
        if ($this->rest) {
            return $this->request->getQueryParam('lang');
        }
        return $this->request->getQueryParam('anylang') ? '' : $this->getContentLang();
    }

    public function getTypeLimit() 
    {
        $type = $this->request->getQueryParam('type') !== '' ? $this->request->getQueryParam('type') : null;
        if ($type && strpos($type, '+')) {
            $type = explode('+', $type);
        } else if ($type && !is_array($type)) {
            // if only one type param given place it into an array regardless
            $type = array($type);
        }
        if ($type === null) {
            $type = array('skos:Concept');
            if ($this->request->getVocab() && $this->request->getVocab()->getConfig()->getArrayClassURI()) {

                array_push($type, $this->request->getVocab()->getConfig()->getArrayClassURI());
            }
            if ($this->request->getVocab() && $this->request->getVocab()->getConfig()->getGroupClassURI()) {

                array_push($type, $this->request->getVocab()->getConfig()->getGroupClassURI());
            }
        }
        return $type;
    }

    public function getGroupLimit() 
    {
        return $this->request->getQueryParam('group') !== '' ? $this->request->getQueryParam('group') : null;
    }
    
    public function getParentLimit() 
    {
        return $this->request->getQueryParam('parent') !== '' ? $this->request->getQueryParam('parent') : null;
    }

    public function getOffset() 
    {
        return ($this->request->getQueryParam('offset') && is_numeric($this->request->getQueryParam('offset')) && $this->request->getQueryParam('offset') >= 0) ? $this->request->getQueryParam('offset') : 0;
    }

    public function getSearchLimit()
    {
        return $this->config->getDefaultSearchLimit();
    }

    public function getUnique() {
        return $this->unique;
    }

    public function setUnique($unique) {
        $this->unique = $unique;
    }

    public function getAdditionalFields() {
        return $this->request->getQueryParam('fields') ? explode(' ', $request->getQueryParam('fields')) : null;

    }
    
    public function getHidden() {
        return $this->hidden;
    }
    
    public function setHidden($hidden) {
        $this->hidden = $hidden;
    }
    
    public function setAdditionalFields($fields) {
        $this->fields = $fields;
class MyClass { }

$x = new MyClass();
$x->foo = true;
    }

}


1		<?php
2
3		/**
4		* ConceptSearchParameters is used for search parameters.
5		*/
6		class ConceptSearchParameters
		0 ignored issues – show Coding Style Compatibility introduced 2016-01-05 13:31 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
7		{
8
9		private $config;
10		private $request;
11		private $vocabs;
12		private $rest;
13		private $hidden;
14		private $unique;
15
16		public function __construct($request, $config, $rest = false)
17		{
18		$this->request = $request;
19		$this->config = $config;
20		$this->rest = $rest;
21		$this->hidden = true;
22		$this->unique = $request->getQueryParamBoolean('unique', false);
23		}
24
25		public function getLang()
26		{
27		if ($this->rest && $this->request->getQueryParam('labellang')) {
28		return $this->request->getQueryParam('labellang');
29		}
30		return $this->request->getLang();
31		}
32
33		public function getVocabs()
34		{
35		if ($this->vocabs) {
36		return $this->vocabs;
37		}
38		if ($this->request->getVocab()) {
39		return array($this->request->getVocab());
40		}
41		return array();
42		}
43
44		public function getVocabIds()
45		{
46	View Code Duplication	if ($this->rest) {
		0 ignored issues – show Duplication introduced 2016-01-05 13:31 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
47		$vocabs = $this->request->getQueryParam('vocab');
48		return ($vocabs !== null && $vocabs !== '') ? explode(' ', $vocabs) : null;
49		}
50	View Code Duplication	if ($this->request->getQueryParam('vocabs')) {
		0 ignored issues – show Duplication introduced 2016-01-05 13:31 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
51		$vocabs = $this->request->getQueryParam('vocabs');
52		return ($vocabs !== null && $vocabs !== '') ? explode(' ', $vocabs) : null;
53		}
54		$vocabs = $this->getVocabs();
55		if ($vocabs[0]) {
56		return array($vocabs[0]->getId());
57		}
58		return null;
59		}
60
61		public function setVocabularies($vocabs)
62		{
63		$this->vocabs = $vocabs;
64		}
65
66		public function getArrayClass()
67		{
68		if (sizeof($this->getVocabs()) == 1) { // search within vocabulary
69		$vocabs = $this->getVocabs();
70		return $vocabs[0]->getConfig()->getArrayClassURI();
71		}
72		return null;
73		}
74
75		public function getSearchTerm()
76		{
77		$term = $this->request->getQueryParam('q') ? $this->request->getQueryParam('q') : $this->request->getQueryParam('query');
78		if (!$term && $this->rest)
79		$term = $this->request->getQueryParam('label');
80		$term = trim($term); // surrounding whitespace is not considered significant
81		return strpos($term, "") === false ? $term . "" : $term; // default to prefix search
82		}
83
84		public function getContentLang()
85		{
86		return $this->request->getContentLang();
87		}
88
89		public function getSearchLang()
90		{
91		if ($this->rest) {
92		return $this->request->getQueryParam('lang');
93		}
94		return $this->request->getQueryParam('anylang') ? '' : $this->getContentLang();
95		}
96
97		public function getTypeLimit()
98		{
99		$type = $this->request->getQueryParam('type') !== '' ? $this->request->getQueryParam('type') : null;
100		if ($type && strpos($type, '+')) {
101		$type = explode('+', $type);
102		} else if ($type && !is_array($type)) {
103		// if only one type param given place it into an array regardless
104		$type = array($type);
105		}
106		if ($type === null) {
107		$type = array('skos:Concept');
108	View Code Duplication	if ($this->request->getVocab() && $this->request->getVocab()->getConfig()->getArrayClassURI()) {
		0 ignored issues – show Duplication introduced 2016-01-08 11:26 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
109		array_push($type, $this->request->getVocab()->getConfig()->getArrayClassURI());
110		}
111	View Code Duplication	if ($this->request->getVocab() && $this->request->getVocab()->getConfig()->getGroupClassURI()) {
		0 ignored issues – show Duplication introduced 2016-01-08 11:26 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
112		array_push($type, $this->request->getVocab()->getConfig()->getGroupClassURI());
113		}
114		}
115		return $type;
116		}
117
118		public function getGroupLimit()
119		{
120		return $this->request->getQueryParam('group') !== '' ? $this->request->getQueryParam('group') : null;
121		}
122
123		public function getParentLimit()
124		{
125		return $this->request->getQueryParam('parent') !== '' ? $this->request->getQueryParam('parent') : null;
126		}
127
128		public function getOffset()
129		{
130		return ($this->request->getQueryParam('offset') && is_numeric($this->request->getQueryParam('offset')) && $this->request->getQueryParam('offset') >= 0) ? $this->request->getQueryParam('offset') : 0;
131		}
132
133		public function getSearchLimit()
134		{
135		return $this->config->getDefaultSearchLimit();
136		}
137
138		public function getUnique() {
139		return $this->unique;
140		}
141
142		public function setUnique($unique) {
143		$this->unique = $unique;
144		}
145
146		public function getAdditionalFields() {
147		return $this->request->getQueryParam('fields') ? explode(' ', $request->getQueryParam('fields')) : null;
		0 ignored issues – show Bug introduced 2016-01-05 13:31 UTC by Report Bug Copy Issue Report The variable `$request` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
148		}
149
150		public function getHidden() {
151		return $this->hidden;
152		}
153
154		public function setHidden($hidden) {
155		$this->hidden = $hidden;
156		}
157
158		public function setAdditionalFields($fields) {
159		$this->fields = $fields;
		0 ignored issues – show Bug introduced 2016-01-05 13:31 UTC by Report Bug Copy Issue Report The property `fields` does not exist. Did you maybe forget to declare it? In PHP it is possible to write to properties without declaring them. For example, the following is perfectly valid PHP code: class MyClass { } $x = new MyClass(); $x->foo = true; Generally, it is a good practice to explictly declare properties to avoid accidental typos and provide IDE auto-completion: class MyClass { public $foo; } $x = new MyClass(); $x->foo = true; Loading history...
160		}
161
162		}
163

NatLibFi / Skosmos