ExecutionsProvider::cast() - Code Metrics - Inspection of "Completely refactoring" - rambler-digital-solutions/hydrogen - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( b17eba...69b436 )

by Kirill

created 2018-09-02 05:44 UTC

ExecutionsProvider::cast() B

↳ Parent: ExecutionsProvider

Complexity

Conditions	6
Paths	6

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	42

Importance

Changes

Metric	Value
dl	0
loc	26
ccs	0
cts	15
cp	0
rs	8.8817
c	0
b	0
f	0
cc	6
nc	6
nop	2
crap	42

<?php
/**
 * This file is part of Hydrogen package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace RDS\Hydrogen\Query;

use RDS\Hydrogen\Query;
use RDS\Hydrogen\Collection\Collection;

/**
 * Class ExecutionsProvider
 * @mixin Query
 */
trait ExecutionsProvider
{
    /**
     * @param string ...$fields
     * @return object[]|iterable
     */
    public function get(string ...$fields): iterable
    {
        $processor = $this->getRepository()->getProcessor();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        if (\count($fields) === 0) {
            return $processor->getResult($this);
        }

        return Collection::wrap($processor->getArrayResult($this))
            ->map(function ($item) use ($fields) {
                $result = [];

                foreach ($fields as $field) {
                    $result[$field] = \data_get($item, $field);
                }

                return $result;
            })
            ->toArray();
    }

    /**
     * @param string $field
     * @param string|null $typeOf
     * @return mixed
     * @throws \LogicException
     */
    public function scalar(string $field, string $typeOf = null)
    {
        $result = \data_get($this->first($field), $field);

        if ($typeOf !== null) {
            return $this->cast($result, $typeOf);
        }

        return $result;
    }

    /**
     * @param mixed $result
     * @param string $typeOf
     * @return array|\Closure|object
     */
    private function cast($result, string $typeOf)
    {
        $typeOf = \strtolower($typeOf);

        switch ($typeOf) {
            case 'callable':
                return function (callable $applicator = null) use ($result) {
                    return ($applicator ?? '\\value')($result);
                };

            case 'object':
                return (object)$result;

            case 'array':
            case 'iterable':
                return (array)$result;
        }

        $function = $typeOf . 'val';

        if (! \function_exists($function)) {
            throw new \InvalidArgumentException('Could not cast to type ' . $typeOf);
        }

        return $function($result);
    }

    /**
     * @param string|null $field
     * @return int
     * @throws \LogicException
     */
    public function count(?string $field = 'id'): int
    {
        return $this->select('COUNT(' . $field . ') AS __count')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ->scalar('__count', 'int');
    }

    /**
     * @param string|null $field
     * @return int
     * @throws \LogicException
     */
    public function sum(string $field = null): int
    {
        return $this->select('SUM(' . $field . ') AS __sum')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ->scalar('__sum', 'int');
    }

    /**
     * @param string|null $field
     * @return int
     * @throws \LogicException
     */
    public function avg(string $field = null): int
    {
        return $this->select('AVG(' . $field . ') AS __avg')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ->scalar('__avg', 'int');
    }

    /**
     * @param string|null $field
     * @return int
     * @throws \LogicException
     */
    public function max(string $field = null): int
    {
        return $this->select('MAX(' . $field . ') AS __max')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ->scalar('__max', 'int');
    }

    /**
     * @param string|null $field
     * @return int
     * @throws \LogicException
     */
    public function min(string $field = null): int
    {
        return $this->select('MIN(' . $field . ') AS __min')
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            ->scalar('__min', 'int');
    }

    /**
     * @param string ...$fields
     * @return Collection
     */
    public function collect(string ...$fields): Collection
    {
        return Collection::wrap($this->get(...$fields));
    }

    /**
     * @param string[] $fields
     * @return object|null
     * @throws \LogicException
     */
    public function first(string ...$fields)
    {
        return \array_first($this->get(...$fields));
    }
}


1			<?php
2			/**
3			* This file is part of Hydrogen package.
4			*
5			* For the full copyright and license information, please view the LICENSE
6			* file that was distributed with this source code.
7			*/
8			declare(strict_types=1);
9
10			namespace RDS\Hydrogen\Query;
11
12			use RDS\Hydrogen\Query;
13			use RDS\Hydrogen\Collection\Collection;
14
15			/**
16			* Class ExecutionsProvider
17			* @mixin Query
18			*/
19			trait ExecutionsProvider
20			{
21			/**
22			* @param string ...$fields
23			* @return object[]\|iterable
24			*/
25			public function get(string ...$fields): iterable
26			{
27			$processor = $this->getRepository()->getProcessor();
			0 ignored issues – show Bug introduced 2018-09-02 05:50 UTC by Report Bug Copy Issue Report It seems like `getRepository()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
28
29			if (\count($fields) === 0) {
30			return $processor->getResult($this);
31			}
32
33			return Collection::wrap($processor->getArrayResult($this))
34			->map(function ($item) use ($fields) {
35			$result = [];
36
37			foreach ($fields as $field) {
38			$result[$field] = \data_get($item, $field);
39			}
40
41			return $result;
42			})
43			->toArray();
44			}
45
46			/**
47			* @param string $field
48			* @param string\|null $typeOf
49			* @return mixed
50			* @throws \LogicException
51			*/
52			public function scalar(string $field, string $typeOf = null)
53			{
54			$result = \data_get($this->first($field), $field);
55
56			if ($typeOf !== null) {
57			return $this->cast($result, $typeOf);
58			}
59
60			return $result;
61			}
62
63			/**
64			* @param mixed $result
65			* @param string $typeOf
66			* @return array\|\Closure\|object
67			*/
68			private function cast($result, string $typeOf)
69			{
70			$typeOf = \strtolower($typeOf);
71
72			switch ($typeOf) {
73			case 'callable':
74			return function (callable $applicator = null) use ($result) {
75			return ($applicator ?? '\\value')($result);
76			};
77
78			case 'object':
79			return (object)$result;
80
81			case 'array':
82			case 'iterable':
83			return (array)$result;
84			}
85
86			$function = $typeOf . 'val';
87
88			if (! \function_exists($function)) {
89			throw new \InvalidArgumentException('Could not cast to type ' . $typeOf);
90			}
91
92			return $function($result);
93			}
94
95			/**
96			* @param string\|null $field
97			* @return int
98			* @throws \LogicException
99			*/
100			public function count(?string $field = 'id'): int
101			{
102			return $this->select('COUNT(' . $field . ') AS __count')
			0 ignored issues – show Bug introduced 2018-09-02 05:50 UTC by Report Bug Copy Issue Report It seems like `select()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
103			->scalar('__count', 'int');
104			}
105
106			/**
107			* @param string\|null $field
108			* @return int
109			* @throws \LogicException
110			*/
111			public function sum(string $field = null): int
112			{
113			return $this->select('SUM(' . $field . ') AS __sum')
			0 ignored issues – show Bug introduced 2018-09-02 05:50 UTC by Report Bug Copy Issue Report It seems like `select()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
114			->scalar('__sum', 'int');
115			}
116
117			/**
118			* @param string\|null $field
119			* @return int
120			* @throws \LogicException
121			*/
122			public function avg(string $field = null): int
123			{
124			return $this->select('AVG(' . $field . ') AS __avg')
			0 ignored issues – show Bug introduced 2018-09-02 05:50 UTC by Report Bug Copy Issue Report It seems like `select()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
125			->scalar('__avg', 'int');
126			}
127
128			/**
129			* @param string\|null $field
130			* @return int
131			* @throws \LogicException
132			*/
133			public function max(string $field = null): int
134			{
135			return $this->select('MAX(' . $field . ') AS __max')
			0 ignored issues – show Bug introduced 2018-09-02 05:50 UTC by Report Bug Copy Issue Report It seems like `select()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
136			->scalar('__max', 'int');
137			}
138
139			/**
140			* @param string\|null $field
141			* @return int
142			* @throws \LogicException
143			*/
144			public function min(string $field = null): int
145			{
146			return $this->select('MIN(' . $field . ') AS __min')
			0 ignored issues – show Bug introduced 2018-09-02 05:50 UTC by Report Bug Copy Issue Report It seems like `select()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
147			->scalar('__min', 'int');
148			}
149
150			/**
151			* @param string ...$fields
152			* @return Collection
153			*/
154			public function collect(string ...$fields): Collection
155			{
156			return Collection::wrap($this->get(...$fields));
157			}
158
159			/**
160			* @param string[] $fields
161			* @return object\|null
162			* @throws \LogicException
163			*/
164			public function first(string ...$fields)
165			{
166			return \array_first($this->get(...$fields));
167			}
168			}
169

rambler-digital-solutions / hydrogen

Push — master ( b17eba...69b436 )

ExecutionsProvider::cast() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like