BitwiseOr::compile() - Code Metrics - Inspection of "[Compiler] Implement AssignOp\BitwiseOr (|=)" - ovr/phpsa - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 57c3b1...6ee928 )

by Дмитрий

created 2016-03-17 14:20 UTC

BitwiseOr::compile() C

↳ Parent: BitwiseOr

Complexity

Conditions	7
Paths	13

Size

Total Lines	26
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

Metric	Value
dl	0
loc	26
rs	6.7272
c	1
b	0
f	1
cc	7
eloc	18
nc	13
nop	2

<?php
/**
 * PHP Static Analysis project 2015-2016
 *
 * @author Patsura Dmitry https://github.com/ovr <[email protected]>
 */

namespace PHPSA\Compiler\Expression\AssignOp;

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

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

    /**
     * It's used in conditions
     * {left-expr} |= {right-expr}
     *
     * @param \PhpParser\Node\Expr\AssignOp\BitwiseOr $expr
     * @param Context $context
     * @return CompiledExpression
     */
    protected function compile($expr, Context $context)
    {
        $expression = new Expression($context);
        $left = $expression->compile($expr->var);

        $expression = new Expression($context);
        $expExpression = $expression->compile($expr->expr);

        switch ($left->getType()) {
            case CompiledExpression::INTEGER:
            case CompiledExpression::DOUBLE:
            case CompiledExpression::NUMBER:
                switch ($expExpression->getType()) {
                    case CompiledExpression::INTEGER:
                    case CompiledExpression::DOUBLE:
                    case CompiledExpression::NUMBER:
                        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 $left;
    }
}


1			<?php
2			/**
3			* PHP Static Analysis project 2015-2016
4			*
5			* @author Patsura Dmitry https://github.com/ovr <[email protected]>
6			*/
7
8			namespace PHPSA\Compiler\Expression\AssignOp;
9
10			use PHPSA\CompiledExpression;
11			use PHPSA\Compiler\Expression;
12			use PHPSA\Compiler\Expression\AbstractExpressionCompiler;
13			use PHPSA\Context;
14
15			class BitwiseOr extends AbstractExpressionCompiler
16			{
17			protected $name = 'PhpParser\Node\Expr\AssignOp\BitwiseOr';
18
19			/**
20			* It's used in conditions
21			* {left-expr} \|= {right-expr}
22			*
23			* @param \PhpParser\Node\Expr\AssignOp\BitwiseOr $expr
24			* @param Context $context
25			* @return CompiledExpression
26			*/
27			protected function compile($expr, Context $context)
28			{
29			$expression = new Expression($context);
30			$left = $expression->compile($expr->var);
31
32			$expression = new Expression($context);
33			$expExpression = $expression->compile($expr->expr);
34
35			switch ($left->getType()) {
36			case CompiledExpression::INTEGER:
37			case CompiledExpression::DOUBLE:
38			case CompiledExpression::NUMBER:
39			switch ($expExpression->getType()) {
40			case CompiledExpression::INTEGER:
41			case CompiledExpression::DOUBLE:
42			case CompiledExpression::NUMBER:
43			return CompiledExpression::fromZvalValue(
44			$left->getValue() \| $expExpression->getValue()
45			);
46			break;
			0 ignored issues – show Unused Code introduced 2016-03-17 15:00 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...
47			}
48			break;
49			}
50
51			return $left;
52			}
53			}
54

ovr / phpsa

Push — master ( 57c3b1...6ee928 )

BitwiseOr::compile() C

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like