[perf] Skip reflection in RemoveUnusedPrivateMethodRector when class has no private method

Author: TomasVotruba

rules/DeadCode/Rector/ClassMethod/RemoveUnusedPrivateMethodRector.php

@@ -81,12 +81,12 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?Node
     {
-        if ($node->getMethods() === []) {
+        // unreliable to detect on anonymous class: doesn't make sense
+        if ($node->isAnonymous()) {
             return null;
         }
 
-        // unreliable to detect on anonymous class: doesn't make sense
-        if ($node->isAnonymous()) {
+        if (! $this->hasPrivateMethod($node)) {
             return null;
         }
 
@@ -145,6 +145,17 @@ public function refactor(Node $node): ?Node
         return null;
     }
 
+    private function hasPrivateMethod(Class_ $class): bool
+    {
+        foreach ($class->getMethods() as $classMethod) {
+            if ($classMethod->isPrivate()) {
+                return true;
+            }
+        }
+
+        return false;
+    }
+
     private function shouldSkip(ClassMethod $classMethod, ClassReflection $classReflection): bool
     {
         // unreliable to detect trait, interface: doesn't make sense