Using::convertToArray() - Code Metrics - puzzle-org/query-builder - Measure and Improve Code Quality continuously with Scrutinizer

Using::convertToArray() A
last analyzed 2017-07-02 16:39 UTC

↳ Parent: Using

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	3.0416

Importance

Changes

Metric	Value
dl	0
loc	14
c	0
b	0
f	0
ccs	5
cts	6
cp	0.8333
rs	9.4285
cc	3
eloc	6
nc	3
nop	1
crap	3.0416

<?php

declare(strict_types = 1);

namespace Puzzle\QueryBuilder\Queries\Snippets;

use Puzzle\QueryBuilder\Snippet;
use Puzzle\Pieces\StringManipulation;

class Using implements Snippet
{
    use StringManipulation;

    private
        $columns;
class A {
    var $property;
}

    /**
     * @param array[string]|string $column

     */
    public function __construct($column)
    {
        $this->columns = $this->convertToArray($column);
    }

    public function toString(): string
    {
        $usingColumns = implode(', ', array_filter($this->columns));

        if(empty($usingColumns))
        {
            return '';
        }

        return sprintf(
            'USING (%s)',
            $usingColumns
        );
    }

    private function convertToArray($input): array
    {
        if(! is_array($input))
        {
            if(! $this->isConvertibleToString($input))
            {
                throw new \InvalidArgumentException("Using argument must be a string (or an array of string)");
            }

            $input = array($input);
        }

        return $input;
    }
}


1		<?php
2
3		declare(strict_types = 1);
4
5		namespace Puzzle\QueryBuilder\Queries\Snippets;
6
7		use Puzzle\QueryBuilder\Snippet;
8		use Puzzle\Pieces\StringManipulation;
9
10		class Using implements Snippet
11		{
12		use StringManipulation;
13
14		private
15		$columns;
		1 ignored issue – show Coding Style introduced 2017-07-02 16:33 UTC by Report Bug Copy Issue Report The visibility should be declared for property `$columns`. The PSR-2 coding standard requires that all properties in a class have their visibility explicitly declared. If you declare a property using class A { var $property; } the property is implicitly global. To learn more about the PSR-2, please see the PHP-FIG site on the PSR-2. Loading history...
16
17		/**
18		* @param array[string]\|string $column
		0 ignored issues – show Documentation introduced 2017-07-02 16:33 UTC by Report Bug Copy Issue Report The doc-type `array[string]\|string` could not be parsed: Expected "]" at position 2, but found "string". (view supported doc-types) This check marks PHPDoc comments that could not be parsed by our parser. To see which comment annotations we can parse, please refer to our documentation on supported doc-types. Loading history...
19		*/
20	14	public function __construct($column)
21		{
22	14	$this->columns = $this->convertToArray($column);
23	14	}
24
25	14	public function toString(): string
26		{
27	14	$usingColumns = implode(', ', array_filter($this->columns));
28
29	14	if(empty($usingColumns))
30		{
31	2	return '';
32		}
33
34	12	return sprintf(
35	12	'USING (%s)',
36	12	$usingColumns
37		);
38		}
39
40	14	private function convertToArray($input): array
41		{
42	14	if(! is_array($input))
43		{
44	12	if(! $this->isConvertibleToString($input))
45		{
46		throw new \InvalidArgumentException("Using argument must be a string (or an array of string)");
47		}
48
49	12	$input = array($input);
50		}
51
52	14	return $input;
53		}
54		}
55

puzzle-org / query-builder

Using::convertToArray() A last analyzed 2017-07-02 16:39 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

Using::convertToArray() A
last analyzed 2017-07-02 16:39 UTC