Datapoint::getValue() - Code Metrics - Inspection of "Fixing unit tests" - webmarketingroi/optimizely-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 4d3008...03e605 )

by Oleg

created 2017-12-13 09:59 UTC

Datapoint::getValue() A

↳ Parent: Datapoint

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	2
CRAP Score	1

Importance

Changes

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

<?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;

/**
 * Optimizely datapoint.
 */
class Datapoint
{
    /**
     * The confidence interval measures the uncertainty around improvement. It 
     * starts out wide and shrinks as more data comes in. Significance means 
     * that the confidence interval is completely above or completely below 0. 
     * If the result is significant and positive, the confidence interval will 
     * be above 0. If the result is significant and negative, confidence interval 
     * will be below 0. If the result is inconclusive, confidence interval includes 0.
     * @var array[number]
     */
    private $confidenceInterval;
    
    /**
     * Indicates that this is the best performing variant for this metric. 
     * Also referred to as the 'Winner'
     * @var boolean 
     */
    private $isMostConclusive;
    
    /**
     * Indicates if significance is above your confidence threshold
     * @var boolean
     */
    private $isSignificant;
    
    /**
     * The likelihood that the observed difference in conversion rate is not due to chance.
     * @var number 
     */
    private $significance;
    
    /**
     * The relative improvement for this variant over the baseline variant.
     * @var number
     */
    private $value;
    
    /**
     * The number of estimated visitors remaining before result becomes statistically significant
     * @var integer 
     */
    private $visitorsRemaining;
    /**
     * Constructor.
     */
    public function __construct($options = array())
    {
        foreach ($options as $name=>$value) {
            switch ($name) {                
                case 'confidence_interval': $this->setConfidenceInterval($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

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

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

    doSomethingElse(); //wrong
    break;
                case 'significance': $this->setSignificance($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 'visitors_remaining': $this->setVisitorsRemaining($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(
            'confidence_interval' => $this->getConfidenceInterval(),
            'is_most_conclusive' => $this->getIsMostConclusive(),
            'is_significant' => $this->getIsSignificant(),
            'significance' => $this->getSignificance(),
            'value' => $this->getValue(),            
            'visitors_remaining' => $this->getVisitorsRemaining(),            
        );
        
        // Remove options with empty values
        $cleanedOptions = array();
        foreach ($options as $name=>$value) {
            if ($value!==null)
                $cleanedOptions[$name] = $value;
        }
        
        return $cleanedOptions;
    }
    
    public function getConfidenceInterval()
    {
        return $this->confidenceInterval;
    }
    
    public function setConfidenceInterval($confidenceInterval)
    {
        $this->confidenceInterval = $confidenceInterval;
    }
    
    public function getIsMostConclusive()
    {
        return $this->isMostConclusive;
    }
    
    public function setIsMostConclusive($isMostConclusive)
    {
        $this->isMostConclusive = $isMostConclusive;
    }
    
    public function getIsSignificant()
    {
        return $this->isSignificant;
    }
    
    public function setIsSignificant($isSignificant)
    {
        $this->isSignificant = $isSignificant;
    }
    
    public function getSignificance()
    {
        return $this->significance;
    }
    
    public function setSignificance($significance)
    {
        $this->significance = $significance;
    }
    
    public function getValue()
    {
        return $this->value;
    }
    
    public function setValue($value)
    {
        $this->value = $value;
    }
    
    public function getVisitorsRemaining()
    {
        return $this->visitorsRemaining;
    }
    
    public function setVisitorsRemaining($visitorsRemaining)
    {
        $this->visitorsRemaining = $visitorsRemaining;
    }
}










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
11		/**
12		* Optimizely datapoint.
13		*/
14		class Datapoint
15		{
16		/**
17		* The confidence interval measures the uncertainty around improvement. It
18		* starts out wide and shrinks as more data comes in. Significance means
19		* that the confidence interval is completely above or completely below 0.
20		* If the result is significant and positive, the confidence interval will
21		* be above 0. If the result is significant and negative, confidence interval
22		* will be below 0. If the result is inconclusive, confidence interval includes 0.
23		* @var array[number]
24		*/
25		private $confidenceInterval;
26
27		/**
28		* Indicates that this is the best performing variant for this metric.
29		* Also referred to as the 'Winner'
30		* @var boolean
31		*/
32		private $isMostConclusive;
33
34		/**
35		* Indicates if significance is above your confidence threshold
36		* @var boolean
37		*/
38		private $isSignificant;
39
40		/**
41		* The likelihood that the observed difference in conversion rate is not due to chance.
42		* @var number
43		*/
44		private $significance;
45
46		/**
47		* The relative improvement for this variant over the baseline variant.
48		* @var number
49		*/
50		private $value;
51
52		/**
53		* The number of estimated visitors remaining before result becomes statistically significant
54		* @var integer
55		*/
56		private $visitorsRemaining;
57		/**
58		* Constructor.
59		*/
60	3	public function __construct($options = array())
61		{
62	3	foreach ($options as $name=>$value) {
63		switch ($name) {
64	3	case 'confidence_interval': $this->setConfidenceInterval($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...
65	3	case 'is_most_conclusive': $this->setIsMostConclusive($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...
66	3	case 'is_significant': $this->setIsSignificant($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...
67	3	case 'significance': $this->setSignificance($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...
68	3	case 'value': $this->setValue($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...
69	3	case 'visitors_remaining': $this->setVisitorsRemaining($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...
70		default:
71		throw new Exception('Unknown option: ' . $name);
72		}
73	3	}
74	3	}
75
76		/**
77		* Returns this object as array.
78		*/
79	1	public function toArray()
80		{
81		$options = array(
82	1	'confidence_interval' => $this->getConfidenceInterval(),
83	1	'is_most_conclusive' => $this->getIsMostConclusive(),
84	1	'is_significant' => $this->getIsSignificant(),
85	1	'significance' => $this->getSignificance(),
86	1	'value' => $this->getValue(),
87	1	'visitors_remaining' => $this->getVisitorsRemaining(),
88	1	);
89
90		// Remove options with empty values
91	1	$cleanedOptions = array();
92	1	foreach ($options as $name=>$value) {
93	1	if ($value!==null)
94	1	$cleanedOptions[$name] = $value;
95	1	}
96
97	1	return $cleanedOptions;
98		}
99
100	1	public function getConfidenceInterval()
101		{
102	1	return $this->confidenceInterval;
103		}
104
105	3	public function setConfidenceInterval($confidenceInterval)
106		{
107	3	$this->confidenceInterval = $confidenceInterval;
108	3	}
109
110	1	public function getIsMostConclusive()
111		{
112	1	return $this->isMostConclusive;
113		}
114
115	3	public function setIsMostConclusive($isMostConclusive)
116		{
117	3	$this->isMostConclusive = $isMostConclusive;
118	3	}
119
120	1	public function getIsSignificant()
121		{
122	1	return $this->isSignificant;
123		}
124
125	3	public function setIsSignificant($isSignificant)
126		{
127	3	$this->isSignificant = $isSignificant;
128	3	}
129
130	1	public function getSignificance()
131		{
132	1	return $this->significance;
133		}
134
135	3	public function setSignificance($significance)
136		{
137	3	$this->significance = $significance;
138	3	}
139
140	1	public function getValue()
141		{
142	1	return $this->value;
143		}
144
145	3	public function setValue($value)
146		{
147	3	$this->value = $value;
148	3	}
149
150	1	public function getVisitorsRemaining()
151		{
152	1	return $this->visitorsRemaining;
153		}
154
155	3	public function setVisitorsRemaining($visitorsRemaining)
156		{
157	3	$this->visitorsRemaining = $visitorsRemaining;
158	3	}
159		}
160
161
162
163
164
165
166
167
168

webmarketingroi / optimizely-php

Push — master ( 4d3008...03e605 )

Datapoint::getValue() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like