AbstractTable - Code Metrics - Inspection of "added property alias to from method" - phuria/under-query - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 1a3ae6...a9ee46 )

by Beniamin

created 2016-09-26 05:54 UTC

AbstractTable A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	159
Duplicated Lines	0 %

Coupling/Cohesion

Components	4
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

Metric	Value
dl	0
loc	159
c	0
b	0
f	0
wmc	14
lcom	4
cbo	2
ccs	32
cts	32
cp	1
rs	10

14 Methods

Rating	Name	Size	Complexity
A	__construct()	4	1
A	__toString()	4	1
	getTableName()	1	?
A	getQueryBuilder()	4	1
A	getAlias()	4	1
A	setAlias()	6	1
A	isJoin()	4	1
A	getJoinType()	4	1
A	setJoinType()	7	1
A	getAliasOrName()	4	2
A	addSelect()	6	1
A	joinOn()	6	1
A	getJoinOn()	4	1
A	column()	4	1

<?php

namespace Phuria\SQLBuilder\Table;

use Phuria\SQLBuilder\QueryBuilder\AbstractBuilder;

/**
 * @author Beniamin Jonatan Šimko <[email protected]>
 */
abstract class AbstractTable
{
    const CROSS_JOIN = 'CROSS JOIN';
    const LEFT_JOIN = 'LEFT JOIN';
    const INNER_JOIN = 'INNER JOIN';

    /**
     * @var AbstractBuilder $qb
     */
    private $qb;

    /**
     * @var string $tableAlias
     */
    private $tableAlias;

    /**
     * @var string $joinType
     */
    private $joinType;

    /**
     * @var string $joinOn
     */
    private $joinOn;

    /**
     * @var bool $join
     */
    private $join = false;

    /**
     * @param AbstractBuilder $qb
     */
    public function __construct(AbstractBuilder $qb)
    {
        $this->qb = $qb;
    }

    /**
     * @return string
     */
    public function __toString()
    {
        return $this->qb->getReferenceManager()->register($this);
    }

    /**
     * @return string
     */
    abstract public function getTableName();

    /**
     * @return AbstractBuilder
     */
    public function getQueryBuilder()
    {
        return $this->qb;
    }

    /**
     * @return string
     */
    public function getAlias()
    {
        return $this->tableAlias;
    }

    /**
     * @param string $alias
     *
     * @return $this
     */
    public function setAlias($alias)
    {
        $this->tableAlias = $alias;

        return $this;
    }

    /**
     * @return bool
     */
    public function isJoin()
    {
        return $this->join;
    }

    /**
     * @return string
     */
    public function getJoinType()
    {
        return $this->joinType;
    }

    /**
     * @param string $joinType
     *
     * @return $this
     */
    public function setJoinType($joinType)
    {
        $this->joinType = $joinType;
        $this->join = true;

        return $this;
    }

    /**
     * @return string
     */
    public function getAliasOrName()
    {
        return $this->getAlias() ?: $this->getTableName();
    }

    /**
     * @param string $clause
     *
     * @return $this
     */
    public function addSelect($clause)
    {
        $this->qb->addSelect($clause);
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

        return $this;
    }

    /**
     * @param string $clause
     *
     * @return $this
     */
    public function joinOn($clause)
    {
        $this->joinOn = $clause;

        return $this;
    }

    /**
     * @return string
     */
    public function getJoinOn()
    {
        return $this->joinOn;
    }

    /**
     * @param string $name
     *
     * @return string
     */
    public function column($name)
    {
        return $this . '.' . $name;
    }
}

1		<?php
2
3		namespace Phuria\SQLBuilder\Table;
4
5		use Phuria\SQLBuilder\QueryBuilder\AbstractBuilder;
6
7		/**
8		* @author Beniamin Jonatan Šimko <[email protected]>
9		*/
10		abstract class AbstractTable
11		{
12		const CROSS_JOIN = 'CROSS JOIN';
13		const LEFT_JOIN = 'LEFT JOIN';
14		const INNER_JOIN = 'INNER JOIN';
15
16		/**
17		* @var AbstractBuilder $qb
18		*/
19		private $qb;
20
21		/**
22		* @var string $tableAlias
23		*/
24		private $tableAlias;
25
26		/**
27		* @var string $joinType
28		*/
29		private $joinType;
30
31		/**
32		* @var string $joinOn
33		*/
34		private $joinOn;
35
36		/**
37		* @var bool $join
38		*/
39		private $join = false;
40
41		/**
42		* @param AbstractBuilder $qb
43		*/
44	30	public function __construct(AbstractBuilder $qb)
45		{
46	30	$this->qb = $qb;
47	30	}
48
49		/**
50		* @return string
51		*/
52	20	public function __toString()
53		{
54	20	return $this->qb->getReferenceManager()->register($this);
55		}
56
57		/**
58		* @return string
59		*/
60		abstract public function getTableName();
61
62		/**
63		* @return AbstractBuilder
64		*/
65	1	public function getQueryBuilder()
66		{
67	1	return $this->qb;
68		}
69
70		/**
71		* @return string
72		*/
73	28	public function getAlias()
74		{
75	28	return $this->tableAlias;
76		}
77
78		/**
79		* @param string $alias
80		*
81		* @return $this
82		*/
83	9	public function setAlias($alias)
84		{
85	9	$this->tableAlias = $alias;
86
87	9	return $this;
88		}
89
90		/**
91		* @return bool
92		*/
93	28	public function isJoin()
94		{
95	28	return $this->join;
96		}
97
98		/**
99		* @return string
100		*/
101	3	public function getJoinType()
102		{
103	3	return $this->joinType;
104		}
105
106		/**
107		* @param string $joinType
108		*
109		* @return $this
110		*/
111	3	public function setJoinType($joinType)
112		{
113	3	$this->joinType = $joinType;
114	3	$this->join = true;
115
116	3	return $this;
117		}
118
119		/**
120		* @return string
121		*/
122	20	public function getAliasOrName()
123		{
124	20	return $this->getAlias() ?: $this->getTableName();
125		}
126
127		/**
128		* @param string $clause
129		*
130		* @return $this
131		*/
132	4	public function addSelect($clause)
133		{
134	4	$this->qb->addSelect($clause);
		0 ignored issues – show Bug introduced 2016-09-26 04:50 UTC by Report Bug Copy Issue Report It seems like you code against a specific sub-type and not the parent class `Phuria\SQLBuilder\QueryBuilder\AbstractBuilder` as the method `addSelect()` does only exist in the following sub-classes of `Phuria\SQLBuilder\QueryBuilder\AbstractBuilder`: `Phuria\SQLBuilder\QueryBuilder\SelectBuilder`. Maybe you want to `instanceof` check for one of these explicitly? Let’s take a look at an example: abstract class User { /** @return string / abstract public function getPassword(); } class MyUser extends User { public function getPassword() { // return something } public function getDisplayName() { // return some name. } } class AuthSystem { public function authenticate(User $user) { $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName())); // do something. } } In the above example, the authenticate() method works fine as long as you just pass instances of MyUser. However, if you now also want to pass a different sub-classes of User which does not have a getDisplayName() method, the code will break. Available Fixes Change the type-hint for the parameter: class AuthSystem { public function authenticate(MyUser $user) { / ... / } } Add an additional type-check: class AuthSystem { public function authenticate(User $user) { if ($user instanceof MyUser) { $this->logger->info(/* ... /); } // or alternatively if ( ! $user instanceof MyUser) { throw new \LogicException( '$user must be an instance of MyUser, ' .'other instances are not supported.' ); } } } Note:* PHP Analyzer uses reverse abstract interpretation to narrow down the types inside the if block in such a case. Add the method to the parent class: abstract class User { /** @return string / abstract public function getPassword(); /* @return string */ abstract public function getDisplayName(); } Loading history...
135
136	4	return $this;
137		}
138
139		/**
140		* @param string $clause
141		*
142		* @return $this
143		*/
144	2	public function joinOn($clause)
145		{
146	2	$this->joinOn = $clause;
147
148	2	return $this;
149		}
150
151		/**
152		* @return string
153		*/
154	28	public function getJoinOn()
155		{
156	28	return $this->joinOn;
157		}
158
159		/**
160		* @param string $name
161		*
162		* @return string
163		*/
164	20	public function column($name)
165		{
166	20	return $this . '.' . $name;
167		}
168		}

phuria / under-query

Push — master ( 1a3ae6...a9ee46 )

AbstractTable A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

14 Methods

Available Fixes

Duplication Side-by-Side

Filter issues like