HintGenerator::generateOne() - Code Metrics - coenmooij/sudoku - Measure and Improve Code Quality continuously with Scrutinizer

HintGenerator::generateOne() A
last analyzed 2018-10-08 11:33 UTC

↳ Parent: HintGenerator

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	5
dl	0
loc	9
c	0
b	0
f	0
rs	10
cc	2
nc	2
nop	1

<?php

declare(strict_types=1);

namespace CoenMooij\Sudoku\Generator;

use CoenMooij\Sudoku\Exception\UnsolvableException;
use CoenMooij\Sudoku\Puzzle\Grid;
use CoenMooij\Sudoku\Puzzle\Location;

class HintGenerator
{
    public function generateOne(Grid $grid): Location
    {
        $locationList = $this->generateAll($grid);
        if (empty($locationList)) {
            throw new UnsolvableException();
        }
        shuffle($locationList);

        return $locationList[0];
    }

    /**
     * @return Location[]
     */
    public function generateAll(Grid $grid): array
    {
        $locations = [];
        foreach ($this->getLocations() as $location) {
            if ($this->hasOnePossibleValue($grid, $location)) {
                $locations[] = $location;
            }
        }

        return $locations;
    }

    /**
     * @return Location[]
     */
    private function getLocations(): array
    {
        $locations = [];
        for ($row = 0; $row < Grid::NUMBER_OF_ROWS; $row++) {
            for ($column = 0; $column < Grid::NUMBER_OF_COLUMNS; $column++) {
                $locations[] = new Location($row, $column);
            }
        }

        return $locations;
    }

    private function hasOnePossibleValue(Grid $grid, Location $location): bool
    {
        $possibleValues = $grid->getAllPossibilitiesFor($location);

        return count($possibleValues) === 1;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace CoenMooij\Sudoku\Generator;
6
7			use CoenMooij\Sudoku\Exception\UnsolvableException;
8			use CoenMooij\Sudoku\Puzzle\Grid;
9			use CoenMooij\Sudoku\Puzzle\Location;
10
11			class HintGenerator
12			{
13			public function generateOne(Grid $grid): Location
14			{
15			$locationList = $this->generateAll($grid);
16			if (empty($locationList)) {
17			throw new UnsolvableException();
18			}
19			shuffle($locationList);
20
21			return $locationList[0];
22			}
23
24			/**
25			* @return Location[]
26			*/
27			public function generateAll(Grid $grid): array
28			{
29			$locations = [];
30			foreach ($this->getLocations() as $location) {
31			if ($this->hasOnePossibleValue($grid, $location)) {
32			$locations[] = $location;
33			}
34			}
35
36			return $locations;
37			}
38
39			/**
40			* @return Location[]
41			*/
42			private function getLocations(): array
43			{
44			$locations = [];
45			for ($row = 0; $row < Grid::NUMBER_OF_ROWS; $row++) {
46			for ($column = 0; $column < Grid::NUMBER_OF_COLUMNS; $column++) {
47			$locations[] = new Location($row, $column);
48			}
49			}
50
51			return $locations;
52			}
53
54			private function hasOnePossibleValue(Grid $grid, Location $location): bool
55			{
56			$possibleValues = $grid->getAllPossibilitiesFor($location);
57
58			return count($possibleValues) === 1;
59			}
60			}
61

coenmooij / sudoku

HintGenerator::generateOne() A last analyzed 2018-10-08 11:33 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

HintGenerator::generateOne() A
last analyzed 2018-10-08 11:33 UTC