Germania\Salesmen\PdoActiveSalesmen

Complexity

Total Complexity

7

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	33	2
A	has()	0	3	1
A	get()	0	8	2
A	getIterator()	0	4	1
A	count()	0	4	1

<?php
namespace Germania\Salesmen;

use Psr\Container\ContainerInterface;

class PdoActiveSalesmen implements ContainerInterface, \IteratorAggregate, \Countable
{
    /**
     * @var \PDO
     */
    public $pdo;

    public $table = "salesmen";

    public $salesmen = array();

    public function __construct (\PDO $pdo, $table, SalesmanInterface $salesman = null)
    {
        $this->table    = $table;
        $this->pdo      = $pdo;

        // aussendienst_nummer ID is listed twice here in order to use it with FETCH_UNIQUE as array key
        $sql = "SELECT
        aussendienst_nummer,
        aussendienst_nummer as salesman_id,
        aussendienst_vorname as first_name,
        aussendienst_nachname as last_name,
        aussendienst_retailer_number as retailer_number,
        aussendienst_email as email,
        user_id,
        is_active

        FROM {$this->table}

        WHERE is_active > 0

        ORDER BY
        aussendienst_nachname ASC,
        aussendienst_vorname ASC,
        aussendienst_nummer ASC";

        $this->stmt = $pdo->prepare( $sql );

The property stmt does not exist. Did you maybe forget to declare it?

        $this->stmt->setFetchMode( \PDO::FETCH_CLASS, $salesman ? get_class($salesman) : Salesman::class );

        $bool = $this->stmt->execute();

$bool is not used, you could remove the assignment.

        $this->salesmen = $this->stmt->fetchAll( \PDO::FETCH_UNIQUE);
    }


    /**
     * @implements ContainerInterface
     */
    public function has ($aussendienst_nummer) {
        return array_key_exists($aussendienst_nummer, $this->salesmen);
    }


    /**
     * @implements ContainerInterface
     */
    public function get ($aussendienst_nummer) {
        if (!$this->has($aussendienst_nummer)) {
            $msg = sprintf("Could not find Salesman with ADM-Nummer '%s'", $aussendienst_nummer);
            throw new SalesmanNotFoundException( $msg );
        }

        return $this->salesmen[$aussendienst_nummer];
    }


    /**
     * @return Iterator
     */
    public function getIterator()
    {
        return new \ArrayIterator( $this->salesmen );
    }


    /**
     * @return int
     */
    public function count()
    {
        return count( $this->salesmen );
    }
}