rector/rules/DeadCode/UselessIfCondBeforeForeachD...

117 lines
3.6 KiB
PHP

<?php
declare (strict_types=1);
namespace Rector\DeadCode;
use PhpParser\Node\Expr;
use PhpParser\Node\Expr\Array_;
use PhpParser\Node\Expr\BinaryOp\NotEqual;
use PhpParser\Node\Expr\BinaryOp\NotIdentical;
use PhpParser\Node\Expr\BooleanNot;
use PhpParser\Node\Expr\Empty_;
use PhpParser\Node\Stmt\If_;
use PhpParser\Node\Stmt\Return_;
use PHPStan\Analyser\Scope;
use Rector\PhpParser\Comparing\NodeComparator;
final class UselessIfCondBeforeForeachDetector
{
/**
* @readonly
* @var \Rector\PhpParser\Comparing\NodeComparator
*/
private $nodeComparator;
public function __construct(NodeComparator $nodeComparator)
{
$this->nodeComparator = $nodeComparator;
}
/**
* Matches:
* empty($values)
*/
public function isMatchingEmptyAndForeachedExpr(If_ $if, Expr $foreachExpr) : bool
{
if (!$if->cond instanceof Empty_) {
return \false;
}
/** @var Empty_ $empty */
$empty = $if->cond;
if (!$this->nodeComparator->areNodesEqual($empty->expr, $foreachExpr)) {
return \false;
}
if ($if->stmts === []) {
return \true;
}
if (\count($if->stmts) !== 1) {
return \false;
}
$stmt = $if->stmts[0];
return $stmt instanceof Return_ && !$stmt->expr instanceof Expr;
}
/**
* Matches:
* !empty($values)
*/
public function isMatchingNotEmpty(If_ $if, Expr $foreachExpr, Scope $scope) : bool
{
$cond = $if->cond;
if (!$cond instanceof BooleanNot) {
return \false;
}
if (!$cond->expr instanceof Empty_) {
return \false;
}
/** @var Empty_ $empty */
$empty = $cond->expr;
return $this->areCondExprAndForeachExprSame($empty, $foreachExpr, $scope);
}
/**
* Matches:
* $values !== []
* $values != []
* [] !== $values
* [] != $values
*/
public function isMatchingNotIdenticalEmptyArray(If_ $if, Expr $foreachExpr) : bool
{
if (!$if->cond instanceof NotIdentical && !$if->cond instanceof NotEqual) {
return \false;
}
/** @var NotIdentical|NotEqual $notIdentical */
$notIdentical = $if->cond;
return $this->isMatchingNotBinaryOp($notIdentical, $foreachExpr);
}
/**
* @param \PhpParser\Node\Expr\BinaryOp\NotIdentical|\PhpParser\Node\Expr\BinaryOp\NotEqual $binaryOp
*/
private function isMatchingNotBinaryOp($binaryOp, Expr $foreachExpr) : bool
{
if ($this->isEmptyArrayAndForeachedVariable($binaryOp->left, $binaryOp->right, $foreachExpr)) {
return \true;
}
return $this->isEmptyArrayAndForeachedVariable($binaryOp->right, $binaryOp->left, $foreachExpr);
}
private function isEmptyArrayAndForeachedVariable(Expr $leftExpr, Expr $rightExpr, Expr $foreachExpr) : bool
{
if (!$this->isEmptyArray($leftExpr)) {
return \false;
}
return $this->nodeComparator->areNodesEqual($foreachExpr, $rightExpr);
}
private function isEmptyArray(Expr $expr) : bool
{
if (!$expr instanceof Array_) {
return \false;
}
return $expr->items === [];
}
private function areCondExprAndForeachExprSame(Empty_ $empty, Expr $foreachExpr, Scope $scope) : bool
{
if (!$this->nodeComparator->areNodesEqual($empty->expr, $foreachExpr)) {
return \false;
}
// is array though?
$arrayType = $scope->getType($empty->expr);
return $arrayType->isArray()->yes();
}
}