Artifacts::parse() - Code Metrics - Inspection of "prepare 0.0.51 release" - ktomk/pipelines - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( ce6411...dc02eb )

by Tom

created 2020-12-09 19:48 UTC

Artifacts::parse() A

↳ Parent: Artifacts

Complexity

Conditions	4
Paths	4

Size

Total Lines	17
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	4

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	4
eloc	8
nc	4
nop	1
dl	0
loc	17
ccs	8
cts	8
cp	1
crap	4
rs	10
c	1
b	0
f	0

<?php

/* this file is part of pipelines */

namespace Ktomk\Pipelines\File;

/**
 * Artifacts of a pipelines step
 *
 * @package Ktomk\Pipelines\File\File
 */
class Artifacts implements \Countable
{
    /**
     * @var array|string[]
     */
    private $artifacts;

    /**
     * Artifacts constructor.
     *
     * @param array|string[] $artifacts
     *
     * @throws ParseException
     */
    public function __construct(array $artifacts)
    {
        $this->parse($artifacts);
    }

    /**
     * @return array|string[]
     */
    public function getPatterns()
    {
        return $this->artifacts;
    }

    /**
     * @return int
     */
    public function count()
    {
        return count($this->artifacts);
    }

    /**
     * @param array|string[] $artifacts
     *
     * @throws ParseException
     *
     * @return void
     */
    private function parse(array $artifacts)
    {
        // quick validation: requires a list of strings
        if (!count($artifacts)) {
            throw new ParseException("'artifacts' requires a list");
        }

        foreach ($artifacts as $index => $string) {
            if (!is_string($string)) {
                throw new ParseException(sprintf(
                    "'artifacts' requires a list of strings, #%d is not a string",
                    $index
                ));
            }
        }

        $this->artifacts = $artifacts;
    }
}


1		<?php
2
3		/* this file is part of pipelines */
4
5		namespace Ktomk\Pipelines\File;
6
7		/**
8		* Artifacts of a pipelines step
9		*
10		* @package Ktomk\Pipelines\File\File
11		*/
12		class Artifacts implements \Countable
13		{
14		/**
15		* @var array\|string[]
16		*/
17		private $artifacts;
18
19		/**
20		* Artifacts constructor.
21		*
22		* @param array\|string[] $artifacts
23		*
24		* @throws ParseException
25		*/
26	5	public function __construct(array $artifacts)
27		{
28	5	$this->parse($artifacts);
29	3	}
30
31		/**
32		* @return array\|string[]
33		*/
34	1	public function getPatterns()
35		{
36	1	return $this->artifacts;
37		}
38
39		/**
40		* @return int
41		*/
42	1	public function count()
43		{
44	1	return count($this->artifacts);
45		}
46
47		/**
48		* @param array\|string[] $artifacts
49		*
50		* @throws ParseException
51		*
52		* @return void
53		*/
54	5	private function parse(array $artifacts)
55		{
56		// quick validation: requires a list of strings
57	5	if (!count($artifacts)) {
58	1	throw new ParseException("'artifacts' requires a list");
59		}
60
61	4	foreach ($artifacts as $index => $string) {
62	4	if (!is_string($string)) {
63	1	throw new ParseException(sprintf(
64	1	"'artifacts' requires a list of strings, #%d is not a string",
65		$index
66		));
67		}
68		}
69
70	3	$this->artifacts = $artifacts;
71	3	}
72		}
73

ktomk / pipelines

Push — master ( ce6411...dc02eb )

Artifacts::parse() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like