alex-patterson-webdev / domino-game

public/index.php

Spacing +3 added lines, -3 removed lines

@@ -7,13 +7,13 @@ 
 
 $playerNames = isset($argv[1]) ? explode(',', $argv[1]) : null;
 if (empty($playerNames)) {
-    echo 'Please provide at least two player names as a comma separated list' . PHP_EOL;
+    echo 'Please provide at least two player names as a comma separated list'.PHP_EOL;
     return 0;
 }
 
 $playerNameCount = count($playerNames);
 if ($playerNameCount > 4) {
-    echo 'Please provide between 1-4 player names' . PHP_EOL;
+    echo 'Please provide between 1-4 player names'.PHP_EOL;
     return 0;
 }
 
@@ -24,6 +24,6 @@ 
     $game = (new DominoGameFactory())->create($playerNames, $maxTileSize);
     $game->run($handSize);
 } catch (DominoGameException $e) {
-    echo 'The game could not be executed: ' . $e->getMessage() . PHP_EOL;
+    echo 'The game could not be executed: '.$e->getMessage().PHP_EOL;
     return 1;
 }

test/phpunit/DominoGameTest.php

Indentation +4 added lines, -4 removed lines

@@ -145,12 +145,12 @@
         $players = $this->createMockPlayersArray(2);
         $playersCollection = $this->createMockPlayersCollection($players);
 
-         $game = new DominoGame($playersCollection, $this->logger, 6);
+            $game = new DominoGame($playersCollection, $this->logger, 6);
 
-         $this->expectException(DominoGameException::class);
-         $this->expectExceptionMessage('Hand sizes must be a minimum of 1');
+            $this->expectException(DominoGameException::class);
+            $this->expectExceptionMessage('Hand sizes must be a minimum of 1');
 
-         $game->deal($handSize);
+            $game->deal($handSize);
     }
 
     /**

Spacing +1 added lines, -1 removed lines

@@ -227,7 +227,7 @@
 
         $players = [];
         for ($x = 0; $x < $playerCount; $x++) {
-            $players[] = new Player('Player ' . $x);
+            $players[] = new Player('Player '.$x);
         }
 
         $game = new DominoGame(new PlayerCollection($players), $this->logger, 6);

src/DominoGame.php

Spacing +11 added lines, -11 removed lines

@@ -89,7 +89,7 @@ 
         // Ensure that we randomise the deck before dealing a domino to each player in turn
         $this->deck->shuffle();
 
-        $this->logger->info(sprintf('Deck shuffled: %s', (string)$this->deck));
+        $this->logger->info(sprintf('Deck shuffled: %s', (string) $this->deck));
         $this->logger->info(
             sprintf('Dealing a hand size of %d dominoes to %d players', $handSize, $playerCount)
         );
@@ -101,7 +101,7 @@ 
                     $this->deck->removeElement($domino);
 
                     $this->logger->info(
-                        sprintf('\'%s\' was dealt domino \'%s\'', (string)$player, (string)$domino)
+                        sprintf('\'%s\' was dealt domino \'%s\'', (string) $player, (string) $domino)
                     );
 
                     // Move on to next domino
@@ -198,7 +198,7 @@ 
                 $this->logger->info(
                     sprintf(
                         '\'%s\' is the winner with the lowest hand value of \'%d\'',
-                        (string)$winner,
+                        (string) $winner,
                         $winner->getHandValue()
                     )
                 );
@@ -209,7 +209,7 @@ 
                 $this->logger->info(
                     sprintf(
                         '\'%s\' is the winner with 0 dominoes left to play',
-                        (string)$player
+                        (string) $player
                     )
                 );
                 return $player;
@@ -234,7 +234,7 @@ 
     {
         $domino = $player->getDominoWithMatchingTile($this->board);
         if (null === $domino) {
-            $this->logger->info(sprintf('\'%s\' was unable to find a matching domino', (string)$player));
+            $this->logger->info(sprintf('\'%s\' was unable to find a matching domino', (string) $player));
 
             // If there are no more dominoes to pick, we need to resolve the winner
             if ($this->deck->isEmpty()) {
@@ -242,7 +242,7 @@ 
                     sprintf(
                         '\'%s\' has no more dominoes available to pick from the deck. '
                         . 'Determining the winner from the lowest total score from each players hand',
-                        (string)$player
+                        (string) $player
                     )
                 );
                 return $this->players->getWithLowestHandValue();
@@ -255,14 +255,14 @@ 
             $this->logger->info(
                 sprintf(
                     '\'%s\' has picked domino %s from the deck',
-                    (string)$player,
-                    (string)$randomPick
+                    (string) $player,
+                    (string) $randomPick
                 )
             );
             return null;
         }
 
-        $this->logger->info(sprintf('\'%s\' has placed domino %s', (string)$player, (string)$domino));
+        $this->logger->info(sprintf('\'%s\' has placed domino %s', (string) $player, (string) $domino));
 
         $this->board->place($domino);
         $player->removeFromHand($domino);
@@ -296,12 +296,12 @@ 
     private function logSummary(): void
     {
         if (!$this->deck->isEmpty()) {
-            $this->logger->info('Remaining tiles: ' . (string)$this->deck);
+            $this->logger->info('Remaining tiles: '.(string) $this->deck);
         }
 
         foreach ($this->players as $player) {
             $this->logger->info(
-                sprintf('\'%s\' hand: %s', (string)$player, (string)$player->getHand())
+                sprintf('\'%s\' hand: %s', (string) $player, (string) $player->getHand())
             );
         }
     }

src/Value/Player.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
         $hand = $this->hand->getElements();
         usort(
             $hand,
-            static function (Domino $a, Domino $b) {
+            static function(Domino $a, Domino $b) {
                 if ($a->isDouble() && !$b->isDouble()) {
                     return 0;
                 }

src/Value/Domino.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
      */
     public function getName(): string
     {
-        return $this->topTile . '-' . $this->bottomTile;
+        return $this->topTile.'-'.$this->bottomTile;
     }
 
     /**

src/Value/DominoCollection.php

Spacing +2 added lines, -2 removed lines

@@ -110,7 +110,7 @@ 
 
         uasort(
             $elements,
-            static function (Domino $dominoA, Domino $dominoB) {
+            static function(Domino $dominoA, Domino $dominoB) {
                 return $dominoA->getValue() <=> $dominoB->getValue();
             }
         );
@@ -180,7 +180,7 @@ 
         $string = '';
 
         foreach ($this->elements as $element) {
-            $string .= $element . ' ';
+            $string .= $element.' ';
         }
 
         return trim($string);

src/Value/Board.php

Spacing +3 added lines, -3 removed lines

@@ -72,7 +72,7 @@ 
     {
         // If there is no virtual domino we can directly add to the board as it is our first piece.
         if (0 === $this->placed->count()) {
-            if (! $domino->isDouble()) {
+            if (!$domino->isDouble()) {
                 throw new InvalidArgumentException(
                     sprintf('The first domino place must be a double \'%s\' provided', $domino->getName())
                 );
@@ -112,7 +112,7 @@ 
      */
     private function placeLeft(Domino $domino): void
     {
-        $leftTitle = $this->virtualDomino->getTopTile();    // Represents left placement
+        $leftTitle = $this->virtualDomino->getTopTile(); // Represents left placement
         $rightTile = $this->virtualDomino->getBottomTile(); // Represents right placement
 
         // Does the top tile match the left side of the placed pieces?
@@ -139,7 +139,7 @@ 
     private function placeRight(Domino $domino): void
     {
         $rightTile = $this->virtualDomino->getBottomTile(); // Represents right placement
-        $leftTitle = $this->virtualDomino->getTopTile();    // Represents left placement
+        $leftTitle = $this->virtualDomino->getTopTile(); // Represents left placement
 
         // Does the top tile match the right side of the placed pieces?
         if ($domino->getTopTile() === $rightTile) {

src/Value/PlayerCollection.php

Spacing +3 added lines, -3 removed lines

@@ -23,7 +23,7 @@ 
 
         usort(
             $elements,
-            static function (Player $playerA, Player $playerB): int {
+            static function(Player $playerA, Player $playerB): int {
                 return $playerA->getHandCount() <=> $playerB->getHandCount();
             }
         );
@@ -42,7 +42,7 @@ 
 
         usort(
             $elements,
-            static function (Player $playerA, Player $playerB): int {
+            static function(Player $playerA, Player $playerB): int {
                 $doubleA = $playerA->getHighestDouble();
                 $doubleB = $playerB->getHighestDouble();
 
@@ -78,7 +78,7 @@ 
 
         usort(
             $elements,
-            static function (Player $playerA, Player $playerB): int {
+            static function(Player $playerA, Player $playerB): int {
                 return $playerA->getHandValue() <=> $playerB->getHandValue();
             }
         );