Error::getData() - Code Metrics - duoshuo/php-cassandra - Measure and Improve Code Quality continuously with Scrutinizer

Error::getData() D
last analyzed 2015-12-16 12:25 UTC

↳ Parent: Error

Complexity

Conditions	16
Paths	16

Size

Total Lines	89
Code Lines	70

Duplication

Lines	26
Ratio	29.21 %

Importance

Changes	6
Bugs	2	Features	3

Metric	Value
c	6
b	2
f	3
dl	26
loc	89
rs	4.8737
cc	16
eloc	70
nc	16
nop	0

How to fix Long Method Complexity

<?php
namespace Cassandra\Response;
use Cassandra\Protocol\Frame;

class Error extends Response {
	const SERVER_ERROR = 0x0000;
	const PROTOCOL_ERROR = 0x000A;
	const BAD_CREDENTIALS = 0x0100;
	const UNAVAILABLE_EXCEPTION = 0x1000;
	const OVERLOADED = 0x1001;
	const IS_BOOTSTRAPPING = 0x1002;
	const TRUNCATE_ERROR = 0x1003;
	const WRITE_TIMEOUT = 0x1100;
	const READ_TIMEOUT = 0x1200;
	const SYNTAX_ERROR = 0x2000;
	const UNAUTHORIZED = 0x2100;
	const INVALID = 0x2200;
	const CONFIG_ERROR = 0x2300;
	const ALREADY_EXIST = 0x2400;
	const UNPREPARED = 0x2500;
	
	/**
	 * Indicates an error processing a request. The body of the message will be an
	 * error code ([int]) followed by a [string] error message. Then, depending on
	 * the exception, more content may follow. The error codes are defined in
	 * Section 7, along with their additional content if any.
	 *
	 * @return array
	 */
	public function getData() {
		$this->_stream->offset(0);
		$data = [];
		$data['code'] = $this->_stream->readInt();

		switch($data['code']){
			case self::SERVER_ERROR:
				$data['message'] = "Server error: " . $this->_stream->readString();
				break;

			case self::PROTOCOL_ERROR:
				$data['message'] = "Protocol error: " . $this->_stream->readString();
				break;

			case self::BAD_CREDENTIALS:
				$data['message'] = "Bad credentials: " . $this->_stream->readString();
				break;

			case self::UNAVAILABLE_EXCEPTION:

				$data['message'] = "Unavailable exception. Error data: " . var_export([
                        'error'=>$this->_stream->readString(),
                        'consistency' => $this->_stream->readShort(),
                        'node' => $this->_stream->readInt(),
						'replica' => $this->_stream->readInt()
					], true);
				break;

			case self::OVERLOADED:
				$data['message'] = "Overloaded: " . $this->_stream->readString();
				break;

			case self::IS_BOOTSTRAPPING:
				$data['message'] = "Is_bootstrapping: " . $this->_stream->readString();
				break;

			case self::TRUNCATE_ERROR:
				$data['message'] = "Truncate_error: " . $this->_stream->readString();
				break;

			case self::WRITE_TIMEOUT:

				$data['message'] = "Write_timeout. Error data: " . var_export([
                        'error'=>$this->_stream->readString(),
                        'consistency' => $this->_stream->readShort(),
						'node' => $this->_stream->readInt(),
						'replica' => $this->_stream->readInt(),
						'writeType' => $this->_stream->readString()
					], true);
				break;

			case self::READ_TIMEOUT:

				$data['message'] = "Read_timeout. Error data: " . var_export([
                        'error'=>$this->_stream->readString(),
                        'consistency' => $this->_stream->readShort(),
						'node' => $this->_stream->readInt(),
						'replica' => $this->_stream->readInt(),
						'dataPresent' => $this->_stream->readChar()
					], true);
				break;

			case self::SYNTAX_ERROR:
				$data['message'] = "Syntax_error: " . $this->_stream->readString();
				break;

			case self::UNAUTHORIZED:
				$data['message'] = "Unauthorized: " . $this->_stream->readString();
				break;

			case self::INVALID:
				$data['message'] = "Invalid: " . $this->_stream->readString();
				break;

			case self::CONFIG_ERROR:
				$data['message'] = "Config_error: " . $this->_stream->readString();
				break;

			case self::ALREADY_EXIST:
				$data['message'] = "Already_exists: " . $this->_stream->readString();
				break;

			case self::UNPREPARED:
				$data['message'] = "Unprepared: " . $this->_stream->readShort();
				break;

			default:
				$data['message'] = 'Unknown error';
		}
		
		return $data;
	}
	
	/**
	 * 
	 * @return Exception
	 */
	public function getException(){
		$data = $this->getData();
		return new Exception($data['message'], $data['code']);
	}
}


1		<?php
2		namespace Cassandra\Response;
3		use Cassandra\Protocol\Frame;
4
5		class Error extends Response {
6		const SERVER_ERROR = 0x0000;
7		const PROTOCOL_ERROR = 0x000A;
8		const BAD_CREDENTIALS = 0x0100;
9		const UNAVAILABLE_EXCEPTION = 0x1000;
10		const OVERLOADED = 0x1001;
11		const IS_BOOTSTRAPPING = 0x1002;
12		const TRUNCATE_ERROR = 0x1003;
13		const WRITE_TIMEOUT = 0x1100;
14		const READ_TIMEOUT = 0x1200;
15		const SYNTAX_ERROR = 0x2000;
16		const UNAUTHORIZED = 0x2100;
17		const INVALID = 0x2200;
18		const CONFIG_ERROR = 0x2300;
19		const ALREADY_EXIST = 0x2400;
20		const UNPREPARED = 0x2500;
21
22		/**
23		* Indicates an error processing a request. The body of the message will be an
24		* error code ([int]) followed by a [string] error message. Then, depending on
25		* the exception, more content may follow. The error codes are defined in
26		* Section 7, along with their additional content if any.
27		*
28		* @return array
29		*/
30		public function getData() {
31		$this->_stream->offset(0);
32		$data = [];
33		$data['code'] = $this->_stream->readInt();
34
35		switch($data['code']){
36		case self::SERVER_ERROR:
37		$data['message'] = "Server error: " . $this->_stream->readString();
38		break;
39
40		case self::PROTOCOL_ERROR:
41		$data['message'] = "Protocol error: " . $this->_stream->readString();
42		break;
43
44		case self::BAD_CREDENTIALS:
45		$data['message'] = "Bad credentials: " . $this->_stream->readString();
46		break;
47
48	View Code Duplication	case self::UNAVAILABLE_EXCEPTION:
		0 ignored issues – show Duplication introduced 2015-12-15 08:58 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
49		$data['message'] = "Unavailable exception. Error data: " . var_export([
50		'error'=>$this->_stream->readString(),
51		'consistency' => $this->_stream->readShort(),
52		'node' => $this->_stream->readInt(),
53		'replica' => $this->_stream->readInt()
54		], true);
55		break;
56
57		case self::OVERLOADED:
58		$data['message'] = "Overloaded: " . $this->_stream->readString();
59		break;
60
61		case self::IS_BOOTSTRAPPING:
62		$data['message'] = "Is_bootstrapping: " . $this->_stream->readString();
63		break;
64
65		case self::TRUNCATE_ERROR:
66		$data['message'] = "Truncate_error: " . $this->_stream->readString();
67		break;
68
69	View Code Duplication	case self::WRITE_TIMEOUT:
		0 ignored issues – show Duplication introduced 2015-12-04 15:29 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
70		$data['message'] = "Write_timeout. Error data: " . var_export([
71		'error'=>$this->_stream->readString(),
72		'consistency' => $this->_stream->readShort(),
73		'node' => $this->_stream->readInt(),
74		'replica' => $this->_stream->readInt(),
75		'writeType' => $this->_stream->readString()
76		], true);
77		break;
78
79	View Code Duplication	case self::READ_TIMEOUT:
		0 ignored issues – show Duplication introduced 2015-12-04 15:29 UTC by Report Bug Copy Issue Report This code seems to be duplicated across your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history...
80		$data['message'] = "Read_timeout. Error data: " . var_export([
81		'error'=>$this->_stream->readString(),
82		'consistency' => $this->_stream->readShort(),
83		'node' => $this->_stream->readInt(),
84		'replica' => $this->_stream->readInt(),
85		'dataPresent' => $this->_stream->readChar()
86		], true);
87		break;
88
89		case self::SYNTAX_ERROR:
90		$data['message'] = "Syntax_error: " . $this->_stream->readString();
91		break;
92
93		case self::UNAUTHORIZED:
94		$data['message'] = "Unauthorized: " . $this->_stream->readString();
95		break;
96
97		case self::INVALID:
98		$data['message'] = "Invalid: " . $this->_stream->readString();
99		break;
100
101		case self::CONFIG_ERROR:
102		$data['message'] = "Config_error: " . $this->_stream->readString();
103		break;
104
105		case self::ALREADY_EXIST:
106		$data['message'] = "Already_exists: " . $this->_stream->readString();
107		break;
108
109		case self::UNPREPARED:
110		$data['message'] = "Unprepared: " . $this->_stream->readShort();
111		break;
112
113		default:
114		$data['message'] = 'Unknown error';
115		}
116
117		return $data;
118		}
119
120		/**
121		*
122		* @return Exception
123		*/
124		public function getException(){
125		$data = $this->getData();
126		return new Exception($data['message'], $data['code']);
127		}
128		}
129

duoshuo / php-cassandra

Error::getData() D last analyzed 2015-12-16 12:25 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like

Error::getData() D
last analyzed 2015-12-16 12:25 UTC