Event::__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 ( 0ca99d...7c70a3 )

by Oleg

created 2016-10-19 06:09 UTC

Event::__construct() C

↳ Parent: Event

Complexity

Conditions	16
Paths	16

Size

Total Lines	23
Code Lines	19

Duplication

Lines	23
Ratio	100 %

Code Coverage

Tests	18
CRAP Score	16.747

Importance

Changes

Metric	Value
cc	16
eloc	19
nc	16
nop	1
dl	23
loc	23
ccs	18
cts	21
cp	0.8571
crap	16.747
rs	5.2678
c	0
b	0
f	0

How to fix Complexity

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

use WebMarketingROI\OptimizelyPHP\Resource\v2\EventFilter;

/**
 * An Optimizely event.
 */
class Event
{
    /**
     * A machine readable name for this Event
     * @var string
     */
    private $apiName;
    
    /**
     * Whether or not this Event has been archived
     * @var boolean 
     */
    private $archived;
    
    /**
     * A category for this Event. Can be 'add_to_cart', 'save', 'search', 'share', 
     * 'purchase', 'convert', 'sign_up', 'subscribe' or 'other'.
     * @var type 
     */
    private $category;
    
    /**
     * Creation date for this Event
     * @var string 
     */
    private $created;
    
    /**
     * A description for this Event
     * @var string 
     */
    private $description;
    
    /**
     * A filter object for this Event
     * @var EventFilter 
     */
    private $eventFilter;
    
    /**
     * The type of this Event. Can be custom, click, pageview, classic_custom, 
     * classic_click, classic_pageview, classic_engagement, classic_revenue, 
     * classic_mobile_tap, classic_mobile_view, classic_mobile_session, 
     * classic_mobile_session_length or classic_mobile_num_session
     * @var string 
     */
    private $eventType;
    
    /**
     * Unique string identifier for this event within the project.
     * @var string
     */
    private $key;
    
    /**
     * A human readable name for this Event
     * @var string 
     */
    private $name;
    
    /**
     * The Page ID associated with this Event
     * @var integer 
     */
    private $pageId;
    
    /**
     * The parent Project ID of this Event
     * @var integer 
     */
    private $projectId;
    
    /**
     * The unique identifier of the Event
     * @var integer
     */
    private $id;
    
    /**
     * Whether or not this Event is a classic Event
     * @var boolean 
     */
    private $isClassic;
    
    /**
     * Whether this Event may be edited
     * @var boolean 
     */
    private $isEditable;
    
    /**
     * Constructor.
     */
    public function __construct($options = array())

    {
        foreach ($options as $name=>$value) {
            switch ($name) {                
                case 'api_name': $this->setApiName($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

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

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

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

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

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

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

    doSomethingElse(); //wrong
    break;
                case 'key': $this->setKey($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 'page_id': $this->setPageId($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                case 'project_id': $this->setProjectId($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;
                case 'is_classic': $this->setIsClassic($value); break;
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
                case 'is_editable': $this->setIsEditable($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(
            'api_name' => $this->getApiName(),
            'archived' => $this->getArchived(),
            'category' => $this->getCategory(),
            'created' => $this->getCreated(),
            'description' => $this->getDescription(),
            'event_filter' => $this->getEventFilter()?$this->getEventFilter()->toArray():null,
            'event_type' => $this->getEventType(),
            'key' => $this->getKey(),
            'name' => $this->getName(),
            'page_id' => $this->getPageId(),
            'project_id' => $this->getProjectId(),
            'id' => $this->getId(),
            'is_classic' => $this->getIsClassic(),
            'is_editable' => $this->getIsEditable()
        );
        
        // Remove options with empty values
        $cleanedOptions = array();
        foreach ($options as $name=>$value) {
            if ($value!==null)
                $cleanedOptions[$name] = $value;
        }
        
        return $cleanedOptions;
    }
    
    public function getApiName()
    {
        return $this->apiName;
    }
    
    public function setApiName($apiName)
    {
        $this->apiName = $apiName;
    }
    
    public function getArchived()
    {
        return $this->archived;
    }
    
    public function setArchived($archived)
    {
        $this->archived = $archived;
    }
    
    public function getCategory()
    {
        return $this->category;
    }
    
    public function setCategory($category)
    {
        $this->category = $category;
    }
    
    public function getCreated()
    {
        return $this->created;
    }
    
    public function setCreated($created)
    {
        $this->created = $created;
    }
        
    public function getDescription()
    {
        return $this->description;
    }
    
    public function setDescription($description)
    {
        $this->description = $description;
    }
        
    public function getEventFilter()
    {
        return $this->eventFilter;
    }
    
    public function setEventFilter($eventFilter)
    {
        $this->eventFilter = $eventFilter;
    }
    
    public function getEventType()
    {
        return $this->eventType;
    }
    
    public function setEventType($eventType)
    {
        $this->eventType = $eventType;
    }
    
    public function getKey()
    {
        return $this->key;
    }
    
    public function setKey($key)
    {
        $this->key = $key;
    }
    
    public function getName()
    {
        return $this->name;
    }
    
    public function setName($name)
    {
        $this->name = $name;
    }
    
    public function getPageId()
    {
        return $this->pageId;
    }
    
    public function setPageId($pageId)
    {
        $this->pageId = $pageId;
    }
    
    public function getProjectId()
    {
        return $this->projectId;
    }
    
    public function setProjectId($projectId)
    {
        $this->projectId = $projectId;
    }
    
    public function getId()
    {
        return $this->id;
    }
    
    public function setId($id)
    {
        $this->id = $id;
    }
    
    public function getIsClassic()
    {
        return $this->isClassic;
    }
    
    public function setIsClassic($isClassic)
    {
        $this->isClassic = $isClassic;
    }
    
    public function getIsEditable()
    {
        return $this->isEditable;
    }
    
    public function setIsEditable($isEditable)
    {
        $this->isEditable = $isEditable;
    }
}








1			<?php
2			/**
3			* @author Oleg Krivtsov <[email protected]>
4			* @date 07 October 2016
5			* @copyright (c) 2016, Web Marketing ROI
6			*/
7			namespace WebMarketingROI\OptimizelyPHP\Resource\v2;
8
9			use WebMarketingROI\OptimizelyPHP\Resource\v2\EventFilter;
10
11			/**
12			* An Optimizely event.
13			*/
14			class Event
15			{
16			/**
17			* A machine readable name for this Event
18			* @var string
19			*/
20			private $apiName;
21
22			/**
23			* Whether or not this Event has been archived
24			* @var boolean
25			*/
26			private $archived;
27
28			/**
29			* A category for this Event. Can be 'add_to_cart', 'save', 'search', 'share',
30			* 'purchase', 'convert', 'sign_up', 'subscribe' or 'other'.
31			* @var type
32			*/
33			private $category;
34
35			/**
36			* Creation date for this Event
37			* @var string
38			*/
39			private $created;
40

webmarketingroi / optimizely-php

Push — master ( 0ca99d...7c70a3 )

Event::__construct() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like