forked from rectorphp/rector
-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathRemoveZeroBreakContinueRector.php
132 lines (111 loc) · 2.96 KB
/
RemoveZeroBreakContinueRector.php
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
<?php
declare(strict_types=1);
namespace Rector\Php54\Rector\Break_;
use PhpParser\Node;
use PhpParser\Node\Expr\Variable;
use PhpParser\Node\Scalar\LNumber;
use PhpParser\Node\Stmt;
use PhpParser\Node\Stmt\Break_;
use PhpParser\Node\Stmt\Continue_;
use PHPStan\Type\Constant\ConstantIntegerType;
use PHPStan\Type\ConstantType;
use Rector\Core\Rector\AbstractRector;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see https://www.php.net/manual/en/control-structures.continue.php
* @see https://www.php.net/manual/en/control-structures.break.php
*
* @see \Rector\Tests\Php54\Rector\Break_\RemoveZeroBreakContinueRector\RemoveZeroBreakContinueRectorTest
*/
final class RemoveZeroBreakContinueRector extends AbstractRector
{
public function getRuleDefinition(): RuleDefinition
{
return new RuleDefinition('Remove 0 from break and continue', [
new CodeSample(
<<<'CODE_SAMPLE'
class SomeClass
{
public function run($random)
{
continue 0;
break 0;
$five = 5;
continue $five;
break $random;
}
}
CODE_SAMPLE
,
<<<'CODE_SAMPLE'
class SomeClass
{
public function run($random)
{
continue;
break;
$five = 5;
continue 5;
break;
}
}
CODE_SAMPLE
),
]);
}
/**
* @return array<class-string<Node>>
*/
public function getNodeTypes(): array
{
return [Break_::class, Continue_::class];
}
/**
* @param Break_|Continue_ $node
*/
public function refactor(Node $node): ?Node
{
if ($node->num === null) {
return null;
}
if ($node->num instanceof LNumber) {
$number = $this->valueResolver->getValue($node->num);
if ($number > 1) {
return null;
}
if ($number === 0) {
$node->num = null;
return $node;
}
return null;
}
if ($node->num instanceof Variable) {
return $this->processVariableNum($node, $node->num);
}
return null;
}
/**
* @param Break_|Continue_ $stmt
*/
private function processVariableNum(Stmt $stmt, Variable $numVariable): ?Node
{
$staticType = $this->getStaticType($numVariable);
if ($staticType instanceof ConstantType) {
if ($staticType instanceof ConstantIntegerType) {
if ($staticType->getValue() === 0) {
$stmt->num = null;
return $stmt;
}
if ($staticType->getValue() > 0) {
$stmt->num = new LNumber($staticType->getValue());
return $stmt;
}
}
return $stmt;
}
// remove variable
$stmt->num = null;
return null;
}
}