PHPSA\Analyzer\Pass\Expression\LogicInversion

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	pass()	0	16	3
A	getRegister()	0	6	1

<?php

/**
 * @author Medvedev Alexandr https://github.com/lexty <alexandr.mdr@gmail.com>
 */
namespace PHPSA\Analyzer\Pass\Expression;

use PhpParser\Node\Expr\BooleanNot;
use PHPSA\Analyzer\Pass\AnalyzerPassInterface;
use PHPSA\Context;

class LogicInversion implements AnalyzerPassInterface
{
    protected static $map = [
        'Expr_BinaryOp_Equal' => ['!=', '=='],
        'Expr_BinaryOp_NotEqual' => ['==', '!='],
        'Expr_BinaryOp_Identical' => ['===', '!=='],
        'Expr_BinaryOp_NotIdentical' => ['!==', '==='],
        'Expr_BinaryOp_Greater' => ['>', '<='],
        'Expr_BinaryOp_GreaterOrEqual' => ['>=', '<'],
        'Expr_BinaryOp_Smaller' => ['<', '>='],
        'Expr_BinaryOp_SmallerOrEqual' => ['<=', '>'],
    ];

    /**
     * @param BooleanNot $expr
     * @param Context $context
     * @return bool
     */
    public function pass(BooleanNot $expr, Context $context)
    {
        if ($expr->expr->getType() === 'Expr_BooleanNot') {
            // BooleanNot expression needs a different message.
            $msg = 'Use "a" expression instead of "!(!a)".';
        } else if (!array_key_exists($expr->expr->getType(), static::$map)) {
            return false;
        } else {
            list($use, $instead) = static::$map[$expr->expr->getType()];
            $msg = sprintf('Use "a %s b" expression instead of "!(a %s b)".', $use, $instead);
        }

        $context->notice('logic_inversion', $msg, $expr);

        return true;
    }

    /**
     * @return array
     */
    public function getRegister()
    {
        return [
            BooleanNot::class,
        ];
    }
}