QRString - Code Metrics - Inspection of "updated readme" - chillerlan/php-qrcode - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 102ef4...1ff5c6 )

by smiley

created 2016-12-17 21:24 UTC

QRString A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

Metric	Value
wmc	7
lcom	1
cbo	1
dl	0
loc	43
rs	10
c	0
b	0
f	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	dump()	0	10	3
A	toString()	0	15	4

<?php
/**
 * Class QRString
 *
 * @filesource   QRString.php
 * @created      05.12.2015
 * @package      chillerlan\QRCode\Output
 * @author       Smiley <[email protected]>
 * @copyright    2015 Smiley
 * @license      MIT
 */

namespace chillerlan\QRCode\Output;

use chillerlan\QRCode\QRCode;

/**
 *
 */
class QRString extends QROutputAbstract{

	protected $optionsInterface = QRStringOptions::class;

	protected $types = [
		QRCode::OUTPUT_STRING_TEXT,
		QRCode::OUTPUT_STRING_JSON,
	];

	/**
	 * @return string
	 */
	public function dump(){

		switch($this->options->type){
			case QRCode::OUTPUT_STRING_TEXT: return $this->toString();
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
			case QRCode::OUTPUT_STRING_JSON:
			default:
				return json_encode($this->matrix);
		}

	}

	/**
	 * @return string
	 */
	protected function toString(){
		$str = '';

		foreach($this->matrix as $row){
			foreach($row as $col){
				$str .= $col
					? $this->options->textDark
					: $this->options->textLight;
			}

			$str .= $this->options->eol;
		}

		return $str;
	}

}


1			<?php
2			/**
3			* Class QRString
4			*
5			* @filesource QRString.php
6			* @created 05.12.2015
7			* @package chillerlan\QRCode\Output
8			* @author Smiley <[email protected]>
9			* @copyright 2015 Smiley
10			* @license MIT
11			*/
12
13			namespace chillerlan\QRCode\Output;
14
15			use chillerlan\QRCode\QRCode;
16
17			/**
18			*
19			*/
20			class QRString extends QROutputAbstract{
21
22			protected $optionsInterface = QRStringOptions::class;
23
24			protected $types = [
25			QRCode::OUTPUT_STRING_TEXT,
26			QRCode::OUTPUT_STRING_JSON,
27			];
28
29			/**
30			* @return string
31			*/
32			public function dump(){
33
34			switch($this->options->type){
35			case QRCode::OUTPUT_STRING_TEXT: return $this->toString();
			2 ignored issues – show Coding Style introduced 2016-03-20 23:07 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-03-20 23:07 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...
36			case QRCode::OUTPUT_STRING_JSON:
37			default:
38			return json_encode($this->matrix);
39			}
40
41			}
42
43			/**
44			* @return string
45			*/
46			protected function toString(){
47			$str = '';
48
49			foreach($this->matrix as $row){
50			foreach($row as $col){
51			$str .= $col
52			? $this->options->textDark
53			: $this->options->textLight;
54			}
55
56			$str .= $this->options->eol;
57			}
58
59			return $str;
60			}
61
62			}
63

chillerlan / php-qrcode

Push — master ( 102ef4...1ff5c6 )

QRString A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

2 Methods

Duplication Side-by-Side

Filter issues like