StreamIterator - Code Metrics - Inspection of "added new tests" - gyselroth/php-stream-iterator - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 5f8ab7...343914 )

by Raffael

created 2018-10-25 08:20 UTC

StreamIterator A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	247
Duplicated Lines	0 %

Test Coverage

Coverage

89.61%

Importance

Changes

Metric	Value
eloc	65
dl	0
loc	247
ccs	69
cts	77
cp	0.8961
rs	9.76
c	0
b	0
f	0
wmc	33

16 Methods

Rating	Name	Size	Complexity
A	rewind()	6	1
A	isSeekable()	3	1
A	isReadable()	3	1
A	getContents()	15	3
A	__construct()	7	2
A	write()	3	1
A	getMetadata()	3	2
A	__toString()	5	1
A	isWritable()	3	1
A	eof()	7	2
A	tell()	3	1
A	getSize()	7	2
A	close()	2	1
A	detach()	6	1
A	read()	17	4
B	seek()	30	9

<?php

declare(strict_types=1);

/**
 * Stream iterator
 *
 * @copyright   Copryright (c) 2018 gyselroth GmbH (https://gyselroth.com)
 * @license     MIT https://opensource.org/licenses/MIT
 */

namespace StreamIterator;

use Closure;
use Countable;
use IteratorAggregate;
use Psr\Http\Message\StreamInterface;
use Traversable;

/**
 * Wraps an interator to iterate through and cast each entry to string via a callback.
 */
class StreamIterator implements StreamInterface
{
    /**
     * @var Traversable
     */
    private $iterator;

    /**
     * Current position in iterator.
     *
     * @var int
     */
    private $position = 0;

    /**
     * Stringify callback.
     *
     * @var Closure
     */
    private $stringify;

    /**
     * Construct a stream instance using an iterator.
     *
     * If the iterator is an IteratorAggregate, pulls the inner iterator
     * and composes that instead, to ensure we have access to the various
     * iterator capabilities.
     */
    public function __construct(Traversable $iterator, Closure $stringify = null)
    {
        if ($iterator instanceof IteratorAggregate) {
            $iterator = $iterator->getIterator();
        }
        $this->iterator = $iterator;
        $this->stringify = $stringify;
    }

    /**
     * @return string
     */
    public function __toString()
    {
        $this->iterator->rewind();
                         rewind();

        return $this->getContents();
    }

    /**
     * No-op.
     */
    public function close()
    {
    }

    /**
     * @return null|Traversable
     */
    public function detach()
    {
        $iterator = $this->iterator;
        $this->iterator = null;

        return $iterator;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
    }

    /**
     * @return null|int returns the size of the iterator, or null if unknown
     */
    public function getSize()
    {
        if ($this->iterator instanceof Countable) {
            return count($this->iterator);
        }

        return null;
    }

    /**
     * @return int Position of the iterator
     */
    public function tell()
    {
        return $this->position;
    }

    /**
     * End of File.
     *
     * @return bool
     */
    public function eof()
    {
        if ($this->iterator instanceof Countable) {
            return $this->position === count($this->iterator);
        }

        return !$this->iterator->valid();
        return !$this->iterator->/** @scrutinizer ignore-call */ valid();
    }

    /**
     * Check if seekable.
     *
     * @return bool
     */
    public function isSeekable()
    {
        return true;
    }

    /**
     * Seek the iterator.
     *
     * @param int $offset Stream offset
     * @param int $whence ignored
     *
     * @return bool returns TRUE on success or FALSE on failure
     */
    public function seek($offset, $whence = SEEK_SET)
    {
        if (!is_int($offset) && !is_numeric($offset)) {

            return false;
        }

        $offset = (int) $offset;
        if ($offset < 0) {
            return false;
        }

        $key = $this->iterator->key();
        $key = $this->iterator->key();
        if (!is_int($key) && !is_numeric($key)) {
            $key = 0;
            $this->iterator->rewind();
        }

        if ($key >= $offset) {
            $key = 0;
            $this->iterator->rewind();
        }

        while ($this->iterator->valid() && $key < $offset) {
            $this->iterator->next();
                             next();
            ++$key;
        }

        $this->position = $key;

        return true;
    }

    /**
     * @see seek()
     *
     * @return bool returns true on success or false on failure
     */
    public function rewind()
    {
        $this->iterator->rewind();
        $this->position = 0;

        return true;
    }

    /**
     * Non-writable.
     *
     * @return bool Always returns false
     */
    public function isWritable()
    {
        return false;
    }

    /**
     * Non-writable.
     *
     * @param string $string the string that is to be written
     *
     * @return bool|int Always returns false
     */
    public function write($string)
    {
        return false;
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
    }

    /**
     * @return bool
     */
    public function isReadable()
    {
        return true;
    }

    /**
     * Read content from iterator with a lenght limit (number of entries).
     *
     * @param int $length Read up to $length items from the iterator
     *
     * @return string
     */
    public function read($length)
    {
        $index = 0;
        $contents = '';
        while ($this->iterator->valid() && $index < $length) {
            if ($this->stringify !== null) {
                $contents .= $this->stringify->call($this, $this->iterator->current());
                $contents .= $this->stringify->call($this, $this->iterator->/** @scrutinizer ignore-call */ current());
            } else {
                $contents .= $this->iterator->current();
            }

            $this->iterator->next();
            ++$this->position;
            ++$index;
        }

        return $contents;
    }

    /**
     * @return string
     */
    public function getContents()
    {
        $contents = '';
        while ($this->iterator->valid()) {
            if ($this->stringify !== null) {
                $contents .= $this->stringify->call($this, $this->iterator->current());
            } else {
                $contents .= $this->iterator->current();
            }

            $this->iterator->next();
            ++$this->position;
        }

        return $contents;
    }

    /**
     * @param string $key specific metadata to retrieve
     *
     * @return null|array returns an empty array if no key is provided, and
     *                    null otherwise
     */
    public function getMetadata($key = null)
    {
        return ($key === null) ? [] : null;
    }
}


gyselroth / php-stream-iterator

Push — master ( 5f8ab7...343914 )

StreamIterator A

Complexity

Size/Duplication

Test Coverage

Importance

16 Methods

Duplication Side-by-Side

Filter issues like