iio\libmergepdf\Pages

Complexity

Total Complexity

10

Size/Duplication

Total Lines	94
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	22	5
A	addPage()	0	5	1
A	addRange()	0	6	1
A	getPages()	0	4	1
A	getIterator()	0	4	1
A	hasPages()	0	4	1

<?php

namespace iio\libmergepdf;

/**
 * Parse page numbers from string
 */
class Pages implements \IteratorAggregate
{
    /**
     * @var integer[] Added integer page numbers
     */
    private $pages = [];

    /**
     * Constructor
     *
     * Pages should be formatted as 1,3,6 or 12-16 or combined. Note that pages
     * are merged in the order that you provide them. If you put pages 12-14
     * before 1-5 then 12-14 will be placed first.
     *
     * @param  string    $expressionString
     * @throws Exception If unable to parse page numbers
     */
    public function __construct($expressionString = '')
    {
        $expressions = explode(
            ',',
            str_replace(' ', '', $expressionString)
        );

        foreach ($expressions as $expr) {
            if (empty($expr)) {
                continue;
            }
            if (ctype_digit($expr)) {
                $this->addPage($expr);
                continue;
            }
            if (preg_match("/^(\d+)-(\d+)/", $expr, $matches)) {
                $this->addRange($matches[1], $matches[2]);
                continue;
            }
            throw new Exception("Invalid page number(s) for expression '$expr'");
        }
    }

    /**
     * Add page to collection
     *
     * @param  int|string $page
     * @return void
     */
    public function addPage($page)
    {
        assert('is_numeric($page)');
        $this->pages[] = intval($page);
    }

    /**
     * Add range of pages
     *
     * @param  int|string $start
     * @param  int|string $end
     * @return void
     */
    public function addRange($start, $end)
    {
        assert('is_numeric($start)');
        assert('is_numeric($end)');
        $this->pages = array_merge($this->pages, range($start, $end));

It seems like array_merge($this->pages, range($start, $end)) of type array is incompatible with the declared type array<integer,integer> of property $pages.

    }

    /**
     * Get array of integer page numbers
     *
     * @return integer[]
     */
    public function getPages()
    {
        return $this->pages;
    }

    /**
     * Get iterator of page numbers
     */
    public function getIterator()
    {
        return new \ArrayIterator($this->getPages());
    }

    /**
     * Check if pages has been sprcified
     *
     * @return boolean
     */
    public function hasPages()
    {
        return !empty($this->getPages());
    }
}