TotalResponseItem::create() - Code Metrics - Inspection of "Added "complaints" to TotalResponse (#509)" - mailgun/mailgun-php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 02e319...091134 )

by Tobias

created 2019-01-05 18:56 UTC

TotalResponseItem::create() A

↳ Parent: TotalResponseItem

Complexity

Conditions	6
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	8
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	10
ccs	8
cts	8
cp	1
rs	9.2222
c	0
b	0
f	0
cc	6
nc	1
nop	1
crap	6

<?php

/*
 * Copyright (C) 2013 Mailgun
 *
 * This software may be modified and distributed under the terms
 * of the MIT license. See the LICENSE file for details.
 */

namespace Mailgun\Model\Stats;

/**
 * @author Tobias Nyholm <[email protected]>
 */
class TotalResponseItem
{
    /**
     * @var \DateTime
     */
    private $time;

    /**
     * @var array
     */
    private $accepted;

    /**
     * @var array
     */
    private $delivered;

    /**
     * @var array
     */
    private $failed;

    /**
     * @var array
     */
    private $complained;

    /**
     * @param array $data
     *
     * @return self
     */
    public static function create(array $data)
    {
        return new self(
            isset($data['time']) ? new \DateTime($data['time']) : null,
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
            isset($data['accepted']) ? $data['accepted'] : [],
            isset($data['delivered']) ? $data['delivered'] : [],
            isset($data['failed']) ? $data['failed'] : [],
            isset($data['complained']) ? $data['complained'] : []
        );
    }

    /**
     * @param \DateTime $time
     * @param array     $accepted
     * @param array     $delivered
     * @param array     $failed
     * @param array     $complained
     */
    private function __construct(\DateTime $time, array $accepted, array $delivered, array $failed, array $complained)
    {
        $this->time = $time;
        $this->accepted = $accepted;
        $this->delivered = $delivered;
        $this->failed = $failed;
        $this->complained = $complained;
    }

    /**
     * @return \DateTime
     */
    public function getTime()
    {
        return $this->time;
    }

    /**
     * @return array
     */
    public function getAccepted()
    {
        return $this->accepted;
    }

    /**
     * @return array
     */
    public function getDelivered()
    {
        return $this->delivered;
    }

    /**
     * @return array
     */
    public function getFailed()
    {
        return $this->failed;
    }

    /**
     * @return array
     */
    public function getComplained()
    {
        return $this->complained;
    }
}


1		<?php
2
3		/*
4		* Copyright (C) 2013 Mailgun
5		*
6		* This software may be modified and distributed under the terms
7		* of the MIT license. See the LICENSE file for details.
8		*/
9
10		namespace Mailgun\Model\Stats;
11
12		/**
13		* @author Tobias Nyholm <[email protected]>
14		*/
15		class TotalResponseItem
16		{
17		/**
18		* @var \DateTime
19		*/
20		private $time;
21
22		/**
23		* @var array
24		*/
25		private $accepted;
26
27		/**
28		* @var array
29		*/
30		private $delivered;
31
32		/**
33		* @var array
34		*/
35		private $failed;
36
37		/**
38		* @var array
39		*/
40		private $complained;
41
42		/**
43		* @param array $data
44		*
45		* @return self
46		*/
47	4	public static function create(array $data)
48		{
49	4	return new self(
50	4	isset($data['time']) ? new \DateTime($data['time']) : null,
		0 ignored issues – show Bug introduced 2016-12-06 06:54 UTC by Report Bug Copy Issue Report It seems like `isset($data['time']) ? n...e($data['time']) : null` can be `null`; however, `__construct()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
51	4	isset($data['accepted']) ? $data['accepted'] : [],
52	4	isset($data['delivered']) ? $data['delivered'] : [],
53	4	isset($data['failed']) ? $data['failed'] : [],
54	4	isset($data['complained']) ? $data['complained'] : []
55	4	);
56		}
57
58		/**
59		* @param \DateTime $time
60		* @param array $accepted
61		* @param array $delivered
62		* @param array $failed
63		* @param array $complained
64		*/
65	4	private function __construct(\DateTime $time, array $accepted, array $delivered, array $failed, array $complained)
66		{
67	4	$this->time = $time;
68	4	$this->accepted = $accepted;
69	4	$this->delivered = $delivered;
70	4	$this->failed = $failed;
71	4	$this->complained = $complained;
72	4	}
73
74		/**
75		* @return \DateTime
76		*/
77	1	public function getTime()
78		{
79	1	return $this->time;
80		}
81
82		/**
83		* @return array
84		*/
85		public function getAccepted()
86		{
87		return $this->accepted;
88		}
89
90		/**
91		* @return array
92		*/
93		public function getDelivered()
94		{
95		return $this->delivered;
96		}
97
98		/**
99		* @return array
100		*/
101		public function getFailed()
102		{
103		return $this->failed;
104		}
105
106		/**
107		* @return array
108		*/
109	1	public function getComplained()
110		{
111	1	return $this->complained;
112		}
113		}
114

mailgun / mailgun-php

Push — master ( 02e319...091134 )

TotalResponseItem::create() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like