CondorcetPHP\Condorcet\Algo\Method

Complexity

Total Complexity

6

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	3
A	getResult()	0	11	2
	getStats()	0	1	?
A	createResult()	0	10	1

<?php
/*
    Condorcet PHP - Election manager and results calculator.
    Designed for the Condorcet method. Integrating a large number of algorithms extending Condorcet. Expandable for all types of voting systems.

    By Julien Boudry and contributors - MIT LICENSE (Please read LICENSE.txt)
    https://github.com/julien-boudry/Condorcet
*/
declare(strict_types=1);

namespace CondorcetPHP\Condorcet\Algo;

use CondorcetPHP\Condorcet\{CondorcetVersion, Election, Result};
use CondorcetPHP\Condorcet\Throwable\CondorcetException;

// Generic for Algorithms
abstract class Method
{
    use CondorcetVersion;

    public static $MaxCandidates = null;

    protected $_selfElection;
    protected $_Result;

    public function __construct (Election $mother)
    {
        $this->_selfElection = $mother;

        if (!is_null(static::$MaxCandidates) && $this->_selfElection->countCandidates() > static::$MaxCandidates) :
            throw new CondorcetException(101, static::METHOD_NAME[0].' is configured to accept only '.static::$MaxCandidates.' candidates');
        endif;
    }

    public function getResult () : Result
    {
        // Cache
        if ( $this->_Result !== null ) :
            return $this->_Result;
        endif;

        $this->compute();

It seems like you code against a specific sub-type and not the parent class CondorcetPHP\Condorcet\Algo\Method as the method compute() does only exist in the following sub-classes of CondorcetPHP\Condorcet\Algo\Method: CondorcetPHP\Condorcet\Algo\Methods\Borda\BordaCount, CondorcetPHP\Condorcet\Algo\Methods\Borda\DowdallSystem, CondorcetPHP\Condorcet\Algo\Methods\Dodgson\DodgsonQuick, CondorcetPHP\Condorcet\Algo\Methods\Ftpt\Ftpt, CondorcetPHP\Condorcet\Algo\Methods\InstantRunoff\InstantRunoff. Maybe you want to instanceof check for one of these explicitly?

        return $this->_Result;
    }

    abstract protected function getStats () : array;

    protected function createResult (array $result) : Result
    {
    	return new Result (
            static::METHOD_NAME[0],
            get_class($this),
    		$this->_selfElection,
    		$result,
            $this->getStats()
    	);
    }
}