contextAnalyzer = $contextAnalyzer; } public function provideMinPhpVersion() : int { return PhpVersionFeature::NO_BREAK_OUTSIDE_LOOP; } public function getRuleDefinition() : RuleDefinition { return new RuleDefinition('Convert break outside for/foreach/switch context to return', [new CodeSample(<<<'CODE_SAMPLE' class SomeClass { public function run() { if ($isphp5) return 1; else return 2; break; } } CODE_SAMPLE , <<<'CODE_SAMPLE' class SomeClass { public function run() { if ($isphp5) return 1; else return 2; return; } } CODE_SAMPLE )]); } /** * @return array> */ public function getNodeTypes() : array { return [Switch_::class, Break_::class]; } /** * @param Switch_|Break_ $node * @return \PhpParser\Node\Stmt\Return_|null|int */ public function refactor(Node $node) { if ($node instanceof Switch_) { $this->traverseNodesWithCallable($node->cases, static function (Node $subNode) : ?int { if ($subNode instanceof Class_ || $subNode instanceof FunctionLike && !$subNode instanceof ArrowFunction) { return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN; } if (!$subNode instanceof Break_) { return null; } $subNode->setAttribute(self::IS_BREAK_IN_SWITCH, \true); return null; }); return null; } if ($this->contextAnalyzer->isInLoop($node)) { return null; } if ($node->getAttribute(self::IS_BREAK_IN_SWITCH) === \true) { return null; } if ($this->contextAnalyzer->isInIf($node)) { return new Return_(); } return NodeTraverser::REMOVE_NODE; } }