Completed
Push — master ( 558809...774145 )
by Дмитрий
03:36
created

BitwiseOr::compile()   D

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 0
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
1
<?php
2
/**
3
 * PHP Smart 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 12
    protected function compile($expr, Context $context)
28
    {
29 12
        $left = $context->getExpressionCompiler()->compile($expr->var);
30 12
        $expExpression = $context->getExpressionCompiler()->compile($expr->expr);
31
32 12
        switch ($left->getType()) {
33 12
            case CompiledExpression::INTEGER:
34 12
            case CompiledExpression::DOUBLE:
35 12
            case CompiledExpression::NUMBER:
36 12
            case CompiledExpression::BOOLEAN:
37 11
                switch ($expExpression->getType()) {
38 11
                    case CompiledExpression::INTEGER:
39 11
                    case CompiledExpression::DOUBLE:
40 11
                    case CompiledExpression::NUMBER:
41 11
                    case CompiledExpression::BOOLEAN:
42 10
                        return CompiledExpression::fromZvalValue(
43 10
                            $left->getValue() | $expExpression->getValue()
44 10
                        );
45
                        break;
0 ignored issues
show
Unused Code introduced by
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...
46 1
                }
47 1
                break;
48 2
        }
49
50 2
        return new CompiledExpression(CompiledExpression::UNKNOWN);
51
    }
52
}
53