thunderer / Logeek

src/Board.php

Doc Comments +19 added lines

@@ -60,6 +60,9 @@ 
         }
     }
 
+    /**
+     * @param string $alias
+     */
     public function hasAction($alias): bool
     {
         return array_key_exists($alias, $this->actions);
@@ -140,6 +143,9 @@ 
         $this->getAction($operation['action'])->execute($this, $alias, $operation);
     }
 
+    /**
+     * @param string $alias
+     */
     public function getActorNextMove($alias): array
     {
         list($x, $y) = $this->getActorPosition($alias);
@@ -163,6 +169,9 @@ 
         return $this->functions[$name];
     }
 
+    /**
+     * @param string $alias
+     */
     public function moveActor($alias)
     {
         $moveMap = [
@@ -191,6 +200,9 @@ 
     {
     }
 
+    /**
+     * @param string $field
+     */
     public function setField($x, $y, $field)
     {
         $this->fields[$x][$y] = $field;
@@ -206,6 +218,9 @@ 
         return $this->actors[$alias]['direction'];
     }
 
+    /**
+     * @param string $alias
+     */
     public function setActorDirection($alias, $direction)
     {
         $this->actors[$alias]['direction'] = $direction;
@@ -216,6 +231,10 @@ 
         return $this->actors[$alias]['pick'];
     }
 
+    /**
+     * @param string $alias
+     * @param string|null $pick
+     */
     public function setActorPick($alias, $pick)
     {
         $this->actors[$alias]['pick'] = $pick;

Spacing +12 added lines, -12 removed lines

@@ -34,7 +34,7 @@ 
      */
     public function addActions(array $actions)
     {
-        foreach($actions as $action) {
+        foreach ($actions as $action) {
             $this->addAction($action);
         }
     }
@@ -46,7 +46,7 @@ 
 
     public function getAction(string $alias): ActionInterface
     {
-        if(!$this->hasAction($alias)) {
+        if (!$this->hasAction($alias)) {
             throw new \RuntimeException(sprintf('Action %s does not exist!', $alias));
         }
 
@@ -55,7 +55,7 @@ 
 
     public function addFieldTypes(array $types)
     {
-        foreach($types as $alias => $symbol) {
+        foreach ($types as $alias => $symbol) {
             $this->addFieldType($alias, $symbol);
         }
     }
@@ -76,9 +76,9 @@ 
         $lines = explode("\n", trim($string));
         $height = \count($lines);
         $width = \strlen(trim($lines[0]));
-        for($i = 0; $i < $height; $i++) {
+        for ($i = 0; $i < $height; $i++) {
             $line = trim($lines[$i]);
-            for($j = 0; $j < $width; $j++) {
+            for ($j = 0; $j < $width; $j++) {
                 $this->fields[$i][$j] = $types[$line[$j]];
             }
         }
@@ -105,7 +105,7 @@ 
 
     public function addFunctions(array $functions)
     {
-        foreach($functions as $name => $program) {
+        foreach ($functions as $name => $program) {
             $this->functions[$name] = $program;
         }
     }
@@ -130,7 +130,7 @@ 
 
     public function runActorProgram($alias, array $program)
     {
-        foreach($program as $operation) {
+        foreach ($program as $operation) {
             $this->runActorOperation($alias, $operation);
         }
     }
@@ -178,7 +178,7 @@ 
         $newY = $actor['y'] + $y;
 
         $this->debug('Move [%s:%s] -> [%s:%s]', $actor['y'], $actor['x'], $newY, $newX);
-        if('wall' === $this->fields[$newX][$newY]) {
+        if ('wall' === $this->fields[$newX][$newY]) {
             throw new \RuntimeException(sprintf('Wall at [%s, %s]', $newX, $newY));
         }
 
@@ -230,8 +230,8 @@ 
     {
         $return = '';
         $return .= "\n";
-        for($i = 0; $i < $this->height; $i++) {
-            for($j = 0; $j < $this->width; $j++) {
+        for ($i = 0; $i < $this->height; $i++) {
+            for ($j = 0; $j < $this->width; $j++) {
                 $symbol = $this->fieldTypes[$this->fields[$i][$j]];
                 $isActor = array_reduce($this->actors, function($state, array $item) use ($i, $j) {
                     $state += $item['x'] === $i && $item['y'] === $j;
@@ -241,10 +241,10 @@ 
                     $state += $item['x'] === $i && $item['y'] === $j;
                     return $state;
                 }, 0);
-                if($isExit) {
+                if ($isExit) {
                     $symbol = 'E';
                 }
-                if($isActor) {
+                if ($isActor) {
                     $symbol = 'A';
                 }
                 $return .= $symbol;

src/Action/ForLoopAction.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
 {
     public function execute(Board $board, string $alias, array $operation)
     {
-        for($i = 0; $i < $operation['iterations']; $i++) {
+        for ($i = 0; $i < $operation['iterations']; $i++) {
             $board->runActorProgram($alias, $operation['program']);
         }
     }

src/Action/MoveAction.php

Spacing +2 added lines, -2 removed lines

@@ -10,14 +10,14 @@
     public function execute(Board $board, string $alias, array $operation)
     {
         $length = $this->getMoveLength($board, $operation);
-        for($i = 0; $i < $length; $i++) {
+        for ($i = 0; $i < $length; $i++) {
             $board->moveActor($alias);
         }
     }
 
     private function getMoveLength(Board $board, array $operation)
     {
-        if(is_numeric($operation['distance'])) {
+        if (is_numeric($operation['distance'])) {
             $length = $operation['distance'];
             $board->debug('Move Length[%s]', $length);
             return $length;

src/Action/PickAction.php

Spacing +2 added lines, -2 removed lines

@@ -21,11 +21,11 @@
         $newX = $x + static::$moveMap[$direction][0];
         $newY = $y + static::$moveMap[$direction][1];
 
-        if('up' === $operation['direction']) {
+        if ('up' === $operation['direction']) {
             $board->debug('Pick[up]');
             $board->setField($newX, $newY, 'ground');
             $board->setActorPick($alias, 'brick');
-        } elseif('down' === $operation['direction']) {
+        } elseif ('down' === $operation['direction']) {
             $board->debug('Pick[down]');
             $board->setField($newX, $newY, 'brick');
             $board->setActorPick($alias, null);

src/Action/WhileLoopAction.php

Spacing +4 added lines, -4 removed lines

@@ -17,18 +17,18 @@
     public function execute(Board $board, string $alias, array $operation)
     {
         $iteration = 0;
-        while(true) {
+        while (true) {
             $left = (string)$board->getVariable($operation['left']);
             $board->debug(sprintf('While Evaluate %s %s %s', $left, $operation['operator'], $operation['right']));
-            if($operation['operator'] === 'is' && $left === (string)$operation['right']) {
+            if ($operation['operator'] === 'is' && $left === (string)$operation['right']) {
                 $board->runActorProgram($alias, $operation['program']);
-            } elseif($operation['operator'] === 'not' && $left !== (string)$operation['right']) {
+            } elseif ($operation['operator'] === 'not' && $left !== (string)$operation['right']) {
                 $board->runActorProgram($alias, $operation['program']);
             } else {
                 $board->debug('While LoopEnd');
                 break;
             }
-            if($iteration > $this->iterations) {
+            if ($iteration > $this->iterations) {
                 $board->debug('Iterations exceeded!');
                 break;
             }

src/Action/OpenAction.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
     public function execute(Board $board, string $alias, array $operation)
     {
         list($newX, $newY) = $board->getActorNextMove($alias);
-        if('door' === $board->getField($newX, $newY)) {
+        if ('door' === $board->getField($newX, $newY)) {
             $board->setField($newX, $newY, 'ground');
             $board->debug('Door Open[%s:%s]', $newY, $newX);
             return;

src/Compiler.php

Spacing +11 added lines, -11 removed lines

@@ -9,9 +9,9 @@ 
         $lines = explode("\n", trim($code));
         $tree = $this->buildTree($lines);
         $functions = [];
-        foreach($tree as $fn) {
+        foreach ($tree as $fn) {
             $tokens = explode(' ', $fn['line']);
-            if(!$tokens || !$fn['line']) {
+            if (!$tokens || !$fn['line']) {
                 continue;
             }
             $functions[$tokens[1]] = $this->compileTree($board, $fn['sub']);
@@ -23,16 +23,16 @@ 
     private function compileTree(Board $board, array $tree): array
     {
         $code = [];
-        while(true) {
-            if(!$tree) {
+        while (true) {
+            if (!$tree) {
                 break;
             }
             $line = array_shift($tree);
             $tokens = explode(' ', $line['line']);
-            if(!$tokens || !$line['line']) {
+            if (!$tokens || !$line['line']) {
                 continue;
             }
-            switch($tokens[0]) {
+            switch ($tokens[0]) {
                 case 'for': {
                     $code[] = [
                         'action' => 'for',
@@ -77,15 +77,15 @@ 
     {
         $action = array_shift($tokens);
         $number = 1;
-        if(is_numeric($action)) {
+        if (is_numeric($action)) {
             $number = $action;
             $action = array_shift($tokens);
         }
 
         $code = [];
-        for($i = 0; $i < $number; $i++) {
+        for ($i = 0; $i < $number; $i++) {
             $args = $board->getAction($action)->getArguments();
-            if(\count($args) !== \count($tokens)) {
+            if (\count($args) !== \count($tokens)) {
                 throw new \RuntimeException(sprintf('Action args %s does not match tokens %s!', json_encode($args), json_encode($tokens)));
             }
             $code[] = array_merge(['action' => $action], array_combine($args, $tokens));
@@ -97,11 +97,11 @@ 
     private function buildTree(array $lines): array
     {
         $tree = [];
-        while($lines) {
+        while ($lines) {
             $line = array_shift($lines);
             $level = $this->getLineLevel($line);
             $subLines = [];
-            while($lines && $this->getLineLevel($lines[0]) > $level) {
+            while ($lines && $this->getLineLevel($lines[0]) > $level) {
                 $subLines[] = array_shift($lines);
             }
             $tree[] = [