JMGQ\AStar\Benchmark\BenchmarkRunner::run()

Complexity

Conditions	3
Paths	3

Size

Total Lines	33
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	19
CRAP Score	3

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function run(array $sizes, $iterations, $seed)
    {
        $results = array();

        $steps = count($sizes) * $iterations;
        $this->progressBar->start($steps);

        foreach ($sizes as $size) {
            for ($i = 0; $i < $iterations; $i++) {
                $terrain = $this->terrainGenerator->generate($size, $size, $seed);
                $aStar = new MyAStar($terrain);

                $start = new MyNode(0, 0);
                $goal = new MyNode($size - 1, $size - 1);

                $stopwatch = new Stopwatch();
                $stopwatch->start('benchmark');

                $solution = $aStar->run($start, $goal);

                $event = $stopwatch->stop('benchmark');

                $solutionFound = !empty($solution);

                $results[] = new Result($size, $event->getDuration(), $solutionFound);

It seems like $event->getDuration() can also be of type double; however, parameter $duration of JMGQ\AStar\Benchmark\Result\Result::__construct() does only seem to accept integer, maybe add an additional type check?

                $this->progressBar->advance();
            }
        }

        $this->progressBar->finish();

        return $results;
    }