Refactored AbstractEvaluation

Author: programarivm

src/Evaluation/AbstractEvaluation.php

@@ -5,6 +5,7 @@
 use Chess\Board;
 use Chess\PGN\AN\Color;
 use Chess\PGN\AN\Piece;
+use Chess\Piece\AbstractPiece;
 
 /**
  * Abstract evaluation.
@@ -33,4 +34,29 @@ public function __construct(Board $board)
             Piece::Q => 8.8,
         ];
     }
+
+    protected function attackedPieces(AbstractPiece $piece): array
+    {
+        $attackedPieces = [];
+        foreach ($sqs = $piece->getSqs() as $sq) {
+            if ($attackedPiece = $this->board->getPieceBySq($sq)) {
+                if ($attackedPiece->getId() !== Piece::P) {
+                    $attackedPieces[] = $attackedPiece;
+                }
+            }
+        }
+
+        return $attackedPieces;
+    }
+
+    protected function isKingAttacked(array $attackedPieces): bool
+    {
+        foreach ($attackedPieces as $attackedPiece) {
+            if ($attackedPiece->getId() === Piece::K) {
+                return true;
+            }
+        }
+
+        return false;
+    }
 }

src/Evaluation/AbstractForkEvaluation.php

@@ -7,31 +7,6 @@
 
 abstract class AbstractForkEvaluation extends AbstractEvaluation
 {
-    protected function attackedPieces(AbstractPiece $piece): array
-    {
-        $attackedPieces = [];
-        foreach ($sqs = $piece->getSqs() as $sq) {
-            if ($attackedPiece = $this->board->getPieceBySq($sq)) {
-                if ($attackedPiece->getId() !== Piece::P) {
-                    $attackedPieces[] = $attackedPiece;
-                }
-            }
-        }
-
-        return $attackedPieces;
-    }
-
-    protected function isKingAttacked(array $attackedPieces): bool
-    {
-        foreach ($attackedPieces as $attackedPiece) {
-            if ($attackedPiece->getId() === Piece::K) {
-                return true;
-            }
-        }
-
-        return false;
-    }
-
     protected function sumValues(AbstractPiece $piece, array $attackedPieces)
     {
         $values = 0;