plPhpClass::constructorParameters() - Code Metrics - Inspection of "refactor: Extract Digraph class from Graphviz proc..." - MontealegreLuis/phuml - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — graphviz-refactoring ( 3f1b14 )

by Luis

created 2017-12-12 20:05 UTC

plPhpClass::constructorParameters() A

↳ Parent: plPhpClass

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	11
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	0

<?php

class plPhpClass implements plHasNodeIdentifier
{
    use plObjectHashIdentifier;

    /** @var string */
    public $name;

    /** @var plPhpAttribute[] */
    public $attributes;

    /** @var plPhpFunction[] */
    public $functions;

    /** @var plPhpInterface[] */
    public $implements;

    /** @var plPhpClass */
    public $extends;

    public function __construct(
        string $name,
        array $attributes = [],
        array $functions = [],
        array $implements = [],
        $extends = null
    ) {
        $this->name = $name;
        $this->attributes = $attributes;
        $this->functions = $functions;
        $this->implements = $implements;
        $this->extends = $extends;
    }


    public function hasConstructor(): bool
    {
        return count(array_filter($this->functions, function (plPhpFunction $function) {
            return $function->isConstructor();
        })) === 1;
    }

    /** @return plPhpVariable[] */
    public function constructorParameters(): array
    {
        if (!$this->hasConstructor()) {
            return [];
        }

        $constructors = array_filter($this->functions, function (plPhpFunction $function) {
            return $function->isConstructor();
        });

        return reset($constructors)->params;
    }

    public function hasParent(): bool
    {
        return $this->extends !== null;
    }
}


1			<?php
2
3			class plPhpClass implements plHasNodeIdentifier
4			{
5			use plObjectHashIdentifier;
6
7			/** @var string */
8			public $name;
9
10			/** @var plPhpAttribute[] */
11			public $attributes;
12
13			/** @var plPhpFunction[] */
14			public $functions;
15
16			/** @var plPhpInterface[] */
17			public $implements;
18
19			/** @var plPhpClass */
20			public $extends;
21
22			public function __construct(
23			string $name,
24			array $attributes = [],
25			array $functions = [],
26			array $implements = [],
27			$extends = null
28			) {
29			$this->name = $name;
30			$this->attributes = $attributes;
31			$this->functions = $functions;
32			$this->implements = $implements;
33			$this->extends = $extends;
34			}
35
36
37			public function hasConstructor(): bool
38			{
39			return count(array_filter($this->functions, function (plPhpFunction $function) {
40			return $function->isConstructor();
41			})) === 1;
42			}
43
44			/** @return plPhpVariable[] */
45			public function constructorParameters(): array
46			{
47			if (!$this->hasConstructor()) {
48			return [];
49			}
50
51			$constructors = array_filter($this->functions, function (plPhpFunction $function) {
52			return $function->isConstructor();
53			});
54
55			return reset($constructors)->params;
56			}
57
58			public function hasParent(): bool
59			{
60			return $this->extends !== null;
61			}
62			}
63

MontealegreLuis / phuml

Push — graphviz-refactoring ( 3f1b14 )

plPhpClass::constructorParameters() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like