PhpDocValueRule: cheapest check first

Author: staabm

src/Rules/PhpDoc/IncompatiblePhpDocTypeRule.php

@@ -42,11 +42,6 @@ public function getNodeType(): string
 
 	public function processNode(Node $node, Scope $scope): array
 	{
-		$docComment = $node->getDocComment();
-		if ($docComment === null) {
-			return [];
-		}
-
 		if ($node instanceof Node\Stmt\ClassMethod) {
 			$functionName = $node->name->name;
 		} elseif ($node instanceof Node\Stmt\Function_) {
@@ -55,6 +50,11 @@ public function processNode(Node $node, Scope $scope): array
 			return [];
 		}
 
+		$docComment = $node->getDocComment();
+		if ($docComment === null) {
+			return [];
+		}
+
 		$resolvedPhpDoc = $this->fileTypeMapper->getResolvedPhpDoc(
 			$scope->getFile(),
 			$scope->isInClass() ? $scope->getClassReflection()->getName() : null,

src/Rules/PhpDoc/InvalidThrowsPhpDocValueRule.php

@@ -29,15 +29,15 @@ public function getNodeType(): string
 
 	public function processNode(Node $node, Scope $scope): array
 	{
+		if ($node instanceof Node\Stmt\Function_ || $node instanceof Node\Stmt\ClassMethod) {
+			return []; // is handled by virtual nodes
+		}
+
 		$docComment = $node->getDocComment();
 		if ($docComment === null) {
 			return [];
 		}
 
-		if ($node instanceof Node\Stmt\Function_ || $node instanceof Node\Stmt\ClassMethod) {
-			return []; // is handled by virtual nodes
-		}
-
 		$functionName = null;
 		if ($scope->getFunction() !== null) {
 			$functionName = $scope->getFunction()->getName();