[Performance] Avoid multiple traverse on BetterNodeFinder::hasInstanceof() (#6742)

Author: samsonasik

src/NodeManipulator/IfManipulator.php

@@ -147,14 +147,7 @@ public function collectNestedIfsWithNonBreaking(Foreach_ $foreach): array
             return [];
         }
 
-        $return = $this->betterNodeFinder->findFirstInstanceOf($currentIf->stmts, Return_::class);
-
-        if ($return instanceof Return_) {
-            return [];
-        }
-
-        $exit = $this->betterNodeFinder->findFirstInstanceOf($currentIf->stmts, Exit_::class);
-        if ($exit instanceof Exit_) {
+        if ($this->betterNodeFinder->hasInstancesOf($currentIf->stmts, [Return_::class, Exit_::class])) {
             return [];
         }
 

src/PhpParser/Node/BetterNodeFinder.php

@@ -112,16 +112,15 @@ public function hasInstancesOf(Node | array $nodes, array $types): bool
     {
         Assert::allIsAOf($types, Node::class);
 
-        foreach ($types as $type) {
-            $foundNode = $this->nodeFinder->findFirstInstanceOf($nodes, $type);
-            if (! $foundNode instanceof Node) {
-                continue;
+        return (bool) $this->nodeFinder->findFirst($nodes, static function (Node $node) use ($types): bool {
+            foreach ($types as $type) {
+                if ($node instanceof $type) {
+                    return true;
+                }
             }
 
-            return true;
-        }
-
-        return false;
+            return false;
+        });
     }
 
     /**