ServerResponse::__construct() - Code Metrics - Inspection of "Let Scrutinizer only check the src folder, ignore..." - noplanman/php-telegram-bot - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — scrutinizer_fixes (e85bf7)

by Armando

created 2016-08-26 23:19 UTC

ServerResponse::__construct() C

↳ Parent: ServerResponse

Complexity

Conditions	25
Paths	21

Size

Total Lines	67
Code Lines	44

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	30
CRAP Score	37.0709

Importance

Changes	4
Bugs	0	Features	2

Metric	Value
c	4
b	0
f	2
dl	0
loc	67
ccs	30
cts	41
cp	0.7317
rs	5.6674
cc	25
eloc	44
nc	21
nop	2
crap	37.0709

How to fix Long Method Complexity

<?php
/**
 * This file is part of the TelegramBot package.
 * (c) Avtandil Kikabidze aka LONGMAN <[email protected]>
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Longman\TelegramBot\Entities;

class ServerResponse extends Entity
{
    /**
     * @var bool
     */
    protected $ok;

    /**
     * @var null
     */
    protected $result;

    /**
     * @var null
     */
    protected $error_code;

    /**
     * @var null
     */
    protected $description;

    /**
     * ServerResponse constructor.
     *
     * @param array $data
     * @param       $bot_name
     *
     * @throws \Longman\TelegramBot\Exception\TelegramException
     */
    public function __construct(array $data, $bot_name)
    {
        if (isset($data['ok'], $data['result'])) {
            if (is_array($data['result'])) {
                if ($data['ok'] && !$this->isAssoc($data['result']) && !isset($data['result'][0]['user'])) {
                    //Get Update
                    foreach ($data['result'] as $update) {
                        $this->result[] = new Update($update, $bot_name);
                    }
                } elseif ($data['ok'] && !$this->isAssoc($data['result']) && isset($data['result'][0]['user'])) {
                    //Response from getChatAdministrators
                    $this->result = [];

                    foreach ($data['result'] as $user) {
                        $this->result[] = new ChatMember($user);
                    }
                } elseif ($data['ok'] && $this->isAssoc($data['result'])) {
                    if (isset($data['result']['total_count'])) {
                        //Response from getUserProfilePhotos
                        $this->result = new UserProfilePhotos($data['result']);

                    } elseif (isset($data['result']['file_id'])) {
                        //Response from getFile
                        $this->result = new File($data['result']);

                    } elseif (isset($data['result']['username'])) {
                        //Response from getMe
                        $this->result = new User($data['result']);

                    } elseif (isset($data['result']['id'])) {
                        //Response from getChat
                        $this->result = new Chat($data['result']);

                    } elseif (isset($data['result']['user'])) {
                        //Response from getChatMember
                        $this->result = new ChatMember($data['result']);

                    } else {
                        //Response from sendMessage
                        $this->result = new Message($data['result'], $bot_name);

                    }
                }

                $this->ok          = $data['ok'];
                $this->error_code  = null;
                $this->description = null;
            } else {
                if ($data['ok'] && $data['result'] === true) {
                    //Response from setWebhook set
                    $this->ok          = $data['ok'];
                    $this->result      = true;

                    $this->error_code  = null;
                    $this->description = isset($data['description']) ? $data['description'] : '';
                } elseif (is_numeric($data['result'])) {
                    //Response from getChatMembersCount
                    $this->result = $data['result'];

                } else {
                    $this->ok          = false;
                    $this->result      = null;
                    $this->error_code  = $data['error_code'];
                    $this->description = $data['description'];
                }
            }
        } else {
            //webHook not set
            $this->ok          = false;
            $this->result      = isset($data['result']) ? $data['result'] : null;
            $this->error_code  = isset($data['error_code']) ? $data['error_code'] : null;
            $this->description = isset($data['description']) ? $data['description'] : null;

            //throw new TelegramException('ok(variable) is not set!');

        }
    }

    /**
     * Check if array is associative
     *
     * @param array $array
     * @return bool
     */
    protected function isAssoc(array $array)
    {
        return (bool) count(array_filter(array_keys($array), 'is_string'));
    }

    /**
     * If response is ok
     *
     * @return bool
     */
    public function isOk()
    {
        return $this->ok;
    }

    /**
     * Get result
     *
     * @return string
     */
    public function getResult()
    {
        return $this->result;
    }

    /**
     * Get error code
     *
     * @return string
     */
    public function getErrorCode()
    {
        return $this->error_code;
    }

    /**
     * Get description
     *
     * @return null
     */
    public function getDescription()
    {
        return $this->description;
    }

    /**
     * Print error
     *
     * @return string
     */
    public function printError()
    {
        return 'Error N: ' . $this->getErrorCode() . ' Description: ' . $this->getDescription();
    }
}


1		<?php
2		/**
3		* This file is part of the TelegramBot package.
4		* (c) Avtandil Kikabidze aka LONGMAN <[email protected]>
5		* For the full copyright and license information, please view the LICENSE
6		* file that was distributed with this source code.
7		*/
8
9		namespace Longman\TelegramBot\Entities;
10
11		class ServerResponse extends Entity
12		{
13		/**
14		* @var bool
15		*/
16		protected $ok;
17
18		/**
19		* @var null
20		*/
21		protected $result;
22
23		/**
24		* @var null
25		*/
26		protected $error_code;
27
28		/**
29		* @var null
30		*/
31		protected $description;
32
33		/**
34		* ServerResponse constructor.
35		*
36		* @param array $data
37		* @param $bot_name
38		*
39		* @throws \Longman\TelegramBot\Exception\TelegramException
40		*/
41	14	public function __construct(array $data, $bot_name)
42		{
43	14	if (isset($data['ok'], $data['result'])) {
44	12	if (is_array($data['result'])) {
45	11	if ($data['ok'] && !$this->isAssoc($data['result']) && !isset($data['result'][0]['user'])) {
46		//Get Update
47	2	foreach ($data['result'] as $update) {
48	2	$this->result[] = new Update($update, $bot_name);
49		}
50	9	} elseif ($data['ok'] && !$this->isAssoc($data['result']) && isset($data['result'][0]['user'])) {
51		//Response from getChatAdministrators
52		$this->result = [];
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `array()` of type `array` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
53		foreach ($data['result'] as $user) {
54		$this->result[] = new ChatMember($user);
55		}
56	9	} elseif ($data['ok'] && $this->isAssoc($data['result'])) {
57	9	if (isset($data['result']['total_count'])) {
58		//Response from getUserProfilePhotos
59	1	$this->result = new UserProfilePhotos($data['result']);
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `new \Longman\TelegramBot...Photos($data['result'])` of type `object<Longman\TelegramB...ties\UserProfilePhotos>` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
60	8	} elseif (isset($data['result']['file_id'])) {
61		//Response from getFile
62	1	$this->result = new File($data['result']);
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `new \Longman\TelegramBot...s\File($data['result'])` of type `object<Longman\TelegramBot\Entities\File>` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
63	7	} elseif (isset($data['result']['username'])) {
64		//Response from getMe
65		$this->result = new User($data['result']);
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `new \Longman\TelegramBot...s\User($data['result'])` of type `object<Longman\TelegramBot\Entities\User>` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
66	7	} elseif (isset($data['result']['id'])) {
67		//Response from getChat
68		$this->result = new Chat($data['result']);
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `new \Longman\TelegramBot...s\Chat($data['result'])` of type `object<Longman\TelegramBot\Entities\Chat>` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
69	7	} elseif (isset($data['result']['user'])) {
70		//Response from getChatMember
71		$this->result = new ChatMember($data['result']);
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `new \Longman\TelegramBot...Member($data['result'])` of type `object<Longman\TelegramBot\Entities\ChatMember>` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
72		} else {
73		//Response from sendMessage
74	7	$this->result = new Message($data['result'], $bot_name);
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `new \Longman\TelegramBot...a['result'], $bot_name)` of type `object<Longman\TelegramBot\Entities\Message>` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
75		}
76		}
77
78	11	$this->ok = $data['ok'];
79	11	$this->error_code = null;
80	11	$this->description = null;
81		} else {
82	2	if ($data['ok'] && $data['result'] === true) {
83		//Response from setWebhook set
84	2	$this->ok = $data['ok'];
85	2	$this->result = true;
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `true` of type `boolean` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
86	2	$this->error_code = null;
87	2	$this->description = isset($data['description']) ? $data['description'] : '';
88		} elseif (is_numeric($data['result'])) {
89		//Response from getChatMembersCount
90		$this->result = $data['result'];
		0 ignored issues – show Documentation Bug introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report It seems like `$data['result']` of type `integer` or `double` or `string` is incompatible with the declared type `null` of property `$result`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
91		} else {
92		$this->ok = false;
93		$this->result = null;
94		$this->error_code = $data['error_code'];
95	12	$this->description = $data['description'];
96		}
97		}
98		} else {
99		//webHook not set
100	2	$this->ok = false;
101	2	$this->result = isset($data['result']) ? $data['result'] : null;
102	2	$this->error_code = isset($data['error_code']) ? $data['error_code'] : null;
103	2	$this->description = isset($data['description']) ? $data['description'] : null;
104
105		//throw new TelegramException('ok(variable) is not set!');
		1 ignored issue – show Unused Code Comprehensibility introduced 2016-08-26 21:13 UTC by Report Bug Copy Issue Report `67%` of this comment could be valid code. Did you maybe forget this after debugging? Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it. The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production. This check looks for comments that seem to be mostly valid code and reports them. Loading history...
106		}
107	14	}
108
109		/**
110		* Check if array is associative
111		*
112		* @param array $array
113		* @return bool
114		*/
115	11	protected function isAssoc(array $array)
116		{
117	11	return (bool) count(array_filter(array_keys($array), 'is_string'));
118		}
119
120		/**
121		* If response is ok
122		*
123		* @return bool
124		*/
125	5	public function isOk()
126		{
127	5	return $this->ok;
128		}
129
130		/**
131		* Get result
132		*
133		* @return string
134		*/
135	14	public function getResult()
136		{
137	14	return $this->result;
138		}
139
140		/**
141		* Get error code
142		*
143		* @return string
144		*/
145	5	public function getErrorCode()
146		{
147	5	return $this->error_code;
148		}
149
150		/**
151		* Get description
152		*
153		* @return null
154		*/
155	5	public function getDescription()
156		{
157	5	return $this->description;
158		}
159
160		/**
161		* Print error
162		*
163		* @return string
164		*/
165		public function printError()
166		{
167		return 'Error N: ' . $this->getErrorCode() . ' Description: ' . $this->getDescription();
168		}
169		}
170

noplanman / php-telegram-bot

Branch — scrutinizer_fixes (e85bf7)

ServerResponse::__construct() C

Complexity

Size

Duplication

Code Coverage

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like