CoenMooij\Sudoku\Generator\PuzzleGenerator

Complexity

Total Complexity

11

Size/Duplication

Total Lines	64
Duplicated Lines	0 %

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	locationInList()	0	9	3
A	generate()	0	7	1
A	digLocations()	0	7	3
A	getRandomLocations()	0	13	3

<?php

declare(strict_types=1);

namespace CoenMooij\Sudoku\Generator;

use CoenMooij\Sudoku\Puzzle\Difficulty;
use CoenMooij\Sudoku\Puzzle\Grid;
use CoenMooij\Sudoku\Puzzle\Location;
use CoenMooij\Sudoku\Puzzle\Puzzle;
use CoenMooij\Sudoku\Validator\DigValidator;

class PuzzleGenerator
{
    /**
     * @var Grid
     */
    private $grid;

    /**
     * @var DigValidator
     */
    private $digValidator;

    public function __construct(DigValidator $digValidator)
    {
        $this->digValidator = $digValidator;
    }

    public function generate(Grid $solutionGrid, Difficulty $difficulty): Puzzle
    {
        $this->grid = $solutionGrid;
        $locationList = $this->getRandomLocations($difficulty);
        $this->digLocations($difficulty, ...$locationList);

        return new Puzzle($this->grid);
    }

    /**
     * @return Location[]
     */
    private function getRandomLocations(Difficulty $difficulty): array
    {
        $locationList = [];
        $numberOfHoles = $difficulty->getNumberOfHoles();
        for ($i = 0; $i < $numberOfHoles; $i++) {
            do {
                $location = new Location(random_int(0, 8), random_int(0, 8));
            } while ($this->locationInList($location, ...$locationList));

            $locationList[] = $location;
        }

        return $locationList;
    }

    private function digLocations(Difficulty $difficulty, Location ...$locationList): void
    {
        $bound = $difficulty->getBound();

        foreach ($locationList as $location) {
            if ($this->digValidator->isDiggableAndUniquelySolvableAfterDigging($this->grid, $location, $bound)) {
                $this->grid->empty($location);
            }
        }
    }

    private function locationInList(Location $needle, Location ...$locationList): bool

The method locationInList() is not used, and could be removed.

    {
        foreach ($locationList as $location) {
            if (Location::match($needle, $location)) {
                return true;
            }
        }

        return false;
    }
}