VariantResults::__construct() - Code Metrics - Inspection of "Merge branch 'master' of https://github.com/webmar..." - webmarketingroi/optimizely-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 979a9d...8b1dd2 )

by Oleg

created 2018-05-14 05:46 UTC

VariantResults::__construct() C

↳ Parent: VariantResults

Complexity

Conditions	14
Paths	14

Size

Total Lines	21
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	14
CRAP Score	14.3828

Importance

Changes

Metric	Value
cc	14
eloc	17
nc	14
nop	1
dl	0
loc	21
ccs	14
cts	16
cp	0.875
crap	14.3828
rs	6.5272
c	0
b	0
f	0

How to fix Complexity

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

use WebMarketingROI\OptimizelyPHP\Exception;
use WebMarketingROI\OptimizelyPHP\Resource\v2\Datapoint;

/**
 * Optimizely campaign variant results.
 */
class VariantResults
{
    /**
     * The unique identifier for the Experiment this entity contains results for (if applicable).
     * @var integer 
     */
    private $experimentId;
    
    /**
     * Indicates that this variant is the baseline that all other entities will 
     * be compared against. Also referred to as the 'Control' or 'Control Group'.
     * @var boolean
     */
    private $isBaseline;
    
    /**
     *
     * @var type 
     */
    private $level;
    
    /**
     * The relative difference in performance of this variant vs. the baseline 
     * variant. Lift is calculated as follows: (Winning Conversion Rate % - Old 
     * Conversion Rate %) - Old Conversion Rate % = % Improvement
     * @var Datapoint
     */
    private $lift;
    
    /**
     * The name of the variant
     * @var string
     */
    private $name;
    
    /**
     *
     * @var number
     */
    private $rate;
    
    /**
     * The scope that this variant represents. Can be 'variation', 'experiment' or 'campaign'
     * @var string 
     */
    private $scope;
    
    /**
     *
     * @var Datapoint
     */
    private $totalIncrease;
    
    /**
     *
     * @var number
     */
    private $value;
    
    /**
     * 
     * @var string 
     */
    private $variationId;
    
    /**
     *
     * @var type 
     */
    private $samples;
    
    /**
     *
     * @var type 
     */
    private $variance;
    
    /**
     * Constructor.
     */
    public function __construct($options = array())
    {
        foreach ($options as $name=>$value) {
            switch ($name) {                
                case 'experiment_id': $this->setExperimentId($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

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

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

    doSomethingElse(); //wrong
    break;
                case 'lift': $this->setLift(new Datapoint($value)); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

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

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

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

    doSomethingElse(); //wrong
    break;
                case 'total_increase': $this->setTotalIncrease(new Datapoint($value)); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

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

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

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

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

    doSomethingElse(); //wrong
    break;
                default:
                    throw new Exception('Unknown option found in the VariantResults entity: ' . $name);
            }
        }
    }
    
    /**
     * Returns this object as array.
     */
    public function toArray()
    {
        $options = array(
            'experiment_id' => $this->getExperimentId(),
            'is_baseline' => $this->getIsBaseline(),
            'level' => $this->getLevel(),
            'lift' => $this->getLift()?$this->getLift()->toArray():null,
            'name' => $this->getName(),
            'rate' => $this->getRate(),
            'scope' => $this->getScope(),
            'total_increase' => $this->getTotalIncrease()?$this->getTotalIncrease()->toArray():null,
            'value' => $this->getValue(),
            'variation_id' => $this->getVariationId(),
            'samples' => $this->getSamples(),
            'variance' => $this->getVariance(),
        );
        
        // Remove options with empty values
        $cleanedOptions = array();
        foreach ($options as $name=>$value) {
            if ($value!==null)
                $cleanedOptions[$name] = $value;
        }
        
        return $cleanedOptions;
    }
    
    public function getExperimentId()
    {
        return $this->experimentId;
    }
    
    public function setExperimentId($experimentId)
    {
        $this->experimentId = $experimentId;
    }
    
    public function getIsBaseline()
    {
        return $this->isBaseline;
    }
    
    public function setIsBaseline($isBaseline)
    {
        $this->isBaseline = $isBaseline;
    }
    
    public function getLevel()
    {
        return $this->level;
    }
    
    public function setLevel($level)
    {
        $this->level = $level;
    }
    
    public function getLift()
    {
        return $this->lift;
    }
    
    public function setLift($lift)
    {
        $this->lift = $lift;
    }
    
    public function getName()
    {
        return $this->name;
    }
    
    public function setName($name)
    {
        $this->name = $name;
    }
    
    public function getRate()
    {
        return $this->rate;
    }
    
    public function setRate($rate)
    {
        $this->rate = $rate;
    }
    
    public function getScope()
    {
        return $this->scope;
    }
    
    public function setScope($scope)
    {
        $this->scope = $scope;
    }
    
    public function getTotalIncrease()
    {
        return $this->totalIncrease;
    }
    
    public function setTotalIncrease($totalIncrease)
    {
        $this->totalIncrease = $totalIncrease;
    }
    
    public function getValue()
    {
        return $this->value;
    }
    
    public function setValue($value)
    {
        $this->value = $value;
    }
    
    public function getVariationId()
    {
        return $this->variationId;
    }
    
    public function setVariationId($variationId)
    {
        $this->variationId = $variationId;
    }
    
    public function getSamples()
    {
        return $this->samples;
    }
    
    public function setSamples($samples)
    {
        $this->samples = $samples;
    }
    
    public function getVariance()
    {
        return $this->variance;
    }
    
    public function setVariance($variance)
    {
        $this->variance = $variance;
    }
}










1		<?php
2		/**
3		* @author Oleg Krivtsov <[email protected]>
4		* @date 12 October 2016
5		* @copyright (c) 2016, Web Marketing ROI
6		*/
7		namespace WebMarketingROI\OptimizelyPHP\Resource\v2;
8
9		use WebMarketingROI\OptimizelyPHP\Exception;
10		use WebMarketingROI\OptimizelyPHP\Resource\v2\Datapoint;
11
12		/**
13		* Optimizely campaign variant results.
14		*/
15		class VariantResults
16		{
17		/**
18		* The unique identifier for the Experiment this entity contains results for (if applicable).
19		* @var integer
20		*/
21		private $experimentId;
22
23		/**
24		* Indicates that this variant is the baseline that all other entities will
25		* be compared against. Also referred to as the 'Control' or 'Control Group'.
26		* @var boolean
27		*/
28		private $isBaseline;
29
30		/**
31		*
32		* @var type
33		*/
34		private $level;
35
36		/**
37		* The relative difference in performance of this variant vs. the baseline
38		* variant. Lift is calculated as follows: (Winning Conversion Rate % - Old
39		* Conversion Rate %) - Old Conversion Rate % = % Improvement
40		* @var Datapoint
41		*/
42		private $lift;
43
44		/**
45		* The name of the variant
46		* @var string
47		*/
48		private $name;
49
50		/**
51		*
52		* @var number
53		*/
54		private $rate;
55
56		/**
57		* The scope that this variant represents. Can be 'variation', 'experiment' or 'campaign'
58		* @var string
59		*/
60		private $scope;
61
62		/**
63		*
64		* @var Datapoint
65		*/
66		private $totalIncrease;
67
68		/**
69		*
70		* @var number
71		*/
72		private $value;
73
74		/**
75		*
76		* @var string
77		*/
78		private $variationId;
79
80		/**
81		*
82		* @var type
83		*/
84		private $samples;
85
86		/**
87		*
88		* @var type
89		*/
90		private $variance;
91
92		/**
93		* Constructor.
94		*/
95	3	public function __construct($options = array())
96		{
97	3	foreach ($options as $name=>$value) {
98		switch ($name) {
99	3	case 'experiment_id': $this->setExperimentId($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...
100	3	case 'is_baseline': $this->setIsBaseline($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...
101	3	case 'level': $this->setLevel($value); break;
		0 ignored issues – show Coding Style introduced 2018-05-14 05:48 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 2018-05-14 05:48 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...
102	3	case 'lift': $this->setLift(new Datapoint($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...
103	3	case 'name': $this->setName($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...
104	3	case 'rate': $this->setRate($value); break;
		0 ignored issues – show Coding Style introduced 2017-05-08 05:41 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 2017-05-08 05:41 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...
105	3	case 'scope': $this->setScope($value); break;
		0 ignored issues – show Coding Style introduced 2017-05-08 05:41 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 2017-05-08 05:41 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...
106	3	case 'total_increase': $this->setTotalIncrease(new Datapoint($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...
107	3	case 'value': $this->setValue($value); break;
		0 ignored issues – show Coding Style introduced 2017-05-08 05:41 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 2017-05-08 05:41 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...
108	3	case 'variation_id': $this->setVariationId($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...
109		case 'samples': $this->setSamples($value); break;
		0 ignored issues – show Coding Style introduced 2018-05-14 05:48 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 2018-05-14 05:48 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...
110		case 'variance': $this->setVariance($value); break;
		0 ignored issues – show Coding Style introduced 2018-05-14 05:48 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 2018-05-14 05:48 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...
111		default:
112	3	throw new Exception('Unknown option found in the VariantResults entity: ' . $name);
113		}
114		}
115	3	}
116
117		/**
118		* Returns this object as array.
119		*/
120	1	public function toArray()
121		{
122		$options = array(
123	1	'experiment_id' => $this->getExperimentId(),
124	1	'is_baseline' => $this->getIsBaseline(),
125	1	'level' => $this->getLevel(),
126	1	'lift' => $this->getLift()?$this->getLift()->toArray():null,
127	1	'name' => $this->getName(),
128	1	'rate' => $this->getRate(),
129	1	'scope' => $this->getScope(),
130	1	'total_increase' => $this->getTotalIncrease()?$this->getTotalIncrease()->toArray():null,
131	1	'value' => $this->getValue(),
132	1	'variation_id' => $this->getVariationId(),
133	1	'samples' => $this->getSamples(),
134	1	'variance' => $this->getVariance(),
135		);
136
137		// Remove options with empty values
138	1	$cleanedOptions = array();
139	1	foreach ($options as $name=>$value) {
140	1	if ($value!==null)
141	1	$cleanedOptions[$name] = $value;
142		}
143
144	1	return $cleanedOptions;
145		}
146
147	1	public function getExperimentId()
148		{
149	1	return $this->experimentId;
150		}
151
152	3	public function setExperimentId($experimentId)
153		{
154	3	$this->experimentId = $experimentId;
155	3	}
156
157	1	public function getIsBaseline()
158		{
159	1	return $this->isBaseline;
160		}
161
162	3	public function setIsBaseline($isBaseline)
163		{
164	3	$this->isBaseline = $isBaseline;
165	3	}
166
167	1	public function getLevel()
168		{
169	1	return $this->level;
170		}
171
172		public function setLevel($level)
173		{
174		$this->level = $level;
175		}
176
177	1	public function getLift()
178		{
179	1	return $this->lift;
180		}
181
182	3	public function setLift($lift)
183		{
184	3	$this->lift = $lift;
185	3	}
186
187	1	public function getName()
188		{
189	1	return $this->name;
190		}
191
192	3	public function setName($name)
193		{
194	3	$this->name = $name;
195	3	}
196
197	1	public function getRate()
198		{
199	1	return $this->rate;
200		}
201
202	3	public function setRate($rate)
203		{
204	3	$this->rate = $rate;
205	3	}
206
207	1	public function getScope()
208		{
209	1	return $this->scope;
210		}
211
212	3	public function setScope($scope)
213		{
214	3	$this->scope = $scope;
215	3	}
216
217	1	public function getTotalIncrease()
218		{
219	1	return $this->totalIncrease;
220		}
221
222	3	public function setTotalIncrease($totalIncrease)
223		{
224	3	$this->totalIncrease = $totalIncrease;
225	3	}
226
227	1	public function getValue()
228		{
229	1	return $this->value;
230		}
231
232	3	public function setValue($value)
233		{
234	3	$this->value = $value;
235	3	}
236
237	1	public function getVariationId()
238		{
239	1	return $this->variationId;
240		}
241
242	3	public function setVariationId($variationId)
243		{
244	3	$this->variationId = $variationId;
245	3	}
246
247	1	public function getSamples()
248		{
249	1	return $this->samples;
250		}
251
252		public function setSamples($samples)
253		{
254		$this->samples = $samples;
255		}
256
257	1	public function getVariance()
258		{
259	1	return $this->variance;
260		}
261
262		public function setVariance($variance)
263		{
264		$this->variance = $variance;
265		}
266		}
267
268
269
270
271
272
273
274
275

webmarketingroi / optimizely-php

Push — master ( 979a9d...8b1dd2 )

VariantResults::__construct() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like