BitwiseXor::compile() - Code Metrics - Inspection of "[Compiler] Add AssignOp tests" - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 558809...774145 )

by Дмитрий

created 2016-09-17 10:50 UTC

BitwiseXor::compile() D

↳ Parent: BitwiseXor

Complexity

Conditions	9
Paths	21

Size

Total Lines	25
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	20
CRAP Score	9

Importance

Changes

Metric	Value
cc	9
eloc	18
nc	21
nop	2
dl	0
loc	25
ccs	20
cts	20
cp	1
crap	9
rs	4.909
c	0
b	0
f	0

<?php

namespace PHPSA\Compiler\Expression\AssignOp;

use PHPSA\CompiledExpression;
use PHPSA\Compiler\Expression;
use PHPSA\Compiler\Expression\AbstractExpressionCompiler;
use PHPSA\Context;

class BitwiseXor extends AbstractExpressionCompiler
{
    protected $name = 'PhpParser\Node\Expr\AssignOp\BitwiseXor';

    /**
     * {left-expr} ^= {right-expr}
     *
     * @param \PhpParser\Node\Expr\AssignOp\BitwiseXor $expr
     * @param Context $context
     * @return CompiledExpression
     */
    protected function compile($expr, Context $context)
    {
        $left = $context->getExpressionCompiler()->compile($expr->var);
        $expExpression = $context->getExpressionCompiler()->compile($expr->expr);

        switch ($left->getType()) {
            case CompiledExpression::INTEGER:
            case CompiledExpression::DOUBLE:
            case CompiledExpression::NUMBER:
            case CompiledExpression::BOOLEAN:
                switch ($expExpression->getType()) {
                    case CompiledExpression::INTEGER:
                    case CompiledExpression::DOUBLE:
                    case CompiledExpression::NUMBER:
                    case CompiledExpression::BOOLEAN:
                        return CompiledExpression::fromZvalValue(
                            $left->getValue() ^ $expExpression->getValue()
                        );
                        break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
                }
                break;
        }

        return new CompiledExpression(CompiledExpression::UNKNOWN);
    }
}


1		<?php
2
3		namespace PHPSA\Compiler\Expression\AssignOp;
4
5		use PHPSA\CompiledExpression;
6		use PHPSA\Compiler\Expression;
7		use PHPSA\Compiler\Expression\AbstractExpressionCompiler;
8		use PHPSA\Context;
9
10		class BitwiseXor extends AbstractExpressionCompiler
11		{
12		protected $name = 'PhpParser\Node\Expr\AssignOp\BitwiseXor';
13
14		/**
15		* {left-expr} ^= {right-expr}
16		*
17		* @param \PhpParser\Node\Expr\AssignOp\BitwiseXor $expr
18		* @param Context $context
19		* @return CompiledExpression
20		*/
21	12	protected function compile($expr, Context $context)
22		{
23	12	$left = $context->getExpressionCompiler()->compile($expr->var);
24	12	$expExpression = $context->getExpressionCompiler()->compile($expr->expr);
25
26	12	switch ($left->getType()) {
27	12	case CompiledExpression::INTEGER:
28	12	case CompiledExpression::DOUBLE:
29	12	case CompiledExpression::NUMBER:
30	12	case CompiledExpression::BOOLEAN:
31	11	switch ($expExpression->getType()) {
32	11	case CompiledExpression::INTEGER:
33	11	case CompiledExpression::DOUBLE:
34	11	case CompiledExpression::NUMBER:
35	11	case CompiledExpression::BOOLEAN:
36	10	return CompiledExpression::fromZvalValue(
37	10	$left->getValue() ^ $expExpression->getValue()
38	10	);
39		break;
		0 ignored issues – show Unused Code introduced 2016-09-13 12:43 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
40	1	}
41	1	break;
42	2	}
43
44	2	return new CompiledExpression(CompiledExpression::UNKNOWN);
45		}
46		}
47

ovr / phpsa

Push — master ( 558809...774145 )

BitwiseXor::compile() D

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like