ErrorHandler::error() - Code Metrics - Inspection of "Initial commit" - gabrielbull/php-waredesk-api - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 6b2c4c...9e30d1 )

by Gabriel

created 2017-05-05 14:06 UTC

ErrorHandler::error() B

↳ Parent: ErrorHandler

Complexity

Conditions	7
Paths	7

Size

Total Lines	12
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	56

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	0
cts	9
cp	0
rs	8.2222
c	0
b	0
f	0
cc	7
eloc	9
nc	7
nop	1
crap	56

<?php

namespace Waredesk;

use Waredesk\Exceptions\AccountBannedException;
use Waredesk\Exceptions\AccountDeletedException;
use Waredesk\Exceptions\AccountInvalidException;
use Waredesk\Exceptions\EmailNotConfirmedException;
use Waredesk\Exceptions\InvalidClientException;
use Waredesk\Exceptions\InvalidRequestException;
use Waredesk\Exceptions\UnknownException;

class ErrorHandler
{
    public static function error(array $error)
    {
        switch($error['error']) {
            case 'invalid_request': throw new InvalidRequestException($error['message']);
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'account_banned': throw new AccountBannedException($error['message']);
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'account_deleted': throw new AccountDeletedException($error['message']);
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'account_invalid': throw new AccountInvalidException($error['message']);
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'email_not_confirmed': throw new EmailNotConfirmedException($error['message']);
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
            case 'invalid_client': throw new InvalidClientException($error['message']);
switch ($expr) {
case "A":
    doSomething(); //right
    break;
case "B":

    doSomethingElse(); //wrong
    break;
        }
        throw new UnknownException();
    }
}


1			<?php
2
3			namespace Waredesk;
4
5			use Waredesk\Exceptions\AccountBannedException;
6			use Waredesk\Exceptions\AccountDeletedException;
7			use Waredesk\Exceptions\AccountInvalidException;
8			use Waredesk\Exceptions\EmailNotConfirmedException;
9			use Waredesk\Exceptions\InvalidClientException;
10			use Waredesk\Exceptions\InvalidRequestException;
11			use Waredesk\Exceptions\UnknownException;
12
13			class ErrorHandler
14			{
15			public static function error(array $error)
16			{
17			switch($error['error']) {
18			case 'invalid_request': throw new InvalidRequestException($error['message']);
			0 ignored issues – show Coding Style introduced 2017-05-05 14:08 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 2017-05-05 14:08 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...
19			case 'account_banned': throw new AccountBannedException($error['message']);
			0 ignored issues – show Coding Style introduced 2017-05-05 14:08 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 2017-05-05 14:08 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...
20			case 'account_deleted': throw new AccountDeletedException($error['message']);
			0 ignored issues – show Coding Style introduced 2017-05-05 14:08 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 2017-05-05 14:08 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...
21			case 'account_invalid': throw new AccountInvalidException($error['message']);
			0 ignored issues – show Coding Style introduced 2017-05-05 14:08 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 2017-05-05 14:08 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...
22			case 'email_not_confirmed': throw new EmailNotConfirmedException($error['message']);
			0 ignored issues – show Coding Style introduced 2017-05-05 14:08 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 2017-05-05 14:08 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...
23			case 'invalid_client': throw new InvalidClientException($error['message']);
			0 ignored issues – show Coding Style introduced 2017-05-05 14:08 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 2017-05-05 14:08 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...
24			}
25			throw new UnknownException();
26			}
27			}
28

gabrielbull / php-waredesk-api

Push — master ( 6b2c4c...9e30d1 )

ErrorHandler::error() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like