Metric::toArray() - Code Metrics - Inspection of "Updated README.md file" - webmarketingroi/optimizely-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( e6c94f...e27413 )

unknown

created 2016-10-20 00:18 UTC

Metric::toArray() A

↳ Parent: Metric

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	10
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	3
eloc	9
c	1
b	0
f	0
nc	3
nop	0
dl	0
loc	16
ccs	10
cts	10
cp	1
crap	3
rs	9.4285

<?php
/**
 * @author Oleg Krivtsov <[email protected]>
 * @date 10 October 2016
 * @copyright (c) 2016, Web Marketing ROI
 */
namespace WebMarketingROI\OptimizelyPHP\Resource\v2;

/**
 * An Optimizely campaign metric.
 */
class Metric
{
    /**
     * The kind of a Metric
     * @var string 
     */
    private $kind;
    
    /**
     * The unique identifier for the Metric
     * @var integer 
     */
    private $id;
    
    /**
     * Constructor.
     */
    public function __construct($options = array())
    {
        foreach ($options as $name=>$value) {
            switch ($name) {                
                case 'kind': $this->setKind($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                case 'id': $this->setId($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                default:
                    throw new \Exception('Unknown option: ' . $name);
            }
        }
    }
    
    /**
     * Returns this object as array.
     */
    public function toArray()
    {
        $options = array(
            'kind' => $this->getKind(),
            'id' => $this->getId(),            
        );
        
        // Remove options with empty values
        $cleanedOptions = array();
        foreach ($options as $name=>$value) {
            if ($value!==null)
                $cleanedOptions[$name] = $value;
        }
        
        return $cleanedOptions;
    }
    
    public function getKind()
    {
        return $this->kind;
    }
    
    public function setKind($kind)
    {
        $this->kind = $kind;
    }
    
    public function getId()
    {
        return $this->id;
    }
    
    public function setId($id)
    {
        $this->id = $id;
    }
}







1		<?php
2		/**
3		* @author Oleg Krivtsov <[email protected]>
4		* @date 10 October 2016
5		* @copyright (c) 2016, Web Marketing ROI
6		*/
7		namespace WebMarketingROI\OptimizelyPHP\Resource\v2;
8
9		/**
10		* An Optimizely campaign metric.
11		*/
12		class Metric
13		{
14		/**
15		* The kind of a Metric
16		* @var string
17		*/
18		private $kind;
19
20		/**
21		* The unique identifier for the Metric
22		* @var integer
23		*/
24		private $id;
25
26		/**
27		* Constructor.
28		*/
29	14	public function __construct($options = array())
30		{
31	14	foreach ($options as $name=>$value) {
32		switch ($name) {
33	12	case 'kind': $this->setKind($value); break;
		0 ignored issues – show Coding Style introduced 2016-10-19 01:09 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2016-10-19 01:09 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
34	8	case 'id': $this->setId($value); break;
		0 ignored issues – show Coding Style introduced 2016-10-19 01:09 UTC by Report Bug Copy Issue Report The case body in a switch statement must start on the line following the statement. According to the PSR-2, the body of a case statement must start on the line immediately following the case statement. switch ($expr) { case "A": doSomething(); //right break; case "B": doSomethingElse(); //wrong break; } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history... Coding Style introduced 2016-10-19 01:09 UTC by Report Bug Copy Issue Report Terminating statement must be on a line by itself As per the PSR-2 coding standard, the `break` (or other terminating) statement must be on a line of its own. switch ($expr) { case "A": doSomething(); break; //wrong case "B": doSomething(); break; //right case "C:": doSomething(); return true; //right } To learn more about the PSR-2 coding standard, please refer to the PHP-Fig. Loading history...
35		default:
36		throw new \Exception('Unknown option: ' . $name);
37		}
38	14	}
39	14	}
40
41		/**
42		* Returns this object as array.
43		*/
44	6	public function toArray()
45		{
46		$options = array(
47	6	'kind' => $this->getKind(),
48	6	'id' => $this->getId(),
49	6	);
50
51		// Remove options with empty values
52	6	$cleanedOptions = array();
53	6	foreach ($options as $name=>$value) {
54	6	if ($value!==null)
55	6	$cleanedOptions[$name] = $value;
56	6	}
57
58	6	return $cleanedOptions;
59		}
60
61	9	public function getKind()
62		{
63	9	return $this->kind;
64		}
65
66	14	public function setKind($kind)
67		{
68	14	$this->kind = $kind;
69	14	}
70
71	6	public function getId()
72		{
73	6	return $this->id;
74		}
75
76	9	public function setId($id)
77		{
78	9	$this->id = $id;
79	9	}
80		}
81
82
83
84
85
86

webmarketingroi / optimizely-php

Push — master ( e6c94f...e27413 )

Metric::toArray() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like